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.

Sign up for GitHub

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

Jump to bottom

SetterTester and GetterTester improvement #90

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

SetterTester and GetterTester improvement #90

wants to merge 2 commits into from
+97 −1

Commits on Sep 7, 2016

  1. Introduces the possibility to skip some fields during Getter/Setter 

    validation.

Sometimes when build unit tests for legacy code, you can find bad
programming from people that put business logic inside getter/setter.

And, since you cannot refactor it, OpenPojo fails. To avoid skipping the
entire class because a field, this modification helps it.
    bobrivers committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    b2e0a20 View commit details
    Browse the repository at this point in the history

  2. Introduces the possibility to skip some fields during Getter/Setter 

    validation.

Sometimes when build unit tests for legacy code, you can find bad
programming from people that put business logic inside getter/setter.

And, since you cannot refactor it, OpenPojo fails. To avoid skipping the
entire class because a field, this modification helps it.
    bobrivers committed Sep 7, 2016
    Configuration menu
    Copy the full SHA
    988e202 View commit details
    Browse the repository at this point in the history