Open TeodorPt opened 8 months ago
Hi, thanks for the issue report. It may make sense to provide an option to ignore the Equal
method.
In the mean time you can do something like:
type EntryNoMethods Entry // type definitions in Go drop methods of the parent
cmp.Equal(...,
// Transform Entry into EntryNoMethods to avoid calling the equal method.
cmp.Transformer("", func(e Entry) EntryNoMethods { return EntryNoMethods(e) }),
// Ignore the ID field on post-transformation EntryNoMethods type.
cmpopts.IgnoreFields(EntryNoMethods{}, "ID"),
)
@dsnet I ran across this suprising issue when I tried to use cmp
inside an Equal
method.
func (m *UserDistrictInfo) Equal(y *UserDistrictInfo) bool {
if y == nil {
return m == y
}
//type NoMethods UserDistrictInfo // type definitions in Go drop methods of the parent
//methodRemover := func(e *UserDistrictInfo) *NoMethods { return (*NoMethods)(e) }
return cmp.Equal(m, y,
// Transform Entry into NoMethods to avoid calling the equal method.
//cmp.Transformer("", methodRemover),
cmp.AllowUnexported(UserDistrictInfo{}),
// Ignore the fields on post-transformation NoMethods type.
cmpopts.IgnoreFields(UserDistrictInfo{}, "DataflowExportData", "ModificationTimestamp", "CreatedAt", "UpdatedAt"),
)
}
will hang forever but this:
func (m *UserDistrictInfo) Equal(y *UserDistrictInfo) bool {
if y == nil {
return m == y
}
type NoMethods UserDistrictInfo // type definitions in Go drop methods of the parent
methodRemover := func(e *UserDistrictInfo) *NoMethods { return (*NoMethods)(e) }
return cmp.Equal(m, y,
// Transform Entry into NoMethods to avoid calling the equal method.
cmp.Transformer("", methodRemover),
cmp.AllowUnexported(NoMethods{}),
// Ignore the fields on post-transformation NoMethods type.
cmpopts.IgnoreFields(NoMethods{}, "DataflowExportData", "ModificationTimestamp", "CreatedAt", "UpdatedAt"),
)
}
works reliably.
It would be nice to have a cmpopts.IgnoreEqual()
option to help document this potential pitfall and make this more generic and succinct.
Summary
If a struct has a method of the form
(T) Equal(T) bool
,cmp.Equal
is using that instead of the default evaluation process, as documented in the Godoc.If one wants to use options such as
cmpopts.IgnoreFields
, they are ignored, because the rule above takes precedence.In some situations (e.g. in tests), it might be useful to be able to ignore fields even on structs that have an
Equal
method.Example
Suggestion
An idea would be to provide an additional
Option
to ignore the struct'sEqual
method even if it exists. Please let me know if there's already a way to bypass this that I might have missed.Thanks!