Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write test for 'MetaProperty.hashCode' #8

Open
nipafx opened this issue Jan 4, 2015 · 0 comments
Open

Write test for 'MetaProperty.hashCode' #8

nipafx opened this issue Jan 4, 2015 · 0 comments

Comments

@nipafx
Copy link
Collaborator

nipafx commented Jan 4, 2015

The method MetaProperty.hashCode() is currently not tested. This issue can be resolved by ...

  • ... writing meaningful tests or
  • ... coming to the conclusion that no tests are necessary / reasonable, in which case this should be documented in the test class

The corresponding ToDo should be removed.

Some Pointers

A short discussion about what a test could look like can be found here.

Tests should be added to AbstractMetaPropertyTest if possible. It might make sense to take a look at the tests for equals() which use a special mock to ensure that only defined fields are used for equality comparison.

(The links above point to a specific version and should be compared to the current version before starting to work on this issue.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant