Allow ChronoUnitReader to accept - (i.e. half-days)

iheartradio · Sep 9, 2021 · e1a1cb4 · e1a1cb4
1 parent 537fd6f
commit e1a1cb4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/main/scala/net/ceedubs/ficus/readers/ChronoUnitReader.scala b/src/main/scala/net/ceedubs/ficus/readers/ChronoUnitReader.scala
@@ -9,7 +9,7 @@ trait ChronoUnitReader {
 
     /** Reads the value at the path `path` in the Config */
     override def read(config: Config, path: String): ChronoUnit =
-      ChronoUnit.valueOf(config.getString(path).toUpperCase)
+      ChronoUnit.valueOf(config.getString(path).toUpperCase.replace('-', '_'))
   }
 }
 

diff --git a/src/test/scala/net/ceedubs/ficus/readers/ChronoUnitReaderSpec.scala b/src/test/scala/net/ceedubs/ficus/readers/ChronoUnitReaderSpec.scala
@@ -35,4 +35,15 @@ class ChronoUnitReaderSpec extends Spec {
     chronoUnit should_== expected
   }
 
+  def readChronoUnitHalfDaysDash = {
+    val cfg        = ConfigFactory.parseString(s"""
+                                                  | foo {
+                                                  |    chrono-unit = "half-days"
+                                                  | }
+       """.stripMargin)
+    val chronoUnit = cfg.as[ChronoUnit]("foo.chrono-unit")
+    val expected   = ChronoUnit.HALF_DAYS
+    chronoUnit should_== expected
+  }
+
 }