From: Ittay Stern Date: Wed, 21 Aug 2019 10:01:21 +0000 (+0000) Subject: Merge "merge 2 TestNg transformers" X-Git-Tag: 5.0.1~32 X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=db18734d1fed426ff5c6a318f3179001a606690e;hp=92ebd799948bab4a5143910009e50ead1359a2d5;p=vid.git Merge "merge 2 TestNg transformers" --- 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 @@ - - + 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 @@ - - + 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 @@ - - + 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 similarity index 61% rename from vid-automation/src/main/java/vid/automation/test/infra/FeatureTogglingTestngTransformer.java rename to 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; + } + } }