[site-server] Add 404 response for bad element paths (#1403)

Author: justinfagnani

package-lock.json

@@ -61,8 +61,7 @@
     "@graphql-codegen/cli": "^2.6.2",
     "@graphql-codegen/typed-document-node": "^2.2.8",
     "@graphql-codegen/typescript": "^2.4.8",
-    "@graphql-codegen/typescript-resolvers": "^2.6.1",
-    "uvu": "^0.5.6"
+    "@graphql-codegen/typescript-resolvers": "^2.6.1"
   },
   "dependencies": {
     "@graphql-tools/schema": "^9.0.4",

packages/catalog-api/package.json

@@ -14,6 +14,7 @@ import {
   Package,
 } from 'custom-elements-manifest';
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 
 import {FirestoreRepository} from '../../../lib/firestore/firestore-repository.js';

packages/catalog-server/src/test/lib/firestore/firestore-repository_test.ts

@@ -5,6 +5,7 @@
  */
 
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 import {distTagListToMap, distTagMapToList} from '../../lib/npm.js';
 

packages/catalog-server/src/test/lib/npm_test.ts

@@ -8,6 +8,7 @@ import {readFile} from 'fs/promises';
 import {fileURLToPath} from 'url';
 
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 
 import type {Package} from 'custom-elements-manifest/schema.js';

packages/custom-elements-manifest-tools/src/test/lib/get-custom-elements_test.ts

@@ -5,6 +5,7 @@
  */
 
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 import {parseReferenceString} from '../../lib/reference-string.js';
 

packages/custom-elements-manifest-tools/src/test/lib/reference-string_test.ts

@@ -6,6 +6,7 @@
 
 import type {Package} from 'custom-elements-manifest/schema.js';
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 import {resolveReference} from '../../lib/resolve-reference.js';
 

packages/custom-elements-manifest-tools/src/test/lib/resolve-reference_test.ts

@@ -5,6 +5,7 @@
  */
 
 import {suite} from 'uvu';
+// eslint-disable-next-line import/extensions
 import * as assert from 'uvu/assert';
 
 import {errorCodes, validatePackage} from '../../lib/validate.js';

packages/custom-elements-manifest-tools/src/test/lib/validate_test.ts

@@ -1 +1,2 @@
-lib
+/lib
+/test

packages/site-server/.gitignore

@@ -13,7 +13,8 @@
     "build:server": "wireit",
     "check": "wireit",
     "start:dev": "wireit",
-    "start:prod": "wireit"
+    "start:prod": "wireit",
+    "test": "wireit"
   },
   "wireit": {
     "build": {
@@ -35,7 +36,7 @@
       ]
     },
     "build:server": {
-      "command": "find src -name \"*.ts\" | xargs esbuild --format=esm --target=es2022 --color --outdir=lib",
+      "command": "find src -name \"*.ts\" | xargs esbuild --format=esm --target=es2022 --color --outdir=.",
       "dependencies": [
         "../site-client:build:dev",
         "../custom-elements-manifest-tools:build",
@@ -66,17 +67,29 @@
     },
     "start:dev": {
       "command": "node --enable-source-maps ./lib/dev-server.js",
-      "service": true,
+      "service": {
+        "readyWhen": {
+          "lineMatches": "Web Dev Server started"
+        }
+      },
+      "files": [],
       "dependencies": [
         "build:dev"
       ]
     },
     "start:prod": {
       "command": "node --enable-source-maps ./lib/prod-server.js",
       "service": true,
+      "files": [],
       "dependencies": [
         "build:prod"
       ]
+    },
+    "test": {
+      "command": "NODE_OPTIONS=--enable-source-maps uvu test \"_test\\.js$\"",
+      "dependencies": [
+        "start:dev"
+      ]
     }
   },
   "dependencies": {
@@ -96,5 +109,8 @@
     "koa-conditional-get": "^3.0.0",
     "koa-etag": "^4.0.0",
     "koa-static": "^5.0.0"
+  },
+  "devDependencies": {
+    "uvu": "^0.5.6"
   }
 }