Revert "ci: Run tests on push (hoarder-app#807)" (hoarder-app#808)

This reverts commit 39358b1.

Author: MohamedBassem

.github/workflows/ci.yml

@@ -48,16 +48,6 @@ jobs:
 
       - name: Typecheck
         run: pnpm typecheck
-  tests:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Setup
-        uses: ./tooling/github/setup
-
-      - name: Tests
-        run: pnpm test
   open-api-spec:
     runs-on: ubuntu-latest
     steps:

apps/web/package.json

@@ -10,6 +10,7 @@
     "build": "next build",
     "start": "next start",
     "lint": "next lint",
+    "test": "vitest",
     "typecheck": "tsc --noEmit",
     "format": "prettier --check . --ignore-path ../../.gitignore",
     "format:fix": "prettier --write . --ignore-path ../../.gitignore",

package.json

@@ -19,8 +19,7 @@
     "lint:fix": "turbo --no-daemon lint --continue -- --fix --cache-location node_modules/.cache/.eslintcache",
     "lint:ws": "pnpm dlx sherif@latest",
     "postinstall": "pnpm lint:ws",
-    "typecheck": "turbo --no-daemon typecheck",
-    "test": "turbo --no-daemon test --continue"
+    "typecheck": "turbo --no-daemon typecheck"
   },
   "devDependencies": {
     "@hoarder/prettier-config": "workspace:^0.1.0",

packages/db/drizzle.ts

@@ -12,7 +12,7 @@ export const db = drizzle(sqlite, { schema });
 
 export function getInMemoryDB(runMigrations: boolean) {
   const mem = new Database(":memory:");
-  const db = drizzle(mem, { schema, logger: false });
+  const db = drizzle(mem, { schema, logger: true });
   if (runMigrations) {
     migrate(db, { migrationsFolder: path.resolve(__dirname, "./drizzle") });
   }

packages/shared/package.json

@@ -26,8 +26,7 @@
     "format:fix": "prettier . --write --ignore-path ../../.prettierignore",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
-    "test": "vitest run",
-    "test:watch": "vitest"
+    "test": "vitest"
   },
   "main": "index.ts",
   "eslintConfig": {

packages/trpc/package.json

@@ -10,8 +10,7 @@
     "format:fix": "prettier . --write --ignore-path ../../.prettierignore",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
-    "test": "vitest run",
-    "test:watch": "vitest"
+    "test": "vitest"
   },
   "dependencies": {
     "@hoarder/db": "workspace:*",

packages/trpc/testUtils.ts

@@ -1,6 +1,5 @@
 import { getInMemoryDB } from "@hoarder/db/drizzle";
 import { users } from "@hoarder/db/schema";
-import { runQueueDBMigrations } from "@hoarder/shared/queues";
 
 import { createCallerFactory } from "./index";
 import { appRouter } from "./routers/_app";
@@ -55,7 +54,6 @@ export interface CustomTestContext {
 export async function buildTestContext(
   seedDB: boolean,
 ): Promise<CustomTestContext> {
-  runQueueDBMigrations();
   const db = getTestDB();
   let users: Awaited<ReturnType<typeof seedUsers>> = [];
   if (seedDB) {

turbo.json

@@ -48,7 +48,6 @@
         "node_modules/.cache/tsbuildinfo.json"
       ]
     },
-    "test": {},
     "clean": {
       "cache": false
     },