diff options
author | daurnimator <quae@daurnimator.com> | 2017-12-13 12:39:42 +1100 |
---|---|---|
committer | daurnimator <quae@daurnimator.com> | 2017-12-13 15:04:05 +1100 |
commit | 66294fd31720f5073fb7cc1540f67e76cb638859 (patch) | |
tree | 29d0507da00d790ee91fba527d61f83c653356ab | |
parent | c05459ce4663efca327eba53d60b1667c126058b (diff) | |
download | fengari-66294fd31720f5073fb7cc1540f67e76cb638859.tar.gz fengari-66294fd31720f5073fb7cc1540f67e76cb638859.tar.bz2 fengari-66294fd31720f5073fb7cc1540f67e76cb638859.zip |
tests/: Don't hard-code string hashes
-rw-r--r-- | tests/lexparse.js | 5 | ||||
-rw-r--r-- | tests/lvm.js | 5 |
2 files changed, 6 insertions, 4 deletions
diff --git a/tests/lexparse.js b/tests/lexparse.js index e7ba0c5..ca57605 100644 --- a/tests/lexparse.js +++ b/tests/lexparse.js @@ -6,6 +6,7 @@ const lua = require('../src/lua.js'); const lauxlib = require('../src/lauxlib.js'); const lualib = require('../src/lualib.js'); const lapi = require('../src/lapi.js'); +const lstring = require("../src/lstring.js"); // Roughly the same tests as test/lvm.js to cover all opcodes @@ -743,7 +744,7 @@ test('SETTABLE, GETTABLE', function (t) { ); t.strictEqual( - lua.lua_topointer(L, -1).strong.get('116|119|111|').value.jsstring(), + lua.lua_topointer(L, -1).strong.get(lstring.luaS_hash(lua.to_luastring("two"))).value.jsstring(), "world", "Program output is correct" ); @@ -836,7 +837,7 @@ test('SETTABUP, GETTABUP', function (t) { ); t.strictEqual( - lua.lua_topointer(L, -1).strong.get('116|119|111|').value.jsstring(), + lua.lua_topointer(L, -1).strong.get(lstring.luaS_hash(lua.to_luastring("two"))).value.jsstring(), "world", "Program output is correct" ); diff --git a/tests/lvm.js b/tests/lvm.js index 209329e..08c0c5e 100644 --- a/tests/lvm.js +++ b/tests/lvm.js @@ -3,6 +3,7 @@ const test = require('tape'); const lua = require("../src/lua.js"); +const lstring = require("../src/lstring.js"); const getState = require("./tests.js").getState; @@ -458,7 +459,7 @@ test('SETTABLE, GETTABLE', function (t) { ); t.deepEqual( - L.stack[L.top - 1].value.strong.get('116|119|111|').value.jsstring(), // "two" + L.stack[L.top - 1].value.strong.get(lstring.luaS_hash(lua.to_luastring("two"))).value.jsstring(), // "two" "world", "Program output is correct" ); @@ -517,7 +518,7 @@ test('SETTABUP, GETTABUP', function (t) { ); t.deepEqual( - L.stack[L.top - 1].value.strong.get('116|119|111|').value.jsstring(), // "two" + L.stack[L.top - 1].value.strong.get(lstring.luaS_hash(lua.to_luastring("two"))).value.jsstring(), // "two" "world", // "world" "Program output is correct" ); |