PR Comments

Author: rosen-vladimirov

queue.ts

@@ -4,7 +4,7 @@ export class Queue<T> implements IQueue<T> {
 	private future: IFuture<void>;
 
 	public constructor(private items?: T[]) {
-		items = items === undefined ? [] : items;
+		this.items = this.items === undefined ? [] : this.items;
 	}
 
 	public enqueue(item: T): void {

test-scripts/istanbul.js

@@ -2,16 +2,17 @@
 
 const childProcess = require("child_process");
 const path = require("path");
-const pathToIstanbul = path.join(__dirname, "..", "node_modules", "istanbul", "lib", "cli.js");
-const pathToMocha = path.join(__dirname, "..", "node_modules", "mocha", "bin", "_mocha");
+const pathToNodeModules = path.join(__dirname, "..", "node_modules");
+const pathToIstanbul = path.join(pathToNodeModules, "istanbul", "lib", "cli.js");
+const pathToMocha = path.join(pathToNodeModules, "mocha", "bin", "_mocha");
 
 const istanbulArgs = [ pathToIstanbul, "cover", pathToMocha ];
 
 const nodeArgs = require("./node-args").getNodeArgs();
 
 const args = nodeArgs.concat(istanbulArgs);
 
-const nodeProcess = childProcess.spawn("node", args, { stdio: "inherit"});
+const nodeProcess = childProcess.spawn("node", args, { stdio: "inherit" });
 nodeProcess.on("close", (code) => {
 	// We need this handler so if any test fails, we'll exit with same exit code as istanbul.
 	process.exit(code);

test-scripts/mocha.js

@@ -8,7 +8,7 @@ const nodeArgs = require("./node-args").getNodeArgs();
 
 const args = nodeArgs.concat(pathToMocha);
 
-const nodeProcess = childProcess.spawn("node", args, { stdio: "inherit"});
+const nodeProcess = childProcess.spawn("node", args, { stdio: "inherit" });
 nodeProcess.on("close", (code) => {
 	// We need this handler so if any test fails, we'll exit with same exit code as mocha.
 	process.exit(code);