|
From: Andreas D. <no...@gi...> - 2025-10-23 14:37:59
|
Branch: refs/heads/main Home: https://github.com/pmd/pmd-regression-tester Commit: ae12930b0e5daaaa81843cac0b0fc6d09f5b4c6f https://github.com/pmd/pmd-regression-tester/commit/ae12930b0e5daaaa81843cac0b0fc6d09f5b4c6f Author: Andreas Dangel <and...@pm...> Date: 2025-10-10 (Fri, 10 Oct 2025) Changed paths: M lib/pmdtester/builders/rule_set_builder.rb A test/resources/rule_set_builder/expected-abstractclasswithoutabstractmethod.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-apex/src/main/resources/category/apex/bestpractices.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/bestpractices.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/codestyle.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/design.xml M test/test_rule_set_builder.rb Log Message: ----------- Generate ruleset for known rules With the previous fix for #124, if a rule named "Abstract..." has been changed, we would create a regression report for all rules. This PR now changes the implementation to create a list of all rules with there java source file name. Commit: 9cb700b3d068c45e330d5810007c1404d3daf8b4 https://github.com/pmd/pmd-regression-tester/commit/9cb700b3d068c45e330d5810007c1404d3daf8b4 Author: Andreas Dangel <and...@pm...> Date: 2025-10-10 (Fri, 10 Oct 2025) Changed paths: M lib/pmdtester/builders/pmd_report_builder.rb M test/test_pmd_report_builder.rb Log Message: ----------- Skip tests completely while building PMD This should make the build a tiny bit faster. Commit: b5c89c69f6bd282a66500744f248f96a08fe1e19 https://github.com/pmd/pmd-regression-tester/commit/b5c89c69f6bd282a66500744f248f96a08fe1e19 Author: Andreas Dangel <and...@pm...> Date: 2025-10-10 (Fri, 10 Oct 2025) Changed paths: M History.md Log Message: ----------- Update History.md Commit: 33891a15915a668ef21d6a7f76754c2d05aea4d4 https://github.com/pmd/pmd-regression-tester/commit/33891a15915a668ef21d6a7f76754c2d05aea4d4 Author: Andreas Dangel <and...@pm...> Date: 2025-10-10 (Fri, 10 Oct 2025) Changed paths: M lib/pmdtester/builders/pmd_report_builder.rb M test/test_pmd_report_builder.rb Log Message: ----------- Revert "Skip tests completely while building PMD" We need to compile tests, as pmd-ant is depending on the test artifacts from pmd-core... This reverts commit 9cb700b3d068c45e330d5810007c1404d3daf8b4. Commit: 5b4294b40ee3248a0ee9ecc3597088f0414b1936 https://github.com/pmd/pmd-regression-tester/commit/5b4294b40ee3248a0ee9ecc3597088f0414b1936 Author: Andreas Dangel <and...@pm...> Date: 2025-10-23 (Thu, 23 Oct 2025) Changed paths: M History.md M lib/pmdtester/builders/rule_set_builder.rb A test/resources/rule_set_builder/expected-abstractclasswithoutabstractmethod.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-apex/src/main/resources/category/apex/bestpractices.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/bestpractices.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/codestyle.xml A test/resources/rule_set_builder/partial-pmd-repo/pmd-java/src/main/resources/category/java/design.xml M test/test_rule_set_builder.rb Log Message: ----------- Support `--auto-gen-config` for rules which start with "Abstract" (#142) This solves #124 slightly differently: As we know, which rules exist, we use that information and build up a map of rule java source files to rule references. E.g. in pmd-java/src/main/categories/java/bestpractices.xml we see, that the rule "AbstractClassWithoutAbstractMethod" is implemented in class "net.sourceforge.pmd.lang.java.rule.bestpractices.AbstractClassWithoutAbstractMethodRule". For this class name we can guess the corresponding java source (.../src/main/java/...). So, when we see this java source in the changes, then we know, we need to regression test that rule. This means, we don't guess anymore, whether a java source file is a rule implementation - we have a list of all rule implementations that we can check now. This also fixes the integration test (https://github.com/pmd/pmd-regression-tester/actions/workflows/manual-integration-tests.yml): In test case 1 we changed the rule AbstractClassWithoutAbstractMethod and we expect, that we only see differences from this rule. As other rules won't be executed, we expected that we get less errors - but with the change from #141 now all rules have been executed again. Refs - #124 - #141 Compare: https://github.com/pmd/pmd-regression-tester/compare/c024e8a481b1...5b4294b40ee3 To unsubscribe from these emails, change your notification settings at https://github.com/pmd/pmd-regression-tester/settings/notifications |
