From f56250824cea56355b2ce0a078905eee551cd8ea Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 13:07:10 +0100 Subject: [PATCH 1/9] Adding a debounced updated at timestamp to applications. --- packages/auth/src/redis/index.js | 9 ++- packages/auth/src/redis/utils.js | 1 + .../server/src/api/controllers/application.js | 3 + .../src/api/routes/tests/application.spec.js | 28 +++++++ packages/server/src/middleware/authorized.js | 34 ++------- packages/server/src/middleware/builder.js | 75 +++++++++++++++++++ .../src/middleware/tests/authorized.spec.js | 3 - packages/server/src/utilities/redis.js | 14 +++- 8 files changed, 130 insertions(+), 37 deletions(-) create mode 100644 packages/server/src/middleware/builder.js diff --git a/packages/auth/src/redis/index.js b/packages/auth/src/redis/index.js index 5db80a216b..24ff11d152 100644 --- a/packages/auth/src/redis/index.js +++ b/packages/auth/src/redis/index.js @@ -6,6 +6,7 @@ const { addDbPrefix, removeDbPrefix, getRedisOptions } = require("./utils") const CLUSTERED = false // for testing just generate the client once +let CONNECTED = false let CLIENT = env.isTest() ? new Redis(getRedisOptions()) : null /** @@ -20,9 +21,8 @@ function init() { return resolve(CLIENT) } // if a connection existed, close it and re-create it - if (CLIENT) { - CLIENT.disconnect() - CLIENT = null + if (CLIENT && CONNECTED) { + return CLIENT } const { opts, host, port } = getRedisOptions(CLUSTERED) if (CLUSTERED) { @@ -32,12 +32,15 @@ function init() { } CLIENT.on("end", err => { reject(err) + CONNECTED = false }) CLIENT.on("error", err => { reject(err) + CONNECTED = false }) CLIENT.on("connect", () => { resolve(CLIENT) + CONNECTED = true }) }) } diff --git a/packages/auth/src/redis/utils.js b/packages/auth/src/redis/utils.js index efdd2aa48d..23702353d8 100644 --- a/packages/auth/src/redis/utils.js +++ b/packages/auth/src/redis/utils.js @@ -10,6 +10,7 @@ exports.Databases = { PW_RESETS: "pwReset", INVITATIONS: "invitation", DEV_LOCKS: "devLocks", + DEBOUNCE: "debounce", } exports.getRedisOptions = (clustered = false) => { diff --git a/packages/server/src/api/controllers/application.js b/packages/server/src/api/controllers/application.js index 386c0f1d7a..69b4d8ea54 100644 --- a/packages/server/src/api/controllers/application.js +++ b/packages/server/src/api/controllers/application.js @@ -214,6 +214,9 @@ exports.update = async function (ctx) { const data = ctx.request.body const newData = { ...application, ...data, url } + if (ctx.request.body._rev !== application._rev) { + newData._rev = application._rev + } // the locked by property is attached by server but generated from // Redis, shouldn't ever store it diff --git a/packages/server/src/api/routes/tests/application.spec.js b/packages/server/src/api/routes/tests/application.spec.js index 9783079124..692a5b8e84 100644 --- a/packages/server/src/api/routes/tests/application.spec.js +++ b/packages/server/src/api/routes/tests/application.spec.js @@ -6,7 +6,12 @@ jest.mock("../../../utilities/redis", () => ({ getAllLocks: () => { return [] }, + doesUserHaveLock: () => { + return true + }, updateLock: jest.fn(), + setDebounce: jest.fn(), + checkDebounce: jest.fn(), })) describe("/applications", () => { @@ -104,4 +109,27 @@ describe("/applications", () => { expect(res.body.rev).toBeDefined() }) }) + + describe("edited at", () => { + it("middleware should set edited at", async () => { + const headers = config.defaultHeaders() + headers["referer"] = `/${config.getAppId()}/test` + const res = await request + .put(`/api/applications/${config.getAppId()}`) + .send({ + name: "UPDATED_NAME", + }) + .set(headers) + .expect('Content-Type', /json/) + .expect(200) + expect(res.body.rev).toBeDefined() + // retrieve the app to check it + const getRes = await request + .get(`/api/applications/${config.getAppId()}/appPackage`) + .set(headers) + .expect('Content-Type', /json/) + .expect(200) + expect(getRes.body.application.updatedAt).toBeDefined() + }) + }) }) diff --git a/packages/server/src/middleware/authorized.js b/packages/server/src/middleware/authorized.js index b22fe245d5..78cd9c1db1 100644 --- a/packages/server/src/middleware/authorized.js +++ b/packages/server/src/middleware/authorized.js @@ -4,8 +4,7 @@ const { doesHaveResourcePermission, doesHaveBasePermission, } = require("@budibase/auth/permissions") -const { APP_DEV_PREFIX } = require("../db/utils") -const { doesUserHaveLock, updateLock } = require("../utilities/redis") +const builderMiddleware = require("./builder") function hasResource(ctx) { return ctx.resourceId != null @@ -15,25 +14,7 @@ const WEBHOOK_ENDPOINTS = new RegExp( ["webhooks/trigger", "webhooks/schema"].join("|") ) -async function checkDevAppLocks(ctx) { - const appId = ctx.appId - // if any public usage, don't proceed - if (!ctx.user._id && !ctx.user.userId) { - return - } - - // not a development app, don't need to do anything - if (!appId || !appId.startsWith(APP_DEV_PREFIX)) { - return - } - if (!(await doesUserHaveLock(appId, ctx.user))) { - ctx.throw(403, "User does not hold app lock.") - } - - // they do have lock, update it - await updateLock(appId, ctx.user) -} module.exports = (permType, permLevel = null) => async (ctx, next) => { // webhooks don't need authentication, each webhook unique @@ -45,13 +26,9 @@ module.exports = (permType, permLevel = null) => async (ctx, next) => { return ctx.throw(403, "No user info found") } - const builderCall = permType === PermissionTypes.BUILDER - const referer = ctx.headers["referer"] - const editingApp = referer ? referer.includes(ctx.appId) : false - // this makes sure that builder calls abide by dev locks - if (builderCall && editingApp) { - await checkDevAppLocks(ctx) - } + // check general builder stuff, this middleware is a good way + // to find API endpoints which are builder focused + await builderMiddleware(ctx, permType) const isAuthed = ctx.isAuthenticated const { basePermissions, permissions } = await getUserPermissions( @@ -62,9 +39,10 @@ module.exports = (permType, permLevel = null) => async (ctx, next) => { // builders for now have permission to do anything // TODO: in future should consider separating permissions with an require("@budibase/auth").isClient check let isBuilder = ctx.user && ctx.user.builder && ctx.user.builder.global + const isBuilderApi = permType === PermissionTypes.BUILDER if (isBuilder) { return next() - } else if (builderCall && !isBuilder) { + } else if (isBuilderApi && !isBuilder) { return ctx.throw(403, "Not Authorized") } diff --git a/packages/server/src/middleware/builder.js b/packages/server/src/middleware/builder.js new file mode 100644 index 0000000000..ff03e208b0 --- /dev/null +++ b/packages/server/src/middleware/builder.js @@ -0,0 +1,75 @@ +const { APP_DEV_PREFIX } = require("../db/utils") +const { + doesUserHaveLock, + updateLock, + checkDebounce, + setDebounce, +} = require("../utilities/redis") +const CouchDB = require("../db") +const { DocumentTypes } = require("../db/utils") +const { PermissionTypes } = require("@budibase/auth/permissions") + +const DEBOUNCE_TIME_SEC = 20 + +/************************************************** * + * This middleware has been broken out of the * + * "authorized" middleware as it had nothing to do * + * with authorization, but requires the perms * + * imparted by it. This middleware shouldn't * + * be called directly, it should always be called * + * through the authorized middleware * + ****************************************************/ + +async function checkDevAppLocks(ctx) { + const appId = ctx.appId + + // if any public usage, don't proceed + if (!ctx.user._id && !ctx.user.userId) { + return + } + + // not a development app, don't need to do anything + if (!appId || !appId.startsWith(APP_DEV_PREFIX)) { + return + } + if (!(await doesUserHaveLock(appId, ctx.user))) { + ctx.throw(403, "User does not hold app lock.") + } + + // they do have lock, update it + await updateLock(appId, ctx.user) +} + +async function updateAppUpdatedAt(ctx) { + const appId = ctx.appId + // if debouncing skip this update + // get methods also aren't updating + if (await checkDebounce(appId) || ctx.method === "GET") { + return + } + const db = new CouchDB(appId) + const metadata = await db.get(DocumentTypes.APP_METADATA) + metadata.updatedAt = new Date().toISOString() + await db.put(metadata) + // set a new debounce record with a short TTL + await setDebounce(appId, DEBOUNCE_TIME_SEC) +} + +module.exports = async (ctx, permType) => { + const appId = ctx.appId + // this only functions within an app context + if (!appId) { + return + } + const isBuilderApi = permType === PermissionTypes.BUILDER + const referer = ctx.headers["referer"] + const editingApp = referer ? referer.includes(appId) : false + // check this is a builder call and editing + if (!isBuilderApi || !editingApp) { + return + } + // check locks + await checkDevAppLocks(ctx) + // set updated at time on app + await updateAppUpdatedAt(ctx) +} \ No newline at end of file diff --git a/packages/server/src/middleware/tests/authorized.spec.js b/packages/server/src/middleware/tests/authorized.spec.js index d51ce4cc4d..50e1b1dcf2 100644 --- a/packages/server/src/middleware/tests/authorized.spec.js +++ b/packages/server/src/middleware/tests/authorized.spec.js @@ -78,9 +78,6 @@ describe("Authorization middleware", () => { }) describe("external web hook call", () => { - let ctx = {} - let middleware - beforeEach(() => { config = new TestConfiguration() config.setEnvironment(true) diff --git a/packages/server/src/utilities/redis.js b/packages/server/src/utilities/redis.js index d1fd3de469..18f967e053 100644 --- a/packages/server/src/utilities/redis.js +++ b/packages/server/src/utilities/redis.js @@ -2,13 +2,13 @@ const { Client, utils } = require("@budibase/auth/redis") const { getGlobalIDFromUserMetadataID } = require("../db/utils") const APP_DEV_LOCK_SECONDS = 600 -const DB_NAME = utils.Databases.DEV_LOCKS -let devAppClient +let devAppClient, debounceClient // we init this as we want to keep the connection open all the time // reduces the performance hit exports.init = async () => { - devAppClient = await new Client(DB_NAME).init() + devAppClient = await new Client(utils.Databases.DEV_LOCKS).init() + debounceClient = await new Client(utils.Databases).init() } exports.doesUserHaveLock = async (devAppId, user) => { @@ -52,3 +52,11 @@ exports.clearLock = async (devAppId, user) => { } await devAppClient.delete(devAppId) } + +exports.checkDebounce = async id => { + return debounceClient.get(id) +} + +exports.setDebounce = async (id, seconds) => { + await debounceClient.store(id, "debouncing", seconds) +} From 9531252f417ce9e89b9bf91015b2d5a63eabf133 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 13:08:59 +0100 Subject: [PATCH 2/9] Adding redis to exclusion from coverage (can't currently test). --- packages/server/package.json | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/server/package.json b/packages/server/package.json index 5afb4db968..2af49cd91d 100644 --- a/packages/server/package.json +++ b/packages/server/package.json @@ -66,7 +66,8 @@ "!src/tests/**/*", "!src/automations/tests/**/*", "!src/utilities/fileProcessor.js", - "!src/utilities/fileSystem/**/*" + "!src/utilities/fileSystem/**/*", + "!src/utilities/redis.js" ], "coverageReporters": [ "lcov", From 3b819892509b10a5eef13a7bff020363b2042985 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 14:38:46 +0100 Subject: [PATCH 3/9] Making sure the email sends out the write port if no platform URL is set. --- hosting/docker-compose.yaml | 1 + packages/worker/scripts/dev/manage.js | 1 + packages/worker/src/environment.js | 1 + packages/worker/src/utilities/templates.js | 3 ++- 4 files changed, 5 insertions(+), 1 deletion(-) diff --git a/hosting/docker-compose.yaml b/hosting/docker-compose.yaml index b609f5d962..37657ce009 100644 --- a/hosting/docker-compose.yaml +++ b/hosting/docker-compose.yaml @@ -35,6 +35,7 @@ services: environment: SELF_HOSTED: 1 PORT: 4003 + CLUSTER_PORT: ${MAIN_PORT} JWT_SECRET: ${JWT_SECRET} MINIO_ACCESS_KEY: ${MINIO_ACCESS_KEY} MINIO_SECRET_KEY: ${MINIO_SECRET_KEY} diff --git a/packages/worker/scripts/dev/manage.js b/packages/worker/scripts/dev/manage.js index f7216befb5..b9d28b6278 100644 --- a/packages/worker/scripts/dev/manage.js +++ b/packages/worker/scripts/dev/manage.js @@ -7,6 +7,7 @@ async function init() { const envFileJson = { SELF_HOSTED: 1, PORT: 4002, + CLUSTER_PORT: 10000, JWT_SECRET: "testsecret", INTERNAL_API_KEY: "budibase", MINIO_ACCESS_KEY: "budibase", diff --git a/packages/worker/src/environment.js b/packages/worker/src/environment.js index 11eed33982..384230b9b3 100644 --- a/packages/worker/src/environment.js +++ b/packages/worker/src/environment.js @@ -19,6 +19,7 @@ if (!LOADED && isDev() && !isTest()) { module.exports = { SELF_HOSTED: process.env.SELF_HOSTED, PORT: process.env.PORT, + CLUSTER_PORT: process.env.CLUSTER_PORT, MINIO_ACCESS_KEY: process.env.MINIO_ACCESS_KEY, MINIO_SECRET_KEY: process.env.MINIO_SECRET_KEY, MINIO_URL: process.env.MINIO_URL, diff --git a/packages/worker/src/utilities/templates.js b/packages/worker/src/utilities/templates.js index 86436a2f29..3ac897c10f 100644 --- a/packages/worker/src/utilities/templates.js +++ b/packages/worker/src/utilities/templates.js @@ -7,8 +7,9 @@ const { EmailTemplatePurpose, } = require("../constants") const { checkSlashesInUrl } = require("./index") +const env = require("../environment") -const LOCAL_URL = `http://localhost:10000` +const LOCAL_URL = `http://localhost:${env.CLUSTER_PORT || 10000}` const BASE_COMPANY = "Budibase" exports.getSettingsTemplateContext = async (purpose, code = null) => { From 46534714127f243ed5a01da09af5e891ee8cfdb2 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 14:38:58 +0100 Subject: [PATCH 4/9] Updating UI to use the updated at. --- packages/auth/src/redis/index.js | 6 +----- packages/builder/src/components/start/AppCard.svelte | 9 ++++++++- packages/server/src/api/controllers/application.js | 2 ++ packages/server/src/utilities/redis.js | 2 +- 4 files changed, 12 insertions(+), 7 deletions(-) diff --git a/packages/auth/src/redis/index.js b/packages/auth/src/redis/index.js index 24ff11d152..78e3ea7acd 100644 --- a/packages/auth/src/redis/index.js +++ b/packages/auth/src/redis/index.js @@ -17,13 +17,9 @@ let CLIENT = env.isTest() ? new Redis(getRedisOptions()) : null function init() { return new Promise((resolve, reject) => { // testing uses a single in memory client - if (env.isTest()) { + if (env.isTest() || (CLIENT && CONNECTED)) { return resolve(CLIENT) } - // if a connection existed, close it and re-create it - if (CLIENT && CONNECTED) { - return CLIENT - } const { opts, host, port } = getRedisOptions(CLUSTERED) if (CLUSTERED) { CLIENT = new Redis.Cluster([{ host, port }], opts) diff --git a/packages/builder/src/components/start/AppCard.svelte b/packages/builder/src/components/start/AppCard.svelte index b7e593b7e2..3f38d83dac 100644 --- a/packages/builder/src/components/start/AppCard.svelte +++ b/packages/builder/src/components/start/AppCard.svelte @@ -11,6 +11,7 @@ import { gradient } from "actions" import { auth } from "stores/portal" import { AppStatus } from "constants" + import { processStringSync } from "@budibase/string-templates" export let app export let exportApp @@ -62,7 +63,13 @@
- Updated {Math.floor(1 + Math.random() * 10)} months ago + {#if app.updatedAt} + {processStringSync("Updated {{ duration time 'millisecond' }} ago", { + time: (new Date().getTime() - new Date(app.updatedAt).getTime()) + })} + {:else} + Never updated + {/if} {#if app.deployed}Published{:else}Unpublished{/if} diff --git a/packages/server/src/api/controllers/application.js b/packages/server/src/api/controllers/application.js index 69b4d8ea54..2511686bed 100644 --- a/packages/server/src/api/controllers/application.js +++ b/packages/server/src/api/controllers/application.js @@ -190,6 +190,8 @@ exports.create = async function (ctx) { url: url, template: ctx.request.body.template, instance: instance, + updatedAt: new Date().toISOString(), + createdAt: new Date().toISOString(), deployment: { type: "cloud", }, diff --git a/packages/server/src/utilities/redis.js b/packages/server/src/utilities/redis.js index 18f967e053..8e0f774f42 100644 --- a/packages/server/src/utilities/redis.js +++ b/packages/server/src/utilities/redis.js @@ -8,7 +8,7 @@ let devAppClient, debounceClient // reduces the performance hit exports.init = async () => { devAppClient = await new Client(utils.Databases.DEV_LOCKS).init() - debounceClient = await new Client(utils.Databases).init() + debounceClient = await new Client(utils.Databases.DEBOUNCE).init() } exports.doesUserHaveLock = async (devAppId, user) => { From 96ad77b41dfcd12e8137d7dd86da52f0bac4a66c Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 14:49:59 +0100 Subject: [PATCH 5/9] Formatting --- packages/builder/src/components/start/AppCard.svelte | 2 +- packages/server/src/middleware/authorized.js | 2 -- packages/server/src/middleware/builder.js | 4 ++-- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/packages/builder/src/components/start/AppCard.svelte b/packages/builder/src/components/start/AppCard.svelte index 3f38d83dac..c5937c5637 100644 --- a/packages/builder/src/components/start/AppCard.svelte +++ b/packages/builder/src/components/start/AppCard.svelte @@ -65,7 +65,7 @@ {#if app.updatedAt} {processStringSync("Updated {{ duration time 'millisecond' }} ago", { - time: (new Date().getTime() - new Date(app.updatedAt).getTime()) + time: new Date().getTime() - new Date(app.updatedAt).getTime(), })} {:else} Never updated diff --git a/packages/server/src/middleware/authorized.js b/packages/server/src/middleware/authorized.js index 78cd9c1db1..8ed4c41db7 100644 --- a/packages/server/src/middleware/authorized.js +++ b/packages/server/src/middleware/authorized.js @@ -14,8 +14,6 @@ const WEBHOOK_ENDPOINTS = new RegExp( ["webhooks/trigger", "webhooks/schema"].join("|") ) - - module.exports = (permType, permLevel = null) => async (ctx, next) => { // webhooks don't need authentication, each webhook unique if (WEBHOOK_ENDPOINTS.test(ctx.request.url)) { diff --git a/packages/server/src/middleware/builder.js b/packages/server/src/middleware/builder.js index ff03e208b0..4b94580a62 100644 --- a/packages/server/src/middleware/builder.js +++ b/packages/server/src/middleware/builder.js @@ -44,7 +44,7 @@ async function updateAppUpdatedAt(ctx) { const appId = ctx.appId // if debouncing skip this update // get methods also aren't updating - if (await checkDebounce(appId) || ctx.method === "GET") { + if ((await checkDebounce(appId)) || ctx.method === "GET") { return } const db = new CouchDB(appId) @@ -72,4 +72,4 @@ module.exports = async (ctx, permType) => { await checkDevAppLocks(ctx) // set updated at time on app await updateAppUpdatedAt(ctx) -} \ No newline at end of file +} From b71d751243071a878f554233dcd6ac4451af8516 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 14:56:06 +0100 Subject: [PATCH 6/9] Making sure roles object is always present, issue #1529. --- packages/worker/src/api/controllers/admin/users.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/worker/src/api/controllers/admin/users.js b/packages/worker/src/api/controllers/admin/users.js index d7b198dfdb..e1aa8c1150 100644 --- a/packages/worker/src/api/controllers/admin/users.js +++ b/packages/worker/src/api/controllers/admin/users.js @@ -42,6 +42,10 @@ exports.save = async ctx => { _id: _id || generateGlobalUserID(), password: hashedPassword, } + // make sure the roles object is always present + if (!user.roles) { + user.roles = {} + } // add the active status to a user if its not provided if (user.status == null) { user.status = UserStatus.ACTIVE From ae8e88f4269034a32a042814b0f316d2bd71278c Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 15:03:28 +0100 Subject: [PATCH 7/9] Switching logic for lazy evaluation. --- packages/server/src/middleware/builder.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/server/src/middleware/builder.js b/packages/server/src/middleware/builder.js index 4b94580a62..597ac39fa4 100644 --- a/packages/server/src/middleware/builder.js +++ b/packages/server/src/middleware/builder.js @@ -44,7 +44,7 @@ async function updateAppUpdatedAt(ctx) { const appId = ctx.appId // if debouncing skip this update // get methods also aren't updating - if ((await checkDebounce(appId)) || ctx.method === "GET") { + if (ctx.method === "GET" || (await checkDebounce(appId))) { return } const db = new CouchDB(appId) From 484e22c62d592fa03f3ed5cad0a4ab00f22d25e1 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 15:14:35 +0100 Subject: [PATCH 8/9] Upping debounce to 30 seconds as it has no real negative. --- packages/server/src/middleware/builder.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/server/src/middleware/builder.js b/packages/server/src/middleware/builder.js index 597ac39fa4..240a2d1912 100644 --- a/packages/server/src/middleware/builder.js +++ b/packages/server/src/middleware/builder.js @@ -9,7 +9,7 @@ const CouchDB = require("../db") const { DocumentTypes } = require("../db/utils") const { PermissionTypes } = require("@budibase/auth/permissions") -const DEBOUNCE_TIME_SEC = 20 +const DEBOUNCE_TIME_SEC = 30 /************************************************** * * This middleware has been broken out of the * From 8c8ac18ac0a3fa2774e38b5b889675f51f258ffe Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Fri, 21 May 2021 16:02:21 +0100 Subject: [PATCH 9/9] Fixing broken server test cases. --- packages/server/__mocks__/pg.js | 18 ++++++++++++++---- .../src/api/routes/tests/application.spec.js | 3 ++- .../routes/tests/utilities/TestFunctions.js | 3 ++- .../src/integrations/tests/postgres.spec.js | 12 ++++++------ 4 files changed, 24 insertions(+), 12 deletions(-) diff --git a/packages/server/__mocks__/pg.js b/packages/server/__mocks__/pg.js index 0d8b8cc26a..5d4e3793fd 100644 --- a/packages/server/__mocks__/pg.js +++ b/packages/server/__mocks__/pg.js @@ -1,9 +1,6 @@ const pg = {} -// constructor -function Client() {} - -Client.prototype.query = jest.fn(() => ({ +const query = jest.fn(() => ({ rows: [ { a: "string", @@ -12,8 +9,21 @@ Client.prototype.query = jest.fn(() => ({ ], })) +// constructor +function Client() {} + +Client.prototype.query = query Client.prototype.connect = jest.fn() +Client.prototype.release = jest.fn() + +function Pool() {} +Pool.prototype.query = query +Pool.prototype.connect = jest.fn(() => { + return new Client() +}) pg.Client = Client +pg.Pool = Pool +pg.queryMock = query module.exports = pg diff --git a/packages/server/src/api/routes/tests/application.spec.js b/packages/server/src/api/routes/tests/application.spec.js index 692a5b8e84..dc550cd237 100644 --- a/packages/server/src/api/routes/tests/application.spec.js +++ b/packages/server/src/api/routes/tests/application.spec.js @@ -1,5 +1,6 @@ const { clearAllApps, checkBuilderEndpoint } = require("./utilities/TestFunctions") const setup = require("./utilities") +const { AppStatus } = require("../../../db/utils") jest.mock("../../../utilities/redis", () => ({ init: jest.fn(), @@ -52,7 +53,7 @@ describe("/applications", () => { await config.createApp(request, "app2") const res = await request - .get("/api/applications?status=dev") + .get(`/api/applications?status=${AppStatus.DEV}`) .set(config.defaultHeaders()) .expect('Content-Type', /json/) .expect(200) diff --git a/packages/server/src/api/routes/tests/utilities/TestFunctions.js b/packages/server/src/api/routes/tests/utilities/TestFunctions.js index f10989bf56..c49e44c949 100644 --- a/packages/server/src/api/routes/tests/utilities/TestFunctions.js +++ b/packages/server/src/api/routes/tests/utilities/TestFunctions.js @@ -1,6 +1,7 @@ const rowController = require("../../../controllers/row") const appController = require("../../../controllers/application") const CouchDB = require("../../../../db") +const { AppStatus } = require("../../../../db/utils") function Request(appId, params) { this.appId = appId @@ -14,7 +15,7 @@ exports.getAllTableRows = async config => { } exports.clearAllApps = async () => { - const req = { query: { status: "dev" } } + const req = { query: { status: AppStatus.DEV } } await appController.fetch(req) const apps = req.body if (!apps || apps.length <= 0) { diff --git a/packages/server/src/integrations/tests/postgres.spec.js b/packages/server/src/integrations/tests/postgres.spec.js index 8a8876a556..dc34fcf6bb 100644 --- a/packages/server/src/integrations/tests/postgres.spec.js +++ b/packages/server/src/integrations/tests/postgres.spec.js @@ -20,7 +20,7 @@ describe("Postgres Integration", () => { const response = await config.integration.create({ sql }) - expect(config.integration.client.query).toHaveBeenCalledWith(sql) + expect(pg.queryMock).toHaveBeenCalledWith(sql) }) it("calls the read method with the correct params", async () => { @@ -28,7 +28,7 @@ describe("Postgres Integration", () => { const response = await config.integration.read({ sql }) - expect(config.integration.client.query).toHaveBeenCalledWith(sql) + expect(pg.queryMock).toHaveBeenCalledWith(sql) }) it("calls the update method with the correct params", async () => { @@ -36,20 +36,20 @@ describe("Postgres Integration", () => { const response = await config.integration.update({ sql }) - expect(config.integration.client.query).toHaveBeenCalledWith(sql) + expect(pg.queryMock).toHaveBeenCalledWith(sql) }) it("calls the delete method with the correct params", async () => { const sql = "delete from users where name = 'todelete';" - const response = await config.integration.delete({ + await config.integration.delete({ sql }) - expect(config.integration.client.query).toHaveBeenCalledWith(sql) + expect(pg.queryMock).toHaveBeenCalledWith(sql) }) describe("no rows returned", () => { beforeEach(() => { - config.integration.client.query.mockImplementation(() => ({ rows: [] })) + pg.queryMock.mockImplementation(() => ({ rows: [] })) }) it("returns the correct response when the create response has no rows", async () => {