move runSmokeTests on the condition of enableTests

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
adoptium · Apr 1, 2021 · 3f6191f · 3f6191f
1 parent 461ca3f
commit 3f6191f
Showing 1 changed file with 6 additions and 9 deletions.
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