Remove dead code and obsolete TODOs

Author: novemberborn

lib/glob-helpers.cjs

@@ -97,13 +97,9 @@ function normalizeFileForMatching(cwd, file) {
 		file = slash(file);
 	}
 
-	if (!cwd) { // TODO: Ensure tests provide an actual value.
-		return file;
-	}
-
-	// TODO: If `file` is outside `cwd` we can't normalize it. Need to figure
-	// out if that's a real-world scenario, but we may have to ensure the file
-	// isn't even selected.
+	// Note that if `file` is outside `cwd` we can't normalize it. If this turns
+	// out to be a real-world scenario we may have to make changes in calling code
+	// to make sure the file isn't even selected for matching.
 	if (!file.startsWith(cwd)) {
 		return file;
 	}

lib/worker/base.js

@@ -39,8 +39,6 @@ const run = async options => {
 		process.exit(); // eslint-disable-line unicorn/no-process-exit
 	}
 
-	// TODO: Initialize providers here, then pass to lineNumberSelection() so they
-	// can be used to parse the test file.
 	let checkSelectedByLineNumbers;
 	try {
 		checkSelectedByLineNumbers = lineNumberSelection({

test/snapshot-regenerate-report/test.js

@@ -37,7 +37,7 @@ test('snapshot report can be regenerated from .snap file', async t => {
 	});
 
 	// Regenerate report
-	snapshots.hasChanges = true; // TODO this is a hack
+	snapshots.hasChanges = true; // Force.
 	snapshots.save();
 
 	// Assert that reports match