Merge pull request #71 from mseele/master

fix: make Video annotation optional, not required
SergeyPirogov · Jul 13, 2022 · 5f7ff89 · 5f7ff89
2 parents 0e301f7 + 7ccce41
commit 5f7ff89
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/junit5/src/main/java/com/automation/remarks/junit5/VideoExtension.java b/junit5/src/main/java/com/automation/remarks/junit5/VideoExtension.java
@@ -49,10 +49,7 @@ public void afterTestExecution(ExtensionContext context) throws Exception {
   private boolean videoDisabled(Method testMethod) {
     Optional<com.automation.remarks.video.annotations.Video> video = AnnotationUtils.findAnnotation(testMethod, com.automation.remarks.video.annotations.Video.class);
 
-    return video.map(v -> !videoEnabled(v))
-            .orElseGet(() -> true);
-
-    //return !video.isPresent() && !videoEnabled(video.get());
+    return !videoEnabled(video.orElse(null));
   }
 
   private String getFileName(Method testMethod) {