Merge pull request #1833 from ironm00n/fix/table-row-err

fix: table `row` method throwing invalid `row-length-mismatch`

Author: jpolitz

src/js/trove/table.js

@@ -837,7 +837,7 @@
         'row': runtime.makeMethodN(function(self, ...args) {
           // NOTE: Deliberately no arity check
           if(headers.length !== args.length) {
-            throw runtime.ffi.throwRowLengthMismatch(makeTable(headers, []), args);
+            throw runtime.ffi.throwRowLengthMismatch(headers, args);
           }
           return makeRow({ headerIndex: headerIndex }, args);
         }),

tests/pyret/regression.arr

@@ -33,3 +33,4 @@ import file("./regression/sloppy-list-filter.arr") as _
 import file("./regression/import-module-defining-but-not-exporting-data.arr") as _
 import file("./regression/render-reason-wrong-op.arr") as _
 import file("./regression/table-reduce.arr") as _
+import file("./regression/table-row-row-length-mismatch.arr") as _

tests/pyret/regression/table-row-row-length-mismatch.arr

@@ -0,0 +1,19 @@
+import either as E
+import error as Err
+
+check:
+  table = table: foo, bar
+    row: 1, 2
+    row: 3, 4
+  end
+
+  cases(E.Either) run-task(lam(): table.row(1) end) block:
+    | left(_) => false is true
+    | right(exn) =>
+      unwrapped = exn-unwrap(exn)
+      unwrapped satisfies Err.is-row-length-mismatch
+      
+      unwrapped.render-reason() does-not-raise
+  end
+end
+