From fdcdde0b0aeaf58b7187798d1a2033263bfee5ea Mon Sep 17 00:00:00 2001 From: hindujaB Date: Fri, 13 Oct 2023 17:41:23 +0530 Subject: [PATCH 1/3] Disable failing tests --- .../test/observability/tracing/TracingTestCase.java | 4 ++-- .../ballerinalang/test/packaging/LockFileTestCase.java | 9 ++++++--- .../test/packaging/PathDependencyTestCase.java | 4 ++-- .../src/test/resources/testng.xml | 2 +- 4 files changed, 11 insertions(+), 8 deletions(-) diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/observability/tracing/TracingTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/observability/tracing/TracingTestCase.java index 0bc53b703c74..0a34e06bb2ed 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/observability/tracing/TracingTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/observability/tracing/TracingTestCase.java @@ -186,7 +186,7 @@ public void testObservePackageUserTraceTrue() throws Exception { }); } - @Test(dependsOnMethods = "testObservePackageUserTraceTrue") + @Test(dependsOnMethods = "testObservePackageUserTraceTrue", enabled = false) public void testOOTBTracingWithWorkers() throws Exception { final String service = "http://localhost:9093/echoService/"; HttpClientRequest.doGet(service + "resourceOne"); @@ -215,7 +215,7 @@ public void testOOTBTracingWithWorkers() throws Exception { .filter(bMockSpan -> bMockSpan.getParentId() == 0).count(), 8, "Mismatch in number of root spans."); } - @Test(dependsOnMethods = "testOOTBTracingWithWorkers") + @Test(dependsOnMethods = "testOOTBTracingWithWorkers", enabled = false) public void testOOTBTracingWithErrors() throws Exception { final String service = "http://localhost:9094/echoService/"; HttpClientRequest.doGet(service + "resourceOne/3"); diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java index 9504e6b83b16..66e7fc0f1ee0 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java @@ -155,7 +155,8 @@ public void testBuildAndPushTestProject1() throws BallerinaTestException { * @throws IOException When updating the module names. * @throws BallerinaTestException When running commands. */ - @Test(description = "Test building and running TestProject2", dependsOnMethods = "testBuildAndPushTestProject1") + @Test(description = "Test building and running TestProject2", dependsOnMethods = "testBuildAndPushTestProject1", + enabled = false) public void testBuildTestProject2() throws IOException, BallerinaTestException, InterruptedException { // Replace module names in source file Path fooSayBal = testProj2Path.resolve("src").resolve("foo").resolve("foo_say.bal"); @@ -201,7 +202,8 @@ public void testBuildTestProject2() throws IOException, BallerinaTestException, * @throws IOException When updating the implementation of the project. * @throws BallerinaTestException When running commands. */ - @Test(description = "Test updating TestProject1 and pushing.", dependsOnMethods = "testBuildTestProject2") + @Test(description = "Test updating TestProject1 and pushing.", dependsOnMethods = "testBuildTestProject2", + enabled = false) public void testModifyProj1AndPush() throws IOException, BallerinaTestException { // Update code in module1 Path module2SourceFile = testProj1Path.resolve("src").resolve(module2Name).resolve("say.bal"); @@ -258,7 +260,8 @@ public void testModifyProj1AndPush() throws IOException, BallerinaTestException * * @throws BallerinaTestException When running commands. */ - @Test(description = "Test rebuilding and running TestProject2", dependsOnMethods = "testModifyProj1AndPush") + @Test(description = "Test rebuilding and running TestProject2", dependsOnMethods = "testModifyProj1AndPush", + enabled = false) public void testRebuildTestProj2() throws BallerinaTestException { // Build module String fooBaloFileName = "foo-" diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PathDependencyTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PathDependencyTestCase.java index 47b3c9f3c403..94eef7660c9b 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PathDependencyTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PathDependencyTestCase.java @@ -200,7 +200,7 @@ public void testBaloPathCase3() throws BallerinaTestException { * * @throws BallerinaTestException Error when executing the commands. */ - @Test() + @Test(enabled = false) public void testBaloPathCase4() throws BallerinaTestException, IOException, InterruptedException { Path caseResources = tempTestResources.resolve("case4"); // Build bee module of TestProject1 @@ -455,7 +455,7 @@ public void testBaloPathCase7() throws BallerinaTestException { * @throws BallerinaTestException Error when executing the commands. */ @Test(description = "Case8: Test single bal file using external module with interop dependency", - dependsOnMethods = "testBaloPathCase4") + dependsOnMethods = "testBaloPathCase4", enabled = false) public void testBaloSingleBalFileCase8() throws BallerinaTestException, IOException { Path caseResources = tempTestResources.resolve("case8"); diff --git a/tests/jballerina-integration-test/src/test/resources/testng.xml b/tests/jballerina-integration-test/src/test/resources/testng.xml index f96d7a0553f6..8d411599c637 100644 --- a/tests/jballerina-integration-test/src/test/resources/testng.xml +++ b/tests/jballerina-integration-test/src/test/resources/testng.xml @@ -315,7 +315,7 @@ - + From a43bf12d458e3868746024d2b2258ffc7507ba2e Mon Sep 17 00:00:00 2001 From: hindujaB Date: Fri, 13 Oct 2023 18:54:11 +0530 Subject: [PATCH 2/3] Disable failing tests --- .../ballerinalang/test/packaging/LockFileTestCase.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java index 66e7fc0f1ee0..9504e6b83b16 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/LockFileTestCase.java @@ -155,8 +155,7 @@ public void testBuildAndPushTestProject1() throws BallerinaTestException { * @throws IOException When updating the module names. * @throws BallerinaTestException When running commands. */ - @Test(description = "Test building and running TestProject2", dependsOnMethods = "testBuildAndPushTestProject1", - enabled = false) + @Test(description = "Test building and running TestProject2", dependsOnMethods = "testBuildAndPushTestProject1") public void testBuildTestProject2() throws IOException, BallerinaTestException, InterruptedException { // Replace module names in source file Path fooSayBal = testProj2Path.resolve("src").resolve("foo").resolve("foo_say.bal"); @@ -202,8 +201,7 @@ public void testBuildTestProject2() throws IOException, BallerinaTestException, * @throws IOException When updating the implementation of the project. * @throws BallerinaTestException When running commands. */ - @Test(description = "Test updating TestProject1 and pushing.", dependsOnMethods = "testBuildTestProject2", - enabled = false) + @Test(description = "Test updating TestProject1 and pushing.", dependsOnMethods = "testBuildTestProject2") public void testModifyProj1AndPush() throws IOException, BallerinaTestException { // Update code in module1 Path module2SourceFile = testProj1Path.resolve("src").resolve(module2Name).resolve("say.bal"); @@ -260,8 +258,7 @@ public void testModifyProj1AndPush() throws IOException, BallerinaTestException * * @throws BallerinaTestException When running commands. */ - @Test(description = "Test rebuilding and running TestProject2", dependsOnMethods = "testModifyProj1AndPush", - enabled = false) + @Test(description = "Test rebuilding and running TestProject2", dependsOnMethods = "testModifyProj1AndPush") public void testRebuildTestProj2() throws BallerinaTestException { // Build module String fooBaloFileName = "foo-" From 9430b1f8eeed2e909316a9aa6ccf5058105f2dd6 Mon Sep 17 00:00:00 2001 From: hindujaB Date: Fri, 13 Oct 2023 21:39:46 +0530 Subject: [PATCH 3/3] Disable failing push tests --- .../test/packaging/PackagingNegativeTestCase.java | 2 +- .../org/ballerinalang/test/packaging/PackagingTestCase.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingNegativeTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingNegativeTestCase.java index 3ddc017d8d2e..118347450ad5 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingNegativeTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingNegativeTestCase.java @@ -121,7 +121,7 @@ public void testBuildWithoutManifest() throws Exception { moduleBuildLeecher.waitForText(5000); } - @Test(description = "Test pushing a module to central without any content in Module.md") + @Test(description = "Test pushing a module to central without any content in Module.md", enabled = false) public void testPushWithoutModuleMDContent() throws Exception { Path projectPath = Files.createDirectories(tempProjectDirectory).resolve("projectWithoutModuleMDContent"); createProjectStructureAndGetProjectPath(projectPath, moduleName); diff --git a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingTestCase.java b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingTestCase.java index 79c8c6edb953..cee867fe0964 100644 --- a/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingTestCase.java +++ b/tests/jballerina-integration-test/src/test/java/org/ballerinalang/test/packaging/PackagingTestCase.java @@ -104,7 +104,7 @@ public void testCreateProject() throws Exception { Assert.assertTrue(Files.isDirectory(projectPath.resolve("src").resolve(moduleName))); } - @Test(description = "Test pushing a package to central", dependsOnMethods = "testCreateProject") + @Test(description = "Test pushing a package to central", dependsOnMethods = "testCreateProject", enabled = false) public void testPush() throws Exception { Path projectPath = tempProjectDirectory.resolve("initProject"); @@ -142,7 +142,7 @@ public void testPush() throws Exception { clientLeecher.waitForText(60000); } - @Test(description = "Test pulling a package from central", dependsOnMethods = "testPush") + @Test(description = "Test pulling a package from central", dependsOnMethods = "testPush", enabled = false) public void testPull() { String baloFileName = moduleName + "-" + ProgramFileConstants.IMPLEMENTATION_VERSION + "-" @@ -165,7 +165,7 @@ public void testPull() { Assert.assertTrue(Files.exists(tempHomeDirectory.resolve(baloPath).resolve(baloFileName))); } - @Test(description = "Test searching a package from central", dependsOnMethods = "testPush") + @Test(description = "Test searching a package from central", dependsOnMethods = "testPush", enabled = false) public void testSearch() throws BallerinaTestException { String actualMsg = balClient.runMainAndReadStdOut("search", new String[]{moduleName}, envVariables, balServer.getServerHome(), false);