move runSmokeTests on the condition of enableTests

adoptium · Apr 1, 2021 · 4dcfb41 · 4dcfb41
1 parent 461ca3f
commit 4dcfb41
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 9 deletions.
diff --git a/.project b/.project
@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<projectDescription>
+	<name>ci-jenkins-pipelines</name>
+	<comment></comment>
+	<projects>
+	</projects>
+	<buildSpec>
+	</buildSpec>
+	<natures>
+	</natures>
+</projectDescription>
diff --git a/pipelines/build/common/openjdk_build_pipeline.groovy b/pipelines/build/common/openjdk_build_pipeline.groovy
@@ -1411,19 +1411,16 @@ class Build {
                 }
 
                 // Run Smoke Tests and AQA Tests
-                try { 
-                    runSmokeTests()
-                    if (enableTests && buildConfig.TEST_LIST.size() > 0) {
-                        try {
-                            // Run tests if we have a test list, don't use timeouts as the jobs have their own
+                if (enableTests) {
+                    try {
+                        runSmokeTests()
+                        if (buildConfig.TEST_LIST.size() > 0) {
                             def testStages = runAQATests()
                             context.parallel testStages
-                        } catch (Exception e) {
-                            context.println (e.message)
                         }
+                    } catch (Exception e) {
+                        context.println(e.message)
                     }
-                } catch (Exception e) {
-                    context.println(e.message)
                 }
 
                 //buildInstaller if needed