Details
-
Improvement
-
Status: Open
-
Major
-
Resolution: Unresolved
-
None
-
None
-
None
Description
FINERACT-1209 introduce a new :integration-tests module (big refactoring).
In https://github.com/apache/fineract/pull/1465/files aleks had to add a new suppression (exclusion) for /integration-tests/ to config/checkstyle/suppressions.xml. (Apparently the integration tests were not yet covered by Checkstyle even in the old location?)
The goal of this issue is to remove that suppression and enable Checkstyle on ITs.
An intermediate solution could be to not fully enable it like on main code, but not entirely disable it either, but just start by enabling the CS rules which already pass for ITs, maybe fix a few trivial ones, but leave those with many violations for separate future fixing.
manthan FYI
Attachments
Issue Links
- relates to
-
FINERACT-1209 Integration Tests using new Swagger Client API ("Fineract SDK")
- Resolved