Merge pull request #49 from MarkEWaite/fix-JCasC-defaultTime

[JENKINS-66939] - Allow JCasC schedule time setting

README.md

@@ -39,8 +39,6 @@ Add to your yaml file:
 ```yaml
 unclassified:
   scheduleBuild:
-    # defaultScheduleTime is not currently supported in JCasC
-    # See JENKINS-66939 for more details
-    # defaultScheduleTime: "11:00:00 PM"
+    defaultScheduleTime: "11:00:00 PM"
     timeZone: "Europe/Paris"
 ```

src/test/java/org/jenkinsci/plugins/schedulebuild/JCasCGlobalAllFields.java

@@ -0,0 +1,32 @@
+package org.jenkinsci.plugins.schedulebuild;
+
+import io.jenkins.plugins.casc.misc.RoundTripAbstractTest;
+import java.util.TimeZone;
+import jenkins.model.GlobalConfiguration;
+import org.jvnet.hudson.test.RestartableJenkinsRule;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.not;
+import static org.hamcrest.CoreMatchers.nullValue;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+public class JCasCGlobalAllFields extends RoundTripAbstractTest {
+
+    @Override
+    protected void assertConfiguredAsExpected(RestartableJenkinsRule j, String configContent) {
+        ScheduleBuildGlobalConfiguration globalConfig = GlobalConfiguration.all().getInstance(ScheduleBuildGlobalConfiguration.class);
+        assertThat(globalConfig, is(not(nullValue())));
+        assertThat(globalConfig.getDefaultScheduleTime(), is("12:34:56 AM"));
+        assertThat(globalConfig.getTimeZone(), is("Europe/Rome"));
+    }
+
+    @Override
+    protected String stringInLogExpected() {
+        return "timeZone = Europe/Rome";
+    }
+
+    @Override
+    protected String configResource() {
+        return "scheduleBuild-all-fields.yaml";
+    }
+}

src/test/resources/org/jenkinsci/plugins/schedulebuild/scheduleBuild-all-fields.yaml

@@ -0,0 +1,4 @@
+unclassified:
+  scheduleBuild:
+    defaultScheduleTime: "12:34:56 AM"
+    timeZone: "Europe/Rome"

src/test/resources/org/jenkinsci/plugins/schedulebuild/scheduleBuild-default-time-zone.yaml

@@ -1,6 +1,3 @@
 unclassified:
   scheduleBuild:
-    # defaultScheduleTime is not currently supported in JCasC
-    # See JENKINS-66939 for more details
-    # defaultScheduleTime: "11:00:00 PM"
     timeZone: "Europe/Paris"