aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIttay Stern <ittay.stern@att.com>2019-08-21 10:01:21 +0000
committerGerrit Code Review <gerrit@onap.org>2019-08-21 10:01:21 +0000
commitdb18734d1fed426ff5c6a318f3179001a606690e (patch)
treebfd53cd9600c1a56b5bf6a60fb21c0efefa3d244
parent92ebd799948bab4a5143910009e50ead1359a2d5 (diff)
parentf076ec79b9b5d4c533b30c227d1387c94b2beb3b (diff)
Merge "merge 2 TestNg transformers"
-rw-r--r--vid-automation/TestNg-ApiTest.xml3
-rw-r--r--vid-automation/TestNg-UI-half.xml3
-rw-r--r--vid-automation/TestNg-dev.xml3
-rw-r--r--vid-automation/src/main/java/vid/automation/test/infra/SkipTestUntilTestngTransformer.java57
-rw-r--r--vid-automation/src/main/java/vid/automation/test/infra/SkipTestsTestngTransformer.java (renamed from vid-automation/src/main/java/vid/automation/test/infra/FeatureTogglingTestngTransformer.java)45
5 files changed, 39 insertions, 72 deletions
diff --git a/vid-automation/TestNg-ApiTest.xml b/vid-automation/TestNg-ApiTest.xml
index 9e9380faa..641f3eba6 100644
--- a/vid-automation/TestNg-ApiTest.xml
+++ b/vid-automation/TestNg-ApiTest.xml
@@ -3,8 +3,7 @@
<suite verbose="1" name="VID API Tests" annotations="JDK">
<listeners>
- <listener class-name="vid.automation.test.infra.FeatureTogglingTestngTransformer" />
- <listener class-name="vid.automation.test.infra.SkipTestUntilTestngTransformer"/>
+ <listener class-name="vid.automation.test.infra.SkipTestsTestngTransformer" />
</listeners>
<test name="test">
diff --git a/vid-automation/TestNg-UI-half.xml b/vid-automation/TestNg-UI-half.xml
index b19073869..27abcd76c 100644
--- a/vid-automation/TestNg-UI-half.xml
+++ b/vid-automation/TestNg-UI-half.xml
@@ -2,8 +2,7 @@
<suite verbose="1" name="VID UI Tests" annotations="JDK">
<listeners>
- <listener class-name="vid.automation.test.infra.FeatureTogglingTestngTransformer"/>
- <listener class-name="vid.automation.test.infra.SkipTestUntilTestngTransformer"/>
+ <listener class-name="vid.automation.test.infra.SkipTestsTestngTransformer"/>
</listeners>
<test name="test">
<method-selectors>
diff --git a/vid-automation/TestNg-dev.xml b/vid-automation/TestNg-dev.xml
index 3dc9d749d..d50785eb5 100644
--- a/vid-automation/TestNg-dev.xml
+++ b/vid-automation/TestNg-dev.xml
@@ -4,8 +4,7 @@
<suite verbose="1" name="VID UI Tests" annotations="JDK">
<listeners>
- <listener class-name="vid.automation.test.infra.FeatureTogglingTestngTransformer"/>
- <listener class-name="vid.automation.test.infra.SkipTestUntilTestngTransformer"/>
+ <listener class-name="vid.automation.test.infra.SkipTestsTestngTransformer"/>
</listeners>
<test name="test">
<groups>
diff --git a/vid-automation/src/main/java/vid/automation/test/infra/SkipTestUntilTestngTransformer.java b/vid-automation/src/main/java/vid/automation/test/infra/SkipTestUntilTestngTransformer.java
deleted file mode 100644
index 2d2ce7cde..000000000
--- a/vid-automation/src/main/java/vid/automation/test/infra/SkipTestUntilTestngTransformer.java
+++ /dev/null
@@ -1,57 +0,0 @@
-package vid.automation.test.infra;
-
-import java.lang.reflect.Constructor;
-import java.lang.reflect.Method;
-import java.time.LocalDate;
-import org.testng.IAnnotationTransformer;
-import org.testng.annotations.ITestAnnotation;
-
-/*
-TestNg listener that skip tests that are annotated with SkipTestUntil annotation
-Pay attention that this listener shall be configured in the testng.xml (or command line)
-*/
-public class SkipTestUntilTestngTransformer implements IAnnotationTransformer {
-
- @Override
- public void transform(ITestAnnotation annotation, Class testClass, Constructor testConstructor, Method testMethod) {
-
- if (testMethod!=null) {
- try {
-
- if (!annotation.getEnabled()) {
- return;
- }
-
- if (!testMethod.isAnnotationPresent(SkipTestUntil.class)) {
- return;
- }
-
- String dateAsStr = testMethod.getAnnotation(SkipTestUntil.class).value();
- if (shallDisableTest(dateAsStr)) {
- disableTest(annotation, testMethod.getDeclaringClass().getName(), dateAsStr);
- }
-
- } catch (Exception e) {
- e.printStackTrace();
- }
- }
- }
-
- private boolean shallDisableTest(String dateAsStr) {
- try {
- return LocalDate.now().isBefore(LocalDate.parse(dateAsStr));
- }
- catch (RuntimeException exception) {
- System.out.println("Failure during processing of SkipTestUntil annotation value is " + dateAsStr);
- exception.printStackTrace();
- return false;
- }
- }
-
- private void disableTest(ITestAnnotation annotation, String name, String dateAsStr) {
- System.out.println("Ignore "+ name+" till "+dateAsStr);
- annotation.setEnabled(false);
- }
-
-}
-
diff --git a/vid-automation/src/main/java/vid/automation/test/infra/FeatureTogglingTestngTransformer.java b/vid-automation/src/main/java/vid/automation/test/infra/SkipTestsTestngTransformer.java
index 46794da10..ed9aaaf69 100644
--- a/vid-automation/src/main/java/vid/automation/test/infra/FeatureTogglingTestngTransformer.java
+++ b/vid-automation/src/main/java/vid/automation/test/infra/SkipTestsTestngTransformer.java
@@ -1,5 +1,6 @@
package vid.automation.test.infra;
+import java.time.LocalDate;
import org.testng.IAnnotationTransformer;
import org.testng.annotations.ITestAnnotation;
import org.togglz.core.context.StaticFeatureManagerProvider;
@@ -9,15 +10,21 @@ import java.lang.reflect.Constructor;
import java.lang.reflect.Method;
/*
-In order to skip test classes regards the state of feature flag we add this listener to our testng configuration
+This transformer skip test we want to ignore during running VID tests.
+Pay attention that this listener shall be configured in the testng.xml (or command line)
+It can't be used as Listener annotation of base class
+
+FeatureTogglingTest:
There are 2 ways to annotate that tests required featureFlags to be active :
In method level - with @FeatureTogglingTest on the test method and list of Required Feature flags on
In Class level - with @FeatureTogglingTest on the test class and list of Required Feature flags on
For each test annotation of method level, we check if the test shall whole class shall run regards the features flag test.
-Pay attention that this listener shall be configured in the testng.xml (or command line)
-It can't be used as Listener annotation of base class
+
+SkipTestUntil:
+If test annotated with SkipTestUntil the transformer check if the due date has already pass
+
*/
-public class FeatureTogglingTestngTransformer implements IAnnotationTransformer {
+public class SkipTestsTestngTransformer implements IAnnotationTransformer {
@Override
public void transform(ITestAnnotation annotation, Class testClass, Constructor testConstructor, Method testMethod) {
@@ -29,23 +36,28 @@ public class FeatureTogglingTestngTransformer implements IAnnotationTransformer
return;
}
- if (isIgnoreTest(testMethod)) {
- disableTest(annotation, testMethod.getDeclaringClass().getName());
+ if (isIgnoreFeatureToggledTest(testMethod)) {
+ disableTest(annotation, testMethod.getName());
return;
}
- if (isIgnoreTest(testMethod.getDeclaringClass())) {
+ if (isIgnoreFeatureToggledTest(testMethod.getDeclaringClass())) {
disableTest(annotation, testMethod.getDeclaringClass().getName());
return;
}
+ if (isIgnoreSkipUntilTest(testMethod)) {
+ disableTest(annotation, testMethod.getName());
+ return;
+ }
+
} catch (Exception e) {
e.printStackTrace();
}
}
}
- private boolean isIgnoreTest(AnnotatedElement annotatedElement) {
+ private boolean isIgnoreFeatureToggledTest(AnnotatedElement annotatedElement) {
return (annotatedElement.isAnnotationPresent(FeatureTogglingTest.class) &&
shallDisableTest(annotatedElement.getAnnotation(FeatureTogglingTest.class)));
@@ -68,9 +80,24 @@ public class FeatureTogglingTestngTransformer implements IAnnotationTransformer
}
private void disableTest(ITestAnnotation annotation, String name) {
- System.out.println("Ignore "+ name+" due to feature flags configuration");
+ System.out.println("Ignore "+ name+" due to annotation");
annotation.setEnabled(false);
}
+ private boolean isIgnoreSkipUntilTest(AnnotatedElement annotatedElement) {
+ if (!annotatedElement.isAnnotationPresent(SkipTestUntil.class)) {
+ return false;
+ }
+
+ String dateAsStr = annotatedElement.getAnnotation(SkipTestUntil.class).value();
+ try {
+ return LocalDate.now().isBefore(LocalDate.parse(dateAsStr));
+ }
+ catch (RuntimeException exception) {
+ System.out.println("Failure during processing of SkipTestUntil annotation value is " + dateAsStr);
+ exception.printStackTrace();
+ return false;
+ }
+ }
}