From a9e29d83e9308cefeafd21d519e04224c78407d4 Mon Sep 17 00:00:00 2001
From: Andrew Kingston <andrew@kingston.dev>
Date: Tue, 11 Mar 2025 15:52:33 +0000
Subject: [PATCH] Add tests to ensure sample data is created for new apps but
 not for imports or templates

---
 .../src/api/routes/tests/application.spec.ts  | 28 ++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

diff --git a/packages/server/src/api/routes/tests/application.spec.ts b/packages/server/src/api/routes/tests/application.spec.ts
index e0b295f78e..f049610794 100644
--- a/packages/server/src/api/routes/tests/application.spec.ts
+++ b/packages/server/src/api/routes/tests/application.spec.ts
@@ -127,10 +127,26 @@ describe("/applications", () => {
   })
 
   describe("create", () => {
-    it("creates empty app", async () => {
+    const checkScreenCount = async (expectedCount: number) => {
+      const res = await config.api.application.getDefinition(
+        config.getProdAppId()
+      )
+      expect(res.screens.length).toEqual(expectedCount)
+    }
+
+    const checkTableCount = async (expectedCount: number) => {
+      const tables = await config.api.table.fetch()
+      expect(tables.length).toEqual(expectedCount)
+    }
+
+    it("creates empty app with sample data", async () => {
       const app = await config.api.application.create({ name: utils.newid() })
       expect(app._id).toBeDefined()
       expect(events.app.created).toHaveBeenCalledTimes(1)
+
+      // Ensure we created sample resources
+      await checkScreenCount(1)
+      await checkTableCount(5)
     })
 
     it("creates app from template", async () => {
@@ -149,6 +165,11 @@ describe("/applications", () => {
       expect(app._id).toBeDefined()
       expect(events.app.created).toHaveBeenCalledTimes(1)
       expect(events.app.templateImported).toHaveBeenCalledTimes(1)
+
+      // Ensure we did not create sample data. This template includes exactly
+      // this many of each resource.
+      await checkScreenCount(1)
+      await checkTableCount(5)
     })
 
     it("creates app from file", async () => {
@@ -160,6 +181,11 @@ describe("/applications", () => {
       expect(app._id).toBeDefined()
       expect(events.app.created).toHaveBeenCalledTimes(1)
       expect(events.app.fileImported).toHaveBeenCalledTimes(1)
+
+      // Ensure we did not create sample data. This file includes exactly
+      // this many of each resource.
+      await checkScreenCount(1)
+      await checkTableCount(5)
     })
 
     it("should apply authorization to endpoint", async () => {