diff --git a/fixup.sh b/fixup.sh index 684b147a..9879956e 100755 --- a/fixup.sh +++ b/fixup.sh @@ -4,7 +4,7 @@ cat >> types/layout.d.ts < { - // @ts-ignore this.httpProxy.register(key, item.newVal); }); } @@ -561,7 +554,6 @@ export default class PodiumLayout { incoming.css = [...this.cssRoute]; incoming.js = [...this.jsRoute]; - // @ts-ignore if (context) await this.context.process(incoming); if (proxy) await this.httpProxy.process(incoming); @@ -587,7 +579,7 @@ export default class PodiumLayout { middleware() { return async (req, res, next) => { const incoming = new HttpIncoming(req, res, res.locals); - // @ts-ignore + // @ts-expect-error The API here is a bit odd. You assign a string, but the `set` constructs a URL from it. incoming.url = `${req.protocol}://${req.get('host')}${ req.originalUrl }`; @@ -660,11 +652,7 @@ export default class PodiumLayout { #sanitize(uri, prefix = false) { const pathname = prefix ? this.#pathname : ''; if (uri) { - // @ts-ignore - return uriIsRelative(uri) - ? // @ts-ignore - pathnameBuilder(pathname, uri) - : uri; + return uriIsRelative(uri) ? pathnameBuilder(pathname, uri) : uri; } return uri; } diff --git a/package.json b/package.json index 11d67c89..f8b9329a 100644 --- a/package.json +++ b/package.json @@ -32,25 +32,25 @@ "scripts": { "lint": "eslint .", "lint:fix": "eslint --fix .", - "test": "tap --disable-coverage --allow-empty-coverage", + "test": "tap --disable-coverage --allow-empty-coverage && tsc --project tsconfig.test.json", "test:snapshots": "tap --snapshot --disable-coverage --allow-empty-coverage", "types": "tsc --declaration --emitDeclarationOnly && ./fixup.sh" }, "dependencies": { "@metrics/client": "2.5.2", - "@podium/client": "5.0.10", - "@podium/context": "5.0.4", - "@podium/proxy": "5.0.8", - "@podium/schemas": "5.0.0", - "@podium/utils": "5.0.3", - "abslog": "2.4.2", + "@podium/client": "5.0.21", + "@podium/context": "5.0.13", + "@podium/proxy": "5.0.16", + "@podium/schemas": "5.0.2", + "@podium/utils": "5.0.6", + "abslog": "2.4.4", "ajv": "8.12.0", "lodash.merge": "4.6.2", "objobj": "1.0.0" }, "devDependencies": { "@babel/eslint-parser": "7.23.3", - "@podium/podlet": "5.1.2", + "@podium/podlet": "5.1.4", "@podium/test-utils": "2.5.2", "@semantic-release/changelog": "6.0.3", "@semantic-release/commit-analyzer": "11.1.0", diff --git a/tap-snapshots/tests/layout.js.test.cjs b/tap-snapshots/tests/layout.test.js.test.cjs similarity index 79% rename from tap-snapshots/tests/layout.js.test.cjs rename to tap-snapshots/tests/layout.test.js.test.cjs index e6602624..c57230aa 100644 --- a/tap-snapshots/tests/layout.js.test.cjs +++ b/tap-snapshots/tests/layout.test.js.test.cjs @@ -5,7 +5,7 @@ * Make sure to inspect the output below. Do not ignore changes! */ 'use strict' -exports[`tests/layout.js > TAP > Layout() - rendering using a string - with assets > must match snapshot 1`] = ` +exports[`tests/layout.test.js > TAP > Layout() - rendering using a string - with assets > must match snapshot 1`] = `
@@ -25,6 +25,6 @@ exports[`tests/layout.js > TAP > Layout() - rendering using a string - with asse ` -exports[`tests/layout.js > TAP > Layout() - setting a custom view template > must match snapshot 1`] = ` +exports[`tests/layout.test.js > TAP > Layout() - setting a custom view template > must match snapshot 1`] = ` extra head stuff