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

Unified 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.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« compiled/bindings.h ('K') | « test/regexpFilters_matching.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test_wrapper.js
===================================================================
--- a/test_wrapper.js
+++ b/test_wrapper.js
@@ -1,5 +1,22 @@
"use strict";
-require("node-babel")();
+require("node-babel")({
+ ignore: /\bcompiled\b/,
+ blacklist: ["useStrict"]
+});
+
+// TEMPORARY HACK: Establish filterClassesNew as an alias for filterClasses for
+// now, in order to allow testing both in parallel.
+let nodeModule = require("module");
+let origResolve = nodeModule._resolveFilename;
+nodeModule._resolveFilename = function()
+{
+ let params = Array.slice(arguments);
+ if (params[0] == "filterClasses")
+ params[0] = "filterClassesNew";
+ else if (params[0] == "../lib/filterClasses")
+ params[0] = "../lib/filterClassesNew";
+ 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.
+};
require("./node_modules/.bin/nodeunit");
« 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