[FINERACT-1246] New :integration-tests module should have Checkstyle enabled

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Labels: Roadmap2022
• Assignee: Unassigned
• Reporter: Michael Vorburger
• Created: 31/Oct/20 13:50
• Updated: 03/Nov/22 11:40

Description

FINERACT-1209 introduce a new :integration-tests module (big refactoring).

In https://github.com/apache/fineract/pull/1465/files Aleksandar Vidakovic 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 Surkar FYI

Issue Links

• relates to: FINERACT-1209 Integration Tests using new Swagger Client API ("Fineract SDK")