golang / mock

GoMock is a mocking framework for the Go programming language.
Apache License 2.0
9.3k stars 610 forks source link

Built-in matcher that compares slices of arbitrary elements order #545

Closed vvkh closed 3 years ago

vvkh commented 3 years ago

Requested feature

It would nice to have built-in matcher that compares slices and returns true if slices have same elements in arbitrary order.

Why the feature is needed

Sometimes code we are testing produces slices with non-determined elements order. For example, when converting map to slice we may end up with many possible orders of elements.

s := make([]interface{}, 0, len(m))
for k, v := range m {
    s = append(s, v)
}

Some testing frameworks like testify include special assert function checking if two slices have same elements.

If we have a mock depending on arguments with non-determined elements order such as s then setting up expected calls for such mock can be quite inconvenient. We either have to use Any() matcher and skip validating this argument at all or implement custom matcher.

Proposed solution

Include built-in matcher like InAnyOrder(s []interface{}) or WithSameElements(s []interface{} implementing checks similar to assert.ElementsMatch

P.S. I will be happy to implement the matcher if we agree on this feature being useful.

codyoss commented 3 years ago

Hey, thanks for the feature request. I will let this issue sit for a little bit to see what others in the community think of the idea. I could see how a matcher like this could be useful.

vvkh commented 3 years ago

Hey @codyoss. I've provided draft implementation for requested issue. Could you please take a look?

vvkh commented 3 years ago

Implemented in https://github.com/golang/mock/pull/546