Filter out control characters from lua error message

Perryvw · Perryvw · commit 57efc144bfc5 · 2019-08-01T23:04:55.000+02:00
diff --git a/test/util.ts b/test/util.ts
@@ -252,7 +252,9 @@ export abstract class TestBuilder {
                 throw new Error(`Unsupported Lua return type: ${returnType}`);
             }
         } else {
-            const message = to_jsstring(lua.lua_tostring(L, -1)).replace(/^\[string "--\.\.\."\]:\d+: /, "");
+            // Filter out control characters appearing on some systems
+            const luaStackString = lua.lua_tostring(L, -1).filter(c => c >= 20);
+            const message = to_jsstring(luaStackString).replace(/^\[string "--\.\.\."\]:\d+: /, "");
             return new ExecutionError(message);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,9 @@ export abstract class TestBuilder {`
`252`	`252`	throw new Error(`Unsupported Lua return type: ${returnType}`);
`253`	`253`	`}`
`254`	`254`	`} else {`
`255`		`- const message = to_jsstring(lua.lua_tostring(L, -1)).replace(/^\[string "--\.\.\."\]:\d+: /, "");`
	`255`	`+ // Filter out control characters appearing on some systems`
	`256`	`+ const luaStackString = lua.lua_tostring(L, -1).filter(c => c >= 20);`
	`257`	`+ const message = to_jsstring(luaStackString).replace(/^\[string "--\.\.\."\]:\d+: /, "");`
`256`	`258`	`return new ExecutionError(message);`
`257`	`259`	`}`
`258`	`260`	`}`