Fix skip logic

wso2 · Aug 22, 2024 · 6baf85c · 6baf85c
1 parent 825d013
commit 6baf85c
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 17 deletions.
diff --git a/...tests/src/test/java/org/wso2/ei/dataservice/integration/test/odata/ODataETagTestCase.java b/...tests/src/test/java/org/wso2/ei/dataservice/integration/test/odata/ODataETagTestCase.java
@@ -228,9 +228,9 @@ public void validateETagConcurrentHandlingTestCaseForDeleteMethod() throws Excep
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             System.out.println("This test is temporarily skipped for this workflow");
             String endpoint = webAppUrl + "/odata/" + serviceName + "/" + configId + "/FILES";
             String content = "{\"FILENAME\": \"WSO2PROD\" ,\"TYPE\" : \"dss\"}";
@@ -286,10 +286,9 @@ public void validateETagConcurrentHandlingTestCaseForUpdatePropertyWithPutMethod
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
-            System.out.println("This test is temporarily skipped for this workflow");
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             String endpoint = webAppUrl + "/odata/" + serviceName + "/" + configId + "/FILES";
             String content = "{\"FILENAME\": \"WSO2PROD\" ,\"TYPE\" : \"dss\"}";
             Map<String, String> headers = new HashMap<>();
@@ -346,9 +345,9 @@ public void validateETagConcurrentHandlingTestCaseForUpdatePropertyWithPatchMeth
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             System.out.println("This test is temporarily skipped for this workflow");
             String entityEndpoint = webAppUrl + "/odata/" + serviceName + "/" + configId + "/FILES(\'WSO2PROD\')";
             String endpoint = webAppUrl + "/odata/" + serviceName + "/" + configId + "/FILES(\'WSO2PROD\')/TYPE";

diff --git a/...va/org/wso2/carbon/esb/mediator/test/aggregate/AggregateWithHighMaxAndLowMinTestCase.java b/...va/org/wso2/carbon/esb/mediator/test/aggregate/AggregateWithHighMaxAndLowMinTestCase.java
@@ -161,9 +161,9 @@ public void testLessNumberThanMaximum() throws IOException, XMLStreamException {
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             int responseCount = 0;
 
             no_of_requests = 60;

diff --git a/...1/src/test/java/org/wso2/carbon/esb/mediator/test/foreach/ForEachJSONPayloadTestCase.java b/...1/src/test/java/org/wso2/carbon/esb/mediator/test/foreach/ForEachJSONPayloadTestCase.java
@@ -53,9 +53,9 @@ public void testForEachMediatorWithJSONPayload() throws Exception {
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             String request = "{\"getQuote\":{\"request\":[{\"symbol\":\"IBM\"},{\"symbol\":\"WSO2\"},{\"symbol\":\"MSFT\"}]}}";
 
             simpleHttpClient = new SimpleHttpClient();

diff --git a/.../test/java/org/wso2/carbon/esb/jms/transport/test/ESBJAVA1832MessageInjectorTestCase.java b/.../test/java/org/wso2/carbon/esb/jms/transport/test/ESBJAVA1832MessageInjectorTestCase.java
@@ -56,9 +56,9 @@ public void testMessageInjection() throws Exception {
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             String queueName = "jmsQueue";
             int numberOfMsgToExpect = 10;
             TimeUnit.SECONDS.sleep(15);

diff --git a/...s-transport/src/test/java/org/wso2/carbon/esb/jms/transport/test/ESBJAVA2907TestCase.java b/...s-transport/src/test/java/org/wso2/carbon/esb/jms/transport/test/ESBJAVA2907TestCase.java
@@ -26,9 +26,9 @@ public void testAddingOMElementPropertyToMessageStore() throws Exception {
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             AxisServiceClient client = new AxisServiceClient();
             client.sendRobust(Utils.getStockQuoteRequest("IBM"), getProxyServiceURLHttp("testPS"), "getQuote");
             Assert.assertTrue(carbonLogReader.checkForLog(GET_QUOTE_REQUEST_BODY, DEFAULT_TIMEOUT), "OMElement is not saved to the message store");

diff --git a/...transport/src/test/java/org/wso2/carbon/esb/jms/transport/test/MSMPCronForwarderCase.java b/...transport/src/test/java/org/wso2/carbon/esb/jms/transport/test/MSMPCronForwarderCase.java
@@ -72,9 +72,9 @@ public void testMessageProcessorCronForwader() throws Exception {
         System.out.println(
                 Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))
                         ? "This test is temporarily skipped for this workflow"
-                        : ""
+                        : "Test not skipped"
         );
-        if (Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
+        if (!Boolean.parseBoolean(System.getenv("CI_BUILD_SKIP"))) {
             // SEND THE REQUEST
             String addUrl = getProxyServiceURLHttp("MSMPRetrytest");
             String payload = "{\"name\":\"Jack\"}";