@teamkeel/functions-runtime 0.405.2 → 0.406.2

package/compose.yaml

@@ -1,6 +1,6 @@
 services:
   postgres:
-    image: postgres:11.13-alpine
+    image: pgvector/pgvector:pg15
     restart: always
     environment:
       - POSTGRES_USER=postgres

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@teamkeel/functions-runtime",
-  "version": "0.405.2",
+  "version": "0.406.2",
   "description": "Internal package used by @teamkeel/sdk",
   "main": "src/index.js",
   "scripts": {
-    "test": "vitest run --reporter verbose --threads false",
+    "test": "vitest run --reporter verbose --pool=threads --poolOptions.threads.singleThread",
     "format": "npx prettier --write src/**/*.js"
   },
   "keywords": [],
@@ -16,7 +16,7 @@
   },
   "devDependencies": {
     "prettier": "3.1.1",
-    "vitest": "^0.34.6"
+    "vitest": "^3.0.7"
   },
   "dependencies": {
     "@aws-sdk/client-s3": "~3.722.0",

package/src/File.js

@@ -14,7 +14,11 @@ const s3Client = (() => {
     return null;
   }
 
-  if (!process.env.TEST_AWS_ENDPOINT) {
+  // Set in integration tests to send all AWS API calls to a test server
+  // for mocking
+  const endpoint = process.env.TEST_AWS_ENDPOINT;
+
+  if (!endpoint) {
     // If no test endpoint is provided then use the env's configuration
     return new S3Client({
       region: process.env.KEEL_REGION,
@@ -22,20 +26,14 @@ const s3Client = (() => {
     });
   }
 
-  // Set in integration tests to send all AWS API calls to a test server
-  // for mocking
-  const endpoint = process.env.TEST_AWS_ENDPOINT;
-
   return new S3Client({
     region: process.env.KEEL_REGION,
 
-    // If a test endpoint is provided then use some test credentials rather than fromEnv()
-    credentials: endpoint
-      ? {
-          accessKeyId: "test",
-          secretAccessKey: "test",
-        }
-      : fromEnv(),
+    // If a test endpoint is provided then use some test credentials
+    credentials: {
+      accessKeyId: "test",
+      secretAccessKey: "test",
+    },
 
     // If a custom endpoint is set we need to use a custom resolver. Just setting the base endpoint isn't enough for S3 as it
     // as the default resolver uses the bucket name as a sub-domain, which likely won't work with the custom endpoint.

package/src/ModelAPI.test.js

@@ -107,7 +107,9 @@ test("ModelAPI.create - throws if not not null constraint violation", async () =
       id: KSUID.randomSync().string,
       name: null,
     })
-  ).rejects.toThrow('null value in column "name" violates not-null constraint');
+  ).rejects.toThrow(
+    'null value in column "name" of relation "author" violates not-null constraint'
+  );
 });
 
 test("ModelAPI.create - throws if database constraint fails", async () => {
@@ -925,7 +927,7 @@ test("ModelAPI.update - throws if not not null constraint violation", async () =
   );
 
   await expect(result).rejects.toThrow(
-    'null value in column "name" violates not-null constraint'
+    'null value in column "name" of relation "author" violates not-null constraint'
   );
 });
 

package/src/applyJoins.js

@@ -1,3 +1,5 @@
+const { snakeCase } = require("./casing");
+
 /**
  * Adds the joins required by the where conditions to the given
  * Kysely instance and returns the resulting new Kysely instance.
@@ -15,7 +17,7 @@ function applyJoins(context, qb, where) {
   const srcTable = context.tableAlias();
 
   for (const key of Object.keys(where)) {
-    const rel = conf[key];
+    const rel = conf[snakeCase(key)];
     if (!rel) {
       continue;
     }

package/src/applyWhereConditions.js

@@ -71,13 +71,12 @@ const opMapping = {
  */
 function applyWhereConditions(context, qb, where = {}) {
   const conf = context.tableConfig();
-
   for (const key of Object.keys(where)) {
     const v = where[key];
 
     // Handle nested where conditions e.g. using a join table
-    if (conf && conf[key]) {
-      const rel = conf[key];
+    if (conf && conf[snakeCase(key)]) {
+      const rel = conf[snakeCase(key)];
       context.withJoin(rel.referencesTable, () => {
         qb = applyWhereConditions(context, qb, v);
       });

package/src/handleJob.test.js

@@ -177,12 +177,14 @@ describe("ModelAPI error handling", () => {
       jsonrpc: "2.0",
       error: {
         code: RuntimeErrors.NotNullConstraintError,
-        message: 'null value in column "title" violates not-null constraint',
+        message:
+          'null value in column "title" of relation "post" violates not-null constraint',
         data: {
           code: "23502",
           column: "title",
           detail: expect.stringContaining("Failing row contains"),
           table: "post",
+          value: undefined,
         },
       },
     });
@@ -229,12 +231,13 @@ describe("ModelAPI error handling", () => {
       error: {
         code: RuntimeErrors.NotNullConstraintError,
         message:
-          'null value in column "author_id" violates not-null constraint',
+          'null value in column "author_id" of relation "post" violates not-null constraint',
         data: {
           code: "23502",
           column: "author_id",
           detail: expect.stringContaining("Failing row contains"),
           table: "post",
+          value: undefined,
         },
       },
     });

package/src/handleRequest.test.js

@@ -374,12 +374,14 @@ describe("ModelAPI error handling", () => {
       jsonrpc: "2.0",
       error: {
         code: RuntimeErrors.NotNullConstraintError,
-        message: 'null value in column "title" violates not-null constraint',
+        message:
+          'null value in column "title" of relation "post" violates not-null constraint',
         data: {
           code: "23502",
           column: "title",
           detail: expect.stringContaining("Failing row contains"),
           table: "post",
+          value: undefined,
         },
       },
     });
@@ -423,12 +425,13 @@ describe("ModelAPI error handling", () => {
       error: {
         code: RuntimeErrors.NotNullConstraintError,
         message:
-          'null value in column "author_id" violates not-null constraint',
+          'null value in column "author_id" of relation "post" violates not-null constraint',
         data: {
           code: "23502",
           column: "author_id",
           detail: expect.stringContaining("Failing row contains"),
           table: "post",
+          value: undefined,
         },
       },
     });