Test: Violations in existing files

pmd/pmd-github-action#59

.github/workflows/build.yml

@@ -13,10 +13,10 @@ jobs:
       fail-fast: false
     steps:
     - uses: actions/checkout@v2
-    - uses: pmd/pmd-github-action@v1
+    #- uses: pmd/pmd-github-action@v1
     #- uses: pmd/pmd-github-action@main
     #- uses: pmd/pmd-github-action@features/annotation
-    #- uses: adangel/pmd-github-action@analyze-modified-files-only
+    - uses: adangel/pmd-github-action@issue-51-annotations-windows
       id: pmd
       with:
         rulesets: 'ruleset.xml,category/java/design.xml/AbstractClassWithoutAnyMethod'

src/main/java/AvoidCatchingThrowableSample.java

@@ -6,4 +6,12 @@ public void bar() {
             th.printStackTrace();
         }
     }
+
+    public void foo() {
+        try {
+            // do something
+        } catch (Throwable th) {  // should be reported as "AvoidCatchingThrowable" (prio 2, failure)
+            th.printStackTrace();
+        }
+    }
 }

src/main/java/ReturnFromFinallyBlockSample.java

@@ -8,4 +8,14 @@ public String bar() {
             return "A. O. K."; // should be reported as "ReturnFromFinallyBlock" (prio 3, warning)
         }
     }
+
+    public String foo() {
+        try {
+            throw new Exception( "My Exception" );
+        } catch (Exception e) {
+            throw e;
+        } finally {
+            return "A. O. K."; // should be reported as "ReturnFromFinallyBlock" (prio 3, warning)
+        }
+    }
 }