From ca5ab6c981ccca170ad4ea85b3b36a24a123bd4b Mon Sep 17 00:00:00 2001
From: Stephen Amar <stephen.amar@databricks.com>
Date: Sat, 28 Dec 2024 22:33:22 -0800
Subject: [PATCH] add tests

---
 sjsonnet/src/sjsonnet/Materializer.scala            | 4 +++-
 sjsonnet/test/src-jvm/sjsonnet/ParseYamlTests.scala | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/sjsonnet/src/sjsonnet/Materializer.scala b/sjsonnet/src/sjsonnet/Materializer.scala
index 70e3299c..3b819690 100644
--- a/sjsonnet/src/sjsonnet/Materializer.scala
+++ b/sjsonnet/src/sjsonnet/Materializer.scala
@@ -59,8 +59,10 @@ abstract class Materializer {
       case Val.Null(pos) => storePos(pos); visitor.visitNull(-1)
       case s: Val.Func =>
         Error.fail("Couldn't manifest function with params [" + s.params.names.mkString(",") + "]", v.pos)
+      case vv: Val =>
+        Error.fail("Unknown value type " + vv.prettyName, vv.pos)
       case _ =>
-        Error.fail("Unknown value type", v.pos)
+        Error.fail("Unknown value type " + v)
     }
   } catch {
     case _: StackOverflowError =>
diff --git a/sjsonnet/test/src-jvm/sjsonnet/ParseYamlTests.scala b/sjsonnet/test/src-jvm/sjsonnet/ParseYamlTests.scala
index cfd5c443..e1a35217 100644
--- a/sjsonnet/test/src-jvm/sjsonnet/ParseYamlTests.scala
+++ b/sjsonnet/test/src-jvm/sjsonnet/ParseYamlTests.scala
@@ -7,10 +7,12 @@ object ParseYamlTests extends TestSuite {
   def tests: Tests = Tests {
     test {
       eval("std.parseYaml('foo: bar')") ==> ujson.Value("""{"foo":"bar"}""")
+      eval("std.parseYaml('- foo: bar')") ==> ujson.Value("""[{"foo":"bar"}]""")
     }
     test {
       eval("std.parseYaml('')") ==> ujson.Value("""{}""")
-      }
+      eval("std.parseYaml(\"0777\")") ==> ujson.Value("""511""")
+    }
     test {
       eval("std.parseYaml('foo: bar\n---\nbar: baz\n')") ==> ujson.Value("""[{"foo": "bar"}, {"bar": "baz"}]""")
     }