Add more tests

stringparser · stringparser · commit 135287d33667 · 2026-02-07T13:23:52.000+01:00
No need to publish. I haven't changed code
diff --git a/test/api.test.ts b/test/api.test.ts
@@ -0,0 +1,60 @@
+import Parth from '../src';
+
+describe('api', () => {
+  describe('invalid inputs', () => {
+    it('set(non-string) returns this and does not add a route', () => {
+      const parth = new Parth();
+      const chain = parth.set('get /:id').set(123 as unknown as string);
+      expect(chain).toBe(parth);
+      expect(parth.get('get /1')).toMatchObject({ params: { id: '1' } });
+      expect(parth.get('123')).toBeNull();
+    });
+
+    it('get(non-string) returns null', () => {
+      const parth = new Parth().set('get /:id');
+      expect(parth.get(123 as unknown as string)).toBeNull();
+      expect(parth.get(null as unknown as string)).toBeNull();
+      expect(parth.get(undefined as unknown as string)).toBeNull();
+    });
+  });
+
+  describe('no match', () => {
+    it('get(path) returns null when no route matches', () => {
+      const parth = new Parth().set('get /:id');
+      expect(parth.get('post /1')).toBeNull();
+      expect(parth.get('get')).toBeNull();
+      expect(parth.get('')).toBeNull();
+    });
+  });
+
+  describe('exact store match', () => {
+    it('get(path) when path is exact registered path returns early with empty notFound and params', () => {
+      const parth = new Parth().set('get /hello/:there');
+      const exactPath = 'get /hello/:there';
+      const result = parth.get(exactPath);
+      expect(result).not.toBeNull();
+      expect(result!.match).toBe(exactPath);
+      expect(result!.notFound).toBe('');
+      expect(result!.params).toEqual({});
+    });
+  });
+
+  describe('constructor', () => {
+    it('can be called with no options', () => {
+      const parth = new Parth();
+      parth.set('/:a');
+      expect(parth.get('/x')).toMatchObject({ params: { a: 'x' } });
+    });
+  });
+
+  describe('result immutability', () => {
+    it('get() returns a clone; mutating result does not affect store', () => {
+      const parth = new Parth().set('/:id');
+      const result1 = parth.get('/foo');
+      const result2 = parth.get('/foo');
+      expect(result1!.params.id).toBe('foo');
+      result1!.params.id = 'mutated';
+      expect(result2!.params.id).toBe('foo');
+    });
+  });
+});
diff --git a/test/lib.test.ts b/test/lib.test.ts
@@ -0,0 +1,21 @@
+import { getQueryString } from '../src/lib';
+
+describe('lib getQueryString', () => {
+  it('returns null for null or undefined url', () => {
+    expect(getQueryString(null)).toBeNull();
+    expect(getQueryString(undefined)).toBeNull();
+  });
+
+  it('returns null when url has no ?', () => {
+    expect(getQueryString('http://example.com/path')).toBeNull();
+  });
+
+  it('returns query string from ? to end when ? is present', () => {
+    expect(getQueryString('http://example.com?foo=1')).toBe('?foo=1');
+    expect(getQueryString('/path?query=string')).toBe('?query=string');
+  });
+
+  it('returns null when ? is followed by : (so ?: is not a query string)', () => {
+    expect(getQueryString('get /path?:optional')).toBeNull();
+  });
+});
diff --git a/test/match-priority.test.ts b/test/match-priority.test.ts
@@ -0,0 +1,20 @@
+import Parth from '../src';
+
+describe('match priority', () => {
+  it('deeper path wins over shallower when both match', () => {
+    const parth = new Parth();
+    parth.set('/:a').set('/:a/:b');
+    const result = parth.get('/x/y');
+    expect(result).not.toBeNull();
+    expect(result!.path).toBe('/:a/:b');
+    expect(result!.params).toEqual({ a: 'x', b: 'y' });
+  });
+
+  it('when depth is equal, stem localeCompare determines winner', () => {
+    const parth = new Parth();
+    parth.set('/foo/:id');
+    parth.set('/bar/:id');
+    expect(parth.get('/foo/1')).toMatchObject({ path: '/foo/:id', params: { id: '1' } });
+    expect(parth.get('/bar/1')).toMatchObject({ path: '/bar/:id', params: { id: '1' } });
+  });
+});
diff --git a/test/normalization.test.ts b/test/normalization.test.ts
@@ -0,0 +1,18 @@
+import Parth from '../src';
+
+describe('normalization', () => {
+  it('set trims and collapses whitespace in path', () => {
+    const parth = new Parth();
+    parth.set('  get   /hello/:there  ');
+    const result = parth.get('get /hello/world');
+    expect(result).not.toBeNull();
+    expect(result!.params).toEqual({ there: 'world' });
+  });
+
+  it('get trims and collapses whitespace in input path', () => {
+    const parth = new Parth().set('get /hello/:there');
+    const result = parth.get('  get   /hello/world  ');
+    expect(result).not.toBeNull();
+    expect(result!.params).toEqual({ there: 'world' });
+  });
+});
diff --git a/test/set-merge.test.ts b/test/set-merge.test.ts
@@ -0,0 +1,20 @@
+import Parth from '../src';
+
+describe('set merge', () => {
+  it('setting same path twice merges options into existing entry', () => {
+    const parth = new Parth();
+    parth.set('/hello/:id');
+    parth.set('/hello/:id', { path: '/hello/:id' });
+    const result = parth.get('/hello/42');
+    expect(result).not.toBeNull();
+    expect(result!.params).toEqual({ id: '42' });
+  });
+
+  it('set(path, opt) passes optional PathOpt into stored entry', () => {
+    const parth = new Parth();
+    parth.set('/a', { path: '/a', custom: 'value' } as unknown as { path?: string; custom?: string });
+    const result = parth.get('/a');
+    expect(result).not.toBeNull();
+    expect((result as { custom?: string })!.custom).toBe('value');
+  });
+});
diff --git a/test/verify-publish/package.json b/test/verify-publish/package.json
@@ -3,6 +3,6 @@
   "private": true,
   "type": "module",
   "dependencies": {
-    "parth": "file:../../parth-4.2.3.tgz"
+    "parth": "file:../../parth-5.0.1.tgz"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,6 @@`
`3`	`3`	`"private": true,`
`4`	`4`	`"type": "module",`
`5`	`5`	`"dependencies": {`
`6`		`- "parth": "file:../../parth-4.2.3.tgz"`
	`6`	`+ "parth": "file:../../parth-5.0.1.tgz"`
`7`	`7`	`}`
`8`	`8`	`}`