add project repo and tests

Author: jjnp

pnpm-lock.yaml

@@ -7,9 +7,11 @@
 		"kysely": "^0.27.3",
 		"pino": "^9.0.0",
 		"pino-pretty": "^11.0.0",
+		"uuid": "^9.0.1",
 		"zod": "^3.23.5"
 	},
 	"devDependencies": {
+		"@types/uuid": "^9.0.8",
 		"vite": "^5.2.11",
 		"vitest": "^1.5.3"
 	},

services/package.json

@@ -0,0 +1,90 @@
+import { beforeEach, describe, expect, it } from 'vitest'
+import { db } from '../db/database'
+import { runMigration } from '../db/database-migration-util'
+import { createProject } from '../project/project.repository'
+import type { ProjectId } from '../../../src/routes/(api)/api/api.model'
+import {
+  createApiKey,
+  deleteApiKey,
+  getApiKeysForProject,
+  projectHasKey
+} from './api-key.repository'
+
+beforeEach(async () => {
+  db.reset()
+  await runMigration()
+})
+
+describe('ApiKey Repository', () => {
+  let projectId: ProjectId
+  beforeEach(async () => {
+    projectId = (await createProject({ name: 'demo project name' })).id as ProjectId
+  })
+
+  describe('createApiKey', () => {
+    it('should create an api key given an existing project', async () => {
+      const result = await createApiKey(projectId)
+      expect(result).toBeTruthy()
+      expect(false).toBeFalsy()
+    })
+  })
+
+  describe('getApiKeysForProject', () => {
+    it('should fetch all apiKeys for a given project', async () => {
+      const key1 = await createApiKey(projectId)
+      const key2 = await createApiKey(projectId)
+
+      const result = (await getApiKeysForProject(projectId)).map((it) => it.key)
+      expect(result).toHaveLength(2)
+      expect(result).includes(key1.key)
+      expect(result).includes(key2.key)
+    })
+
+    it('should no longer find deleted keys', async () => {
+      const key = await createApiKey(projectId)
+
+      const keysBeforeDelete = await getApiKeysForProject(projectId)
+      expect(keysBeforeDelete.map((it) => it.key)).toContain(key.key)
+
+      await deleteApiKey(projectId, key.key)
+      const keysAfterDelete = await getApiKeysForProject(projectId)
+      expect(keysAfterDelete.map((it) => it.key).includes(key.key)).toBe(false)
+    })
+
+    it('should return an empty list in case the project does not exist', async () => {
+      const keys = await getApiKeysForProject(projectId)
+      expect(keys).toHaveLength(0)
+    })
+
+    it('should return an empty list if there are no keys', async () => {
+      const result = await getApiKeysForProject(projectId)
+      expect(result).toHaveLength(0)
+    })
+  })
+
+  describe('projectHasKey', () => {
+    it('should return true if there is a key for the project', async () => {
+      const key = await createApiKey(projectId)
+      const result = await projectHasKey(projectId, key.key)
+      expect(result).toBe(true)
+    })
+
+    it('should return false if the project does not have the corresponding key', async () => {
+      const result = await projectHasKey(projectId, 'nonexiststant-id')
+      expect(result).toBe(false)
+    })
+
+    it('should return false if the project does not exist', async () => {
+      const result = await projectHasKey(4242, 'nonexiststant-id')
+      expect(result).toBe(false)
+    })
+
+    it('should return false if key and project exist, but do not match', async () => {
+      const otherProjectId = (await createProject({ name: 'another Project' })).id as ProjectId
+      const key = await createApiKey(projectId)
+
+      const result = await projectHasKey(otherProjectId, key.key)
+      expect(result).toBe(false)
+    })
+  })
+})

services/src/auth/api-key-repository.integration.test.ts

@@ -1,27 +1,38 @@
 import { db } from '../db/database'
 import type { ApiKeyCreationParams, SelectableApiKey } from './api-key'
+import { v4 as uuid } from 'uuid'
 
 export function createApiKey(projectId: number): Promise<SelectableApiKey> {
-  const insertKey: ApiKeyCreationParams = {
-    key: 'something',
-    project_id: projectId
-  }
-  return db
-    .insertInto('apikeys')
-    .values(insertKey)
-    .returningAll()
-    .executeTakeFirstOrThrow(() => new Error('Error creating Api Access Key'))
+	const insertKey: ApiKeyCreationParams = {
+		key: uuid(),
+		project_id: projectId
+	}
+	return db
+		.insertInto('apikeys')
+		.values(insertKey)
+		.returningAll()
+		.executeTakeFirstOrThrow(() => new Error('Error creating Api Access Key'))
 }
 
 export function getApiKeysForProject(projectId: number): Promise<SelectableApiKey[]> {
-  return db.selectFrom('apikeys').selectAll().where('project_id', '==', projectId).execute()
+	return db.selectFrom('apikeys').selectAll().where('project_id', '==', projectId).execute()
 }
 
 export async function projectHasKey(projectId: number, key: string): Promise<boolean> {
-  const result = await db
-    .selectFrom('apikeys')
-    .where('project_id', '==', projectId)
-    .where('key', '==', key)
-    .execute()
-  return !!result.length
+	const result = await db
+		.selectFrom('apikeys')
+		.selectAll()
+		.where('project_id', '==', projectId)
+		.where('key', '==', key)
+		.execute()
+
+	return !!result.length
+}
+
+export async function deleteApiKey(projectId: number, key: string): Promise<void> {
+	await db
+		.deleteFrom('apikeys')
+		.where('project_id', '==', projectId)
+		.where('key', '==', key)
+		.execute()
 }