Remove \r in test input, avoid issue running test with CRLF fixtures

zuohaocheng · zuohaocheng · commit a19a9b6fc5af · 2017-03-07T19:30:47.000+08:00
diff --git a/test/index.test.js b/test/index.test.js
@@ -33,7 +33,9 @@ function execLoader(filename, callback) {
 			return this.callback;
 		}
 	};
-	var res = loader.call(context, fs.readFileSync(filename, "utf-8"));
+	// Remove CRs to make test line ending invariant
+	var fixtureContent = fs.readFileSync(filename, "utf-8").replace(/\r/g, '');
+	var res = loader.call(context, fixtureContent);
 	if(!async) return callback(null, res, null, deps, warns);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,9 @@ function execLoader(filename, callback) {`
`33`	`33`	`return this.callback;`
`34`	`34`	`}`
`35`	`35`	`};`
`36`		`- var res = loader.call(context, fs.readFileSync(filename, "utf-8"));`
	`36`	`+ // Remove CRs to make test line ending invariant`
	`37`	`+ var fixtureContent = fs.readFileSync(filename, "utf-8").replace(/\r/g, '');`
	`38`	`+ var res = loader.call(context, fixtureContent);`
`37`	`39`	`if(!async) return callback(null, res, null, deps, warns);`
`38`	`40`	`}`
`39`	`41`