Implement /update

A 500 just seems more appropriate in this case IMO.

.forgejo/workflows/boot-test.yml

@@ -1,18 +0,0 @@
-name: "Make sure the server is actually able to start"
-author: "9pfs@amcforum.wiki (h @ nest)"
-on:
-  push:
-    branches:
-      - "main"
-      - "fp-edits"
-  pull_request:
-    types: [opened, synchronize, reopened]
-jobs:
-  boot-test:
-    runs-on: docker
-    steps:
-      - uses: actions/checkout@v3
-      - id: dependencies
-        run: NODE_ENV=development npm ci
-      - id: boot-test
-        run: TEST_MODE=boot LISTEN_PATH=/tmp/webserver.sock node index.js

.forgejo/workflows/node-check.yml

@@ -1,16 +0,0 @@
-name: "Run node --check on everything"
-author: "9pfs@amcforum.wiki (h @ nest)"
-on:
-  push:
-    branches:
-      - "main"
-      - "fp-edits"
-  pull_request:
-    types: [opened, synchronize, reopened]
-jobs:
-  node-check:
-    runs-on: docker
-    steps:
-      - uses: actions/checkout@v3
-      - id: check
-        run: for a in `find|grep -v node_modules|grep '\.js$'`; do node --check $a && echo "$a   OK"; done

.forgejo/workflows/prettier.yml

@@ -1,18 +0,0 @@
-name: "Check project using prettier"
-author: "9pfs@amcforum.wiki (h @ nest)"
-on:
-  push:
-    branches:
-      - "main"
-      - "fp-edits"
-  pull_request:
-    types: [opened, synchronize, reopened]
-jobs:
-  prettier:
-    runs-on: docker
-    steps:
-      - uses: actions/checkout@v3
-      - id: dependencies
-        run: NODE_ENV=development npm ci
-      - id: prettier
-        run: npx prettier --check .

.forgejo/workflows/tests.yml

@@ -0,0 +1,46 @@
+on:
+  push:
+    branches:
+      - "main"
+      - "fp-edits"
+  pull_request:
+    types: [opened, synchronize, reopened]
+jobs:
+  tests:
+    runs-on: docker
+    container: node:20-bookworm
+    services:
+      test-db:
+        image: postgres:16
+        env:
+          POSTGRES_DB: test-db
+          POSTGRES_PASSWORD: postgres
+    steps:
+      - uses: actions/checkout@v3
+      - run: apt-get update
+      - run: apt-get install -y postgresql-client
+      - run: NODE_ENV=production npm ci
+      - run: cat migrations/*.sql|PGPASSWORD=postgres psql -h test-db -U postgres test-db
+      - run: node tests/index.js
+  prettier:
+    runs-on: docker
+    steps:
+      - uses: actions/checkout@v3
+      - id: dependencies
+        run: NODE_ENV=development npm ci
+      - id: prettier
+        run: npx prettier --check .
+  node-check:
+    runs-on: docker
+    steps:
+      - uses: actions/checkout@v3
+      - id: check
+        run: for a in `find|grep -v node_modules|grep '\.js$'`; do node --check $a && echo "$a   OK"; done
+  boot:
+    runs-on: docker
+    steps:
+      - uses: actions/checkout@v3
+      - id: dependencies
+        run: NODE_ENV=production npm ci
+      - id: boot-test
+        run: TEST_MODE=boot LISTEN_PATH=/tmp/webserver.sock node index.js

index.js

@@ -108,13 +108,35 @@ app.get("/hit/:ns/:key", async function (req, res) {
     true,
   );
   return res
-    .status(counter_res == null ? 403 : 200)
+    .status(counter_res == null ? 500 : 200)
     .json({ value: counter_res });
 });
 app.get("/hit/:key", async function (req, res) {
   var counter_res = await counter_increment("default", req.params.key, 1, true);
   return res
-    .status(counter_res == null ? 403 : 200)
+    .status(counter_res == null ? 500 : 200)
+    .json({ value: counter_res });
+});
+app.get("/update/:ns/:key", async function (req, res) {
+  var counter_res = await counter_increment(
+    req.params.ns,
+    req.params.key,
+    req.query.amount,
+    false,
+  );
+  return res
+    .status(counter_res == null ? 500 : 200)
+    .json({ value: counter_res });
+});
+app.get("/update/:key", async function (req, res) {
+  var counter_res = await counter_increment(
+    "default",
+    req.params.key,
+    req.query.amount,
+    false,
+  );
+  return res
+    .status(counter_res == null ? 500 : 200)
     .json({ value: counter_res });
 });
 app.get("/get/:ns/:key", async function (req, res) {

migrations/0003-keys-namespace-id-not-null.sql

@@ -1,7 +1,7 @@
 -- Make sure that keys.namespace_id isn't able to be null
 BEGIN;
 -- Remove all null keys.namespace_id values. Nuking data like this is reckless, but it's the only option.
-DROP FROM keys WHERE namespace_id = NULL;
+DELETE FROM keys WHERE namespace_id = NULL;
 -- Delete before adding to avoid conflicts
 ALTER TABLE keys ALTER COLUMN namespace_id DROP NOT NULL;
 ALTER TABLE keys ALTER COLUMN namespace_id SET NOT NULL;

tests/db-sanity.js

@@ -0,0 +1 @@
+import "./test-db.js";

tests/index.js

@@ -0,0 +1,4 @@
+// Each test should be imported here.
+import "./db-sanity.js";
+// This should be the last line.
+console.log("Tests successful.");

tests/test-db.js

@@ -0,0 +1,4 @@
+/* only intended to be used in the forgejo actions test environment */
+import postgres from "postgres";
+const sql = postgres("postgres://postgres:postgres@test-db/test-db");
+export default sql;