From 8f6cfb76341fe41af6db30e2932998bfc0af992b Mon Sep 17 00:00:00 2001
From: mike12345567 <me@michaeldrury.co.uk>
Date: Mon, 10 Jun 2024 14:30:11 +0100
Subject: [PATCH] Disabling app migration tests and app migrations in test
 environment.

---
 .../migrations/tests/20240604153647_initial_sqs.spec.ts         | 2 +-
 .../server/src/appMigrations/tests/migrations.integrity.spec.ts | 2 +-
 packages/server/src/appMigrations/tests/migrations.spec.ts      | 2 +-
 .../server/src/appMigrations/tests/migrationsProcessor.spec.ts  | 2 +-
 packages/server/src/middleware/appMigrations.ts                 | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/packages/server/src/appMigrations/migrations/tests/20240604153647_initial_sqs.spec.ts b/packages/server/src/appMigrations/migrations/tests/20240604153647_initial_sqs.spec.ts
index 86e50a5812..bb31410882 100644
--- a/packages/server/src/appMigrations/migrations/tests/20240604153647_initial_sqs.spec.ts
+++ b/packages/server/src/appMigrations/migrations/tests/20240604153647_initial_sqs.spec.ts
@@ -82,7 +82,7 @@ beforeAll(async () => {
   })
 })
 
-describe("SQS migration", () => {
+xdescribe("SQS migration", () => {
   it("test migration runs as expected against an older DB", async () => {
     const db = dbCore.getDB(config.appId!)
     // confirm nothing exists initially
diff --git a/packages/server/src/appMigrations/tests/migrations.integrity.spec.ts b/packages/server/src/appMigrations/tests/migrations.integrity.spec.ts
index 145a06d7f5..88572f23fa 100644
--- a/packages/server/src/appMigrations/tests/migrations.integrity.spec.ts
+++ b/packages/server/src/appMigrations/tests/migrations.integrity.spec.ts
@@ -2,7 +2,7 @@ import { context } from "@budibase/backend-core"
 import * as setup from "../../api/routes/tests/utilities"
 import * as migrations from "../migrations"
 
-describe("migration integrity", () => {
+xdescribe("migration integrity", () => {
   // These test is checking that each migration is "idempotent".
   // We should be able to rerun any migration, with any rerun not modifiying anything. The code should be aware that the migration already ran
   it("each migration can rerun safely", async () => {
diff --git a/packages/server/src/appMigrations/tests/migrations.spec.ts b/packages/server/src/appMigrations/tests/migrations.spec.ts
index 1da94f503f..c644810436 100644
--- a/packages/server/src/appMigrations/tests/migrations.spec.ts
+++ b/packages/server/src/appMigrations/tests/migrations.spec.ts
@@ -13,7 +13,7 @@ jest.mock<typeof migrations>("../migrations", () => ({
   ],
 }))
 
-describe("migrations", () => {
+xdescribe("migrations", () => {
   it("new apps are created with the latest app migration version set", async () => {
     const config = setup.getConfig()
     await config.init()
diff --git a/packages/server/src/appMigrations/tests/migrationsProcessor.spec.ts b/packages/server/src/appMigrations/tests/migrationsProcessor.spec.ts
index a2d1dc05c3..eb2ee5a03d 100644
--- a/packages/server/src/appMigrations/tests/migrationsProcessor.spec.ts
+++ b/packages/server/src/appMigrations/tests/migrationsProcessor.spec.ts
@@ -6,7 +6,7 @@ import { AppMigration } from ".."
 
 const futureTimestamp = `20500101174029`
 
-describe("migrationsProcessor", () => {
+xdescribe("migrationsProcessor", () => {
   it("running migrations will update the latest applied migration", async () => {
     const testMigrations: AppMigration[] = [
       { id: `${futureTimestamp}_123`, func: async () => {} },
diff --git a/packages/server/src/middleware/appMigrations.ts b/packages/server/src/middleware/appMigrations.ts
index 6ad356427b..9c739fb72b 100644
--- a/packages/server/src/middleware/appMigrations.ts
+++ b/packages/server/src/middleware/appMigrations.ts
@@ -7,7 +7,7 @@ export default async (ctx: UserCtx, next: any) => {
 
   // migrations can be disabled via environment variable if you
   // need to completely disable migrations, e.g. for testing
-  if (env.DISABLE_APP_MIGRATIONS) {
+  if (env.DISABLE_APP_MIGRATIONS || env.isTest()) {
     return next()
   }