In all cases these updates address the fact that go-cmp requires the caller to specify explicitly how it should handle unexported fields; cmp.Diff will panic if it encounters an unexpected unexported field. In many cases we encounter unexported fields while comparing errors types from github.com/pkg/error. I've added an EquateErrors() comparer that maintains the error comparison semantics of https://github.com/go-test/deep. In most other cases I've also attempted to maintain the comparison semantics of https://github.com/go-test/deep, which is to ignore unexported fields. In cases where the test appeared to intend to compare the unexported fields of a type under our control, and where comparing those unexported fields did not break the test, I updated the test to compare unexported fields. In cases where the test appeared to intend to compare unexported fields, but doing so broke the test, I added a BUG comment. Signed-off-by: Nic Cope <negz@rk0n.org> |
||
---|---|---|
.. | ||
application | ||
resource | ||
scheduler |