🔀 Merge pull request #554 from FrostCo/additional_tests

Additional tests

package.json

@@ -53,7 +53,7 @@
     "release": "npm run release:build",
     "test:addon": "npm run build --environment=test --target=firefox-mv2 && npx addons-linter ./dist",
     "test:all": "npm run test && npm run test:lint:tests",
-    "test:cov": "nyc --reporter=lcov --reporter=text npm run test:nocov",
+    "test:cov": "nyc --reporter=lcov --reporter=html --reporter=text npm run test:nocov",
     "test:debug": "node --inspect-brk node_modules/mocha/bin/_mocha --paths --reporter tap",
     "test:lint:all": "npm run test:lint && npm run test:lint:bin && npm run test:lint:tests",
     "test:lint:bin": "eslint \"bin/**/*.*js\"",
@@ -86,6 +86,7 @@
     "mocha": "^10.2.0",
     "npm-check-updates": "^16.14.12",
     "nyc": "^15.1.0",
+    "sinon": "^17.0.2",
     "terser-webpack-plugin": "^5.3.10",
     "ts-loader": "^9.5.1",
     "tsconfig-paths": "^4.2.0",

src/script/lib/filter.ts

@@ -155,14 +155,8 @@ export default class Filter {
                 sub = sub.replace(`\\${i + 1}`, this.cfg.preserveCase ? captureGroup : captureGroup.toLowerCase());
               });
 
-              // Only return if something was substituted
-              if (sub !== word.sub) {
-                if (this.cfg.substitutionMark) {
-                  sub = '[' + sub + ']';
-                }
-
-                return sub;
-              }
+              if (this.cfg.substitutionMark) sub = '[' + sub + ']';
+              return sub;
             }
 
             // Make substitution match case of original match