You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We need to increase test coverage in the core module, on which all other modules depend
Current Behavior
The current coverage for the core module is at 76%. Room for improvement.
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.
-The code has a lot of static methods, supposed to make use of the api easier. This consists the bulk of useful untested code. We could instead initiate such classes using double-check locking singletons, turn the static methods to instance methods.
Use dependency injection pattern instead of configuring dependencies inside the classes. This would make the code more testable and maintainable.
As much as possible reduce side effects
The text was updated successfully, but these errors were encountered:
Expected Behavior
We need to increase test coverage in the core module, on which all other modules depend
Current Behavior
The current coverage for the core module is at 76%. Room for improvement.
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.
-The code has a lot of static methods, supposed to make use of the api easier. This consists the bulk of useful untested code. We could instead initiate such classes using double-check locking singletons, turn the static methods to instance methods.
The text was updated successfully, but these errors were encountered: