[Number of Public Attributes Check] Should ignore local test doubles

💡 Code pal for ABAP follows the Clean ABAP. If the issue relates to coding style, please submit it here.

Check Name
Number of Public Attributes Check

Actual Behavior
The current check also covers and reports findings for local test doubles, like the following:

CLASS ltd_my_double DEFINITION FOR TESTING.

  PUBLIC SECTION.
    INTERFACES if_my_interface.

    DATA mv_my_injected_attribute TYPE string.
ENDCLASS.

As an explanation: According to this test design pattern, the double behavior is controlled by these attributes. In addition, getter/setters are not needed due to the locality and the scope of the test fixture.

Still, the application of this pattern is reported by the check.

Expected Behavior
The check should ignore local classes for testing.

Locate the implementation of the Number of Public Attributes Check and the tests covering local test doubles. Start by tracing how local classes declared FOR TESTING are represented, then verify that their public attributes are excluded while ordinary classes remain checked. Done means the regression behavior is covered by tests.