Add Seconds as a supported unit for IntervalSchedule (#849)

* Add Seconds as a supported unit for IntervalSchedule

Signed-off-by: Craig Perkins <[email protected]>

* Fix test

Signed-off-by: Craig Perkins <[email protected]>

* Added a comment

Signed-off-by: Craig Perkins <[email protected]>

---------

Signed-off-by: Craig Perkins <[email protected]>

Author: cwperks

src/main/kotlin/org/opensearch/commons/alerting/model/Schedule.kt

@@ -282,8 +282,9 @@ data class IntervalSchedule(
         sin.readEnum(ChronoUnit::class.java) // unit
     )
     companion object {
+        // Including SECONDS in this list for testing purposes to run test monitors in order of seconds
         @Transient
-        private val SUPPORTED_UNIT = listOf(ChronoUnit.MINUTES, ChronoUnit.HOURS, ChronoUnit.DAYS)
+        private val SUPPORTED_UNIT = listOf(ChronoUnit.SECONDS, ChronoUnit.MINUTES, ChronoUnit.HOURS, ChronoUnit.DAYS)
 
         @JvmStatic
         @Throws(IOException::class)

src/test/kotlin/org/opensearch/commons/alerting/model/ScheduleTest.kt

@@ -339,7 +339,7 @@ class ScheduleTest : XContentTestBase {
     @Test
     fun `test invalid interval units`() {
         assertFailsWith(IllegalArgumentException::class, "Expected IllegalArgumentException") {
-            IntervalSchedule(1, ChronoUnit.SECONDS)
+            IntervalSchedule(1, ChronoUnit.YEARS)
         }
 
         assertFailsWith(IllegalArgumentException::class, "Expected IllegalArgumentException") {