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

Jump to bottom

Fixing a couple of Sonar warnings, adding ITs #401

Closed

escowles wants to merge 3 commits into master from sonar-coverage

Contributor

escowles commented May 30, 2014

Moving some duplicate code to a shared superclass
Removing unreachable error handling
Adding several ITs

Addresses https://www.pivotaltracker.com/story/show/70678842

escowles added 3 commits

May 29, 2014 09:00


          Moving duplicated code to AbstractResource, removing unreachable erro…

2e046e8

…r handling, adding IT for move/copy to non-existent parent


          Removing unused predicate, adding tests

1f4d140


          Adding ITs for TransactionServiceImpl and VersionServiceImpl

587cea3

awoods commented Jun 2, 2014

Resolved with: 2501668

awoods closed this

awoods deleted the sonar-coverage branch

June 2, 2014 20:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment