fix(mocha-runner): properly restrict tests to run

Correctly restrict tests to run by including the start and end characters (`^` and `$`) inside the `--grep` regex.
stryker-mutator · Aug 1, 2024 · f9ef08e · f9ef08e
1 parent 23dcc8f
commit f9ef08e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/mocha-runner/src/mocha-test-runner.ts b/packages/mocha-runner/src/mocha-test-runner.ts
@@ -114,7 +114,7 @@ export class MochaTestRunner implements TestRunner {
     this.instrumenterContext.hitLimit = hitLimit;
     this.instrumenterContext.hitCount = hitLimit ? 0 : undefined;
     if (testFilter) {
-      const metaRegExp = testFilter.map((testId) => `(${escapeRegExp(testId)})`).join('|');
+      const metaRegExp = testFilter.map((testId) => `(^${escapeRegExp(testId)}$)`).join('|');
       const regex = new RegExp(metaRegExp);
       this.mocha.grep(regex);
     } else {

diff --git a/packages/mocha-runner/test/unit/mocha-test-runner.spec.ts b/packages/mocha-runner/test/unit/mocha-test-runner.spec.ts
@@ -314,12 +314,12 @@ describe(MochaTestRunner.name, () => {
 
     it('should use `grep` to when the test filter is specified', async () => {
       await actMutantRun(factory.mutantRunOptions({ testFilter: ['foo should be bar', 'baz should be qux'] }));
-      expect(mocha.grep).calledWith(new RegExp('(foo should be bar)|(baz should be qux)'));
+      expect(mocha.grep).calledWith(new RegExp('(^foo should be bar$)|(^baz should be qux$)'));
     });
 
     it('should escape regex characters when filtering', async () => {
       await actMutantRun(factory.mutantRunOptions({ testFilter: ['should escape *.\\, but not /'] }));
-      expect(mocha.grep).calledWith(new RegExp('(should escape \\*\\.\\\\, but not /)'));
+      expect(mocha.grep).calledWith(new RegExp('(^should escape \\*\\.\\\\, but not /$)'));
     });
 
     it('should be able to report a killed mutant when a test fails', async () => {