Skip to content
This repository was archived by the owner on Jul 22, 2020. It is now read-only.

Comments

More tests#145

Merged
prymitive merged 2 commits intomasterfrom
more-tests
Jul 18, 2017
Merged

More tests#145
prymitive merged 2 commits intomasterfrom
more-tests

Conversation

@prymitive
Copy link
Contributor

No description provided.

Depending on exact value breaks when testing with a different Go version since implementation details might cause structhash to return a different hash. We don't depend on exact values in the UI, we only require that those values are unique to a unique set of labels and selected attributes, so that's what we should test rather than hardcoded values. This PR allows unsee to pass tests under Go 1.9beta2
@prymitive prymitive assigned Tenzer and jamesog and unassigned Tenzer and jamesog Jul 18, 2017
@prymitive prymitive requested review from Tenzer and jamesog July 18, 2017 02:56
@prymitive prymitive merged commit ac60d6d into master Jul 18, 2017
@prymitive prymitive deleted the more-tests branch July 18, 2017 16:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants