ValidateError should use StringData instead of std::string

The ValidateError test in collection_test.cpp creates a std::string "\0\0\0\0\0". BF-12300 is likely caused by the assumption that this string is length 5 but strlen() returning 0. Change this to use StringData instead.