Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: test_wrapper.js

Issue 29333474: Issue 4125 - [emscripten] Convert filter classes to C++ (Closed)
Patch Set: Rebased, addressed comments, changed StringMap::find() return value Created Feb. 18, 2016, 4:02 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« compiled/bindings.h ('K') | « test/regexpFilters_matching.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 "use strict"; 1 "use strict";
2 2
3 require("node-babel")(); 3 require("node-babel")({
4 ignore: /\bcompiled\b/,
5 blacklist: ["useStrict"]
6 });
7
8 // TEMPORARY HACK: Establish filterClassesNew as an alias for filterClasses for
9 // now, in order to allow testing both in parallel.
10 let nodeModule = require("module");
11 let origResolve = nodeModule._resolveFilename;
12 nodeModule._resolveFilename = function()
13 {
14 let params = Array.slice(arguments);
15 if (params[0] == "filterClasses")
16 params[0] = "filterClassesNew";
17 else if (params[0] == "../lib/filterClasses")
18 params[0] = "../lib/filterClassesNew";
19 return origResolve.apply(this, params);
sergei 2016/02/22 12:46:15 Actually to run tests I have to put there some con
Wladimir Palant 2016/02/23 12:37:45 Are you running this file directly? You shouldn't
sergei 2016/02/23 15:07:32 Yes, thanks, overlooked it in package.json.
20 };
4 21
5 require("./node_modules/.bin/nodeunit"); 22 require("./node_modules/.bin/nodeunit");
OLDNEW
« compiled/bindings.h ('K') | « test/regexpFilters_matching.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld