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

Add tests to the adhoc module #14

Open
ghacupha opened this issue Aug 14, 2018 · 0 comments
Open

Add tests to the adhoc module #14

ghacupha opened this issue Aug 14, 2018 · 0 comments
Assignees
@ghacupha
Labels
enhancement New feature or request
Projects
Test Project

Comments

@ghacupha
Copy link
Member

ghacupha commented Aug 14, 2018 •
edited

Expected Behavior

We need to increase test coverage in the adhoc module, which seems to contain the lowest coverage so far

Current Behavior

The current coverage report shows that adhoc is now at 59.68%.

Possible Solution

  • Stick to Junit as amuch as possible. Mockito may not be necessary. We can generate class and XSDs to do round-trip tests where we are using JAXB api.
  • Replace internally configured objects with dependency injection
  • Reduce usage of static methods
@ghacupha ghacupha added the enhancement New feature or request label Aug 14, 2018
@ghacupha ghacupha self-assigned this Aug 14, 2018
@ghacupha ghacupha added this to To do in Test Project via automation Aug 14, 2018
@ghacupha ghacupha moved this from To do to In progress in Test Project Aug 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ghacupha ghacupha

Labels
enhancement New feature or request
Projects
Test Project
  
In progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ghacupha