Fix error message for too many arguments whith at least one named arg

.gitignore

@@ -4,4 +4,5 @@ output/
 .DS_STORE
 .idea_modules
 .idea
+.metals
 out/

sjsonnet/src/sjsonnet/Val.scala

@@ -212,6 +212,13 @@ object Val{
         .defaults
         .map{ case (index, default) => (index, Lazy(evalDefault(default, newScope, evaluator)))}
 
+      // Check if expected args match passed args before  
+      if (params.args.length < args.length) {
+        Error.fail(
+          "Too many args, function has " + params.args.length + " parameter(s)",
+          outerOffset
+        )
+      } 
       lazy val passedArgsBindings = try
         args.zipWithIndex.map{
           case ((Some(name), v), _) =>

sjsonnet/test/resources/imports/error.too_many_arg_with_named_arg.jsonnet

@@ -0,0 +1,2 @@
+local foo (x) = x;
+foo(1, x = "bar")

sjsonnet/test/src-jvm/sjsonnet/ErrorTests.scala

@@ -288,20 +288,12 @@ object ErrorTests extends TestSuite{
          |    at .(sjsonnet/test/resources/imports/error.too_many_arg.jsonnet:3:6)
          |""".stripMargin
     )
-    //    test("overflow") - check(
-//      """sjsonnet.Error: my error message
-//        |    at .(sjsonnet/test/resources/test_suite/error.invariant.simple3.jsonnet:18:10)
-//        |""".stripMargin
-//    )
-//    test("overflow2") - check(
-//      """sjsonnet.Error: my error message
-//        |    at .(sjsonnet/test/resources/test_suite/error.invariant.simple3.jsonnet:18:10)
-//        |""".stripMargin
-//    )
-//    test("overflow3") - check(
-//      """sjsonnet.Error: my error message
-//        |    at .(sjsonnet/test/resources/test_suite/error.invariant.simple3.jsonnet:18:10)
-//        |""".stripMargin
-//    )
+
+    test("too_many_arg_with_named_arg") - checkImports(
+      """|sjsonnet.Error: Too many args, function has 1 parameter(s)
+         |    at .(sjsonnet/test/resources/imports/error.too_many_arg_with_named_arg.jsonnet:2:4)
+         |    at .(sjsonnet/test/resources/imports/error.too_many_arg_with_named_arg.jsonnet:2:4)
+         |""".stripMargin
+    )
   }
 }

sjsonnet/test/src/sjsonnet/EvaluatorTests.scala

@@ -225,12 +225,17 @@ object EvaluatorTests extends TestSuite{
             |  welcome: 'Hello ' + name + '!',
             |};
             |{
-            |  person2: Person('Bob', hello=123),
+            |  person2: Person(hello=123),
             |}
         """.stripMargin
         )
       }
 
+      /* 
+       * Wrong number of params error takes precedence on invalid param
+       * For that reason, to assert for invalid param we need to pass the 
+       * right number of params in the first place.
+       */
       assert(ex.getMessage.contains("Function has no parameter hello"))
     }