fixed empty filter situation (via #16)

eroshenkoam · Dec 10, 2020 · 2a5f792 · 2a5f792
1 parent d0f44c4
commit 2a5f792
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/main/java/io/github/eroshenkoam/allure/AllurePDFGenerator.java b/src/main/java/io/github/eroshenkoam/allure/AllurePDFGenerator.java
@@ -54,7 +54,9 @@ public AllurePDFGenerator(final String reportName, final Path reportPath) {
     }
 
     public void filter(final Map<String, String> tags) {
-        this.filter.putAll(tags);
+        if (tags != null) {
+            this.filter.putAll(tags);
+        }
     }
 
     public void generate(final Path outputPath) throws IOException {
@@ -98,9 +100,8 @@ public void generate(final Path outputPath) throws IOException {
                 boolean shouldSkip = Objects.nonNull(filter)
                         && !filter.isEmpty()
                         && !filter.entrySet().stream().allMatch(e -> labels.getOrDefault(e.getKey(), new ArrayList<>()).contains(e.getValue()));
-                if (!shouldSkip) {
-                    printTestResultDetails(document, result, fontHolder);
-                }
+                if (shouldSkip) continue;
+                printTestResultDetails(document, result, fontHolder);
             }
         }
     }