fix: check matcher test types (4Catalyzer#1057)

package.json

@@ -38,7 +38,7 @@
     "release": "4c release --conventional-commits",
     "tdd": "jest --watch",
     "test": "yarn lint && yarn test:ts && yarn testonly -- --coverage",
-    "test:ts": "dtslint --expectOnly types && yarn tsc --noEmit",
+    "test:ts": "dtslint --expectOnly types && yarn tsc --noEmit && yarn tsc --noEmit -p test",
     "testonly": "jest --runInBand --verbose",
     "docs": "yarn --cwd www start"
   },

test/Matcher.test.js → test/Matcher.test.ts

@@ -1,4 +1,9 @@
 import Matcher from '../src/Matcher';
+import {
+  type IsActiveOptions,
+  type LocationDescriptorObject,
+  Match,
+} from '../src/typeUtils';
 
 describe('Matcher', () => {
   describe('route hierarchies', () => {
@@ -32,11 +37,11 @@ describe('Matcher', () => {
       ['nested matching', '/foo/bar', [0, 1]],
       ['route fallthrough', '/foo/baz', [2]],
     ].forEach(([scenario, pathname, expectedRouteIndices]) => {
-      describe(scenario, () => {
+      describe(scenario as string, () => {
         it('should be supported', () => {
           expect(
             matcher.match({
-              pathname,
+              pathname: pathname as string,
             }),
           ).toMatchObject({
             routeIndices: expectedRouteIndices,
@@ -265,7 +270,7 @@ describe('Matcher', () => {
       it(`should match ${scenario}`, () => {
         expect(
           matcher.match({
-            pathname,
+            pathname: pathname as string,
           }),
         ).toMatchObject({
           routeIndices: expectedRouteIndices,
@@ -350,7 +355,7 @@ describe('Matcher', () => {
   });
 
   describe('#joinPaths', () => {
-    const matcher = new Matcher();
+    const matcher = new Matcher([]);
 
     [
       ['no extra slashes', '/foo', 'bar'],
@@ -359,6 +364,7 @@ describe('Matcher', () => {
       ['slashes everywhere', '/foo/', '/bar'],
     ].forEach(([scenario, basePath, path]) => {
       it(`should support ${scenario}`, () => {
+        // @ts-ignore
         expect(matcher.joinPaths(basePath, path)).toBe('/foo/bar');
       });
     });
@@ -415,7 +421,11 @@ describe('Matcher', () => {
       ].forEach(([scenario, matchLocation, location, options]) => {
         it(`should be active on ${scenario}`, () => {
           expect(
-            matcher.isActive({ location: matchLocation }, location, options),
+            matcher.isActive(
+              { location: matchLocation } as any as Match,
+              location as LocationDescriptorObject,
+              options as IsActiveOptions,
+            ),
           ).toBe(true);
         });
       });
@@ -459,7 +469,11 @@ describe('Matcher', () => {
       ].forEach(([scenario, matchLocation, location, options]) => {
         it(`should not be active on ${scenario}`, () => {
           expect(
-            matcher.isActive({ location: matchLocation }, location, options),
+            matcher.isActive(
+              { location: matchLocation } as any as Match,
+              location as LocationDescriptorObject,
+              options as IsActiveOptions,
+            ),
           ).toBe(false);
         });
       });

test/tsconfig.json

@@ -0,0 +1,4 @@
+{
+  "extends": "../tsconfig.json",
+  "include": ["**/*.ts"]
+}