GODRIVER-1893 Cleanup comments and tests

Author: Divjot Arora

bson/primitive/objectid.go

@@ -88,14 +88,10 @@ func ObjectIDFromHex(s string) (ObjectID, error) {
 	return oid, nil
 }
 
-// IsValidObjectID judges given string format is valid or invalid for ObjectID.
+// IsValidObjectID returns true if the provided hex string represents a valid ObjectID and false if not.
 func IsValidObjectID(s string) bool {
 	_, err := ObjectIDFromHex(s)
-	if err != nil {
-		return false
-	}
-
-	return true
+	return err == nil
 }
 
 // MarshalJSON returns the ObjectID as a string

bson/primitive/objectid_test.go

@@ -63,7 +63,7 @@ func TestIsValidObjectID(t *testing.T) {
 
 	for _, testcase := range testCases {
 		got := IsValidObjectID(testcase.givenID)
-		require.Equal(t, testcase.expected, got)
+		assert.Equal(t, testcase.expected, got, "expected hex string to be valid ObjectID: %v, got %v", testcase.expected, got)
 	}
 }