fix broken CASC_JENKINS_CONFIG_ENV support + test case

[FIX JENKINS-51283] close #203
jenkinsci · May 14, 2018 · 8ab67e7 · 8ab67e7
1 parent 624af92
commit 8ab67e7
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 5 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/casc/ConfigurationAsCode.java b/src/main/java/org/jenkinsci/plugins/casc/ConfigurationAsCode.java
@@ -125,11 +125,13 @@ public void configure() throws ConfiguratorException {
                 CASC_JENKINS_CONFIG_PROPERTY,
                 System.getenv(CASC_JENKINS_CONFIG_ENV)
         );
-        if (configParameter == null  && Files.exists(Paths.get(DEFAULT_JENKINS_YAML_PATH))) {
-            configParameter = DEFAULT_JENKINS_YAML_PATH;
-        } else {
-            // No configuration set nor default config file
-            return;
+        if (configParameter == null) {
+            if (Files.exists(Paths.get(DEFAULT_JENKINS_YAML_PATH))) {
+                configParameter = DEFAULT_JENKINS_YAML_PATH;
+            } else {
+                // No configuration set nor default config file
+                return;
+            }
         }
 
         configure(configParameter);

diff --git a/src/test/java/org/jenkinsci/plugins/casc/JenkinsConfigTest.java b/src/test/java/org/jenkinsci/plugins/casc/JenkinsConfigTest.java
@@ -0,0 +1,7 @@
+package org.jenkinsci.plugins.casc;
+
+/**
+ * @author <a href="mailto:nicolas.deloof@gmail.com">Nicolas De Loof</a>
+ */
+public class JenkinsConfigTest {
+}
diff --git a/src/test/resources/org/jenkinsci/plugins/casc/JenkinsConfigTest.yml b/src/test/resources/org/jenkinsci/plugins/casc/JenkinsConfigTest.yml