SpringBoot2 - JUnit5 - skip test - next improvement

Author: bvn13

springboot2-junit5-skiptest/README.md

@@ -176,22 +176,22 @@ There is no way avoiding new annotation processing:
 public class TestEnabledCondition implements ExecutionCondition {
 
     static class AnnotationDescription {
-        String name;
-        Boolean value;
-        AnnotationDescription(String prefix, String property) {
-            this.name = prefix + property;
+            String name;
+            Boolean annotationEnabled;
+            AnnotationDescription(String prefix, String property) {
+                this.name = prefix + property;
+            }
+            String getName() {
+                return name;
+            }
+            AnnotationDescription setAnnotationEnabled(Boolean value) {
+                this.annotationEnabled = value;
+                return this;
+            }
+            Boolean isAnnotationEnabled() {
+                return annotationEnabled;
+            }
         }
-        String getName() {
-            return name;
-        }
-        AnnotationDescription setValue(Boolean value) {
-            this.value = value;
-            return this;
-        }
-        Boolean getValue() {
-            return value;
-        }
-    }
 
     /* ... */
 }
@@ -235,9 +235,9 @@ public class TestEnabledCondition implements ExecutionCondition {
                 .map(e -> e.getAnnotation(TestEnabled.class))
                 .map(TestEnabled::property)
                 .map(property -> makeDescription(context, property))
-                .map(description -> description.setValue(environment.getProperty(description.getName(), Boolean.class)))
+                .map(description -> description.setAnnotationEnabled(environment.getProperty(description.getName(), Boolean.class)))
                 .map(description -> {
-                    if (Boolean.TRUE.equals(description.getValue())) {
+                    if (description.isAnnotationEnabled()) {
                         return ConditionEvaluationResult.enabled("Enabled by property: "+description.getName());
                     } else {
                         return ConditionEvaluationResult.disabled("Disabled by property: "+description.getName());

springboot2-junit5-skiptest/src/test/java/com/bvn13/example/springboot/junit/skiptest/TestEnabledCondition.java

@@ -15,19 +15,19 @@ public class TestEnabledCondition implements ExecutionCondition {
 
     static class AnnotationDescription {
         String name;
-        Boolean value;
+        Boolean annotationEnabled;
         AnnotationDescription(String prefix, String property) {
             this.name = prefix + property;
         }
         String getName() {
             return name;
         }
-        AnnotationDescription setValue(Boolean value) {
-            this.value = value;
+        AnnotationDescription setAnnotationEnabled(Boolean value) {
+            this.annotationEnabled = value;
             return this;
         }
-        Boolean getValue() {
-            return value;
+        Boolean isAnnotationEnabled() {
+            return annotationEnabled;
         }
     }
 
@@ -48,9 +48,9 @@ public ConditionEvaluationResult evaluateExecutionCondition(ExtensionContext con
                 .map(e -> e.getAnnotation(TestEnabled.class))
                 .map(TestEnabled::property)
                 .map(property -> makeDescription(context, property))
-                .map(description -> description.setValue(environment.getProperty(description.getName(), Boolean.class)))
+                .map(description -> description.setAnnotationEnabled(environment.getProperty(description.getName(), Boolean.class)))
                 .map(description -> {
-                    if (Boolean.TRUE.equals(description.getValue())) {
+                    if (description.isAnnotationEnabled()) {
                         return ConditionEvaluationResult.enabled("Enabled by property: "+description.getName());
                     } else {
                         return ConditionEvaluationResult.disabled("Disabled by property: "+description.getName());