Merge pull request #26 from HF6-PROJECT/ara-17

feat: Added starred endpoints
HF6-PROJECT · Oct 11, 2023 · 3a6b0c2 · 3a6b0c2
2 parents 151c4c2 + 2200473
commit 3a6b0c2
Show file tree

Hide file tree

Showing 20 changed files with 1,113 additions and 3 deletions.
diff --git a/prisma/migrations/20231011083919_item_starred/migration.sql b/prisma/migrations/20231011083919_item_starred/migration.sql
@@ -0,0 +1,25 @@
+-- CreateTable
+CREATE TABLE "ItemStarred" (
+    "id" SERIAL NOT NULL,
+    "itemId" INTEGER NOT NULL,
+    "userId" INTEGER NOT NULL,
+    "createdAt" TIMESTAMP(3) NOT NULL DEFAULT CURRENT_TIMESTAMP,
+    "updatedAt" TIMESTAMP(3) NOT NULL,
+
+    CONSTRAINT "ItemStarred_pkey" PRIMARY KEY ("id")
+);
+
+-- CreateIndex
+CREATE INDEX "ItemStarred_itemId_idx" ON "ItemStarred"("itemId");
+
+-- CreateIndex
+CREATE INDEX "ItemStarred_userId_idx" ON "ItemStarred"("userId");
+
+-- CreateIndex
+CREATE UNIQUE INDEX "ItemStarred_itemId_userId_key" ON "ItemStarred"("itemId", "userId");
+
+-- AddForeignKey
+ALTER TABLE "ItemStarred" ADD CONSTRAINT "ItemStarred_itemId_fkey" FOREIGN KEY ("itemId") REFERENCES "Item"("id") ON DELETE CASCADE ON UPDATE CASCADE;
+
+-- AddForeignKey
+ALTER TABLE "ItemStarred" ADD CONSTRAINT "ItemStarred_userId_fkey" FOREIGN KEY ("userId") REFERENCES "User"("id") ON DELETE CASCADE ON UPDATE CASCADE;
diff --git a/prisma/schema.prisma b/prisma/schema.prisma
@@ -20,6 +20,7 @@ model User {
     sessions    UserSession[]
     Item        Item[]
     ItemSharing ItemSharing[]
+    ItemStarred ItemStarred[]
 
     createdAt DateTime @default(now())
     updatedAt DateTime @updatedAt
@@ -57,6 +58,7 @@ model Item {
     ItemSharing           ItemSharing[]
     ItemShortcut          ItemShortcut? @relation("shortcutItem")
     LinkedItemShortcut    ItemShortcut[] @relation("linkedItem")
+    ItemStarred           ItemStarred[]
 
     createdAt DateTime  @default(now())
     updatedAt DateTime  @updatedAt
@@ -118,3 +120,19 @@ model ItemShortcut {
     
     @@index([linkedItemId])
 }
+
+model ItemStarred {
+    id           Int @id @default(autoincrement())
+    itemId       Int
+    userId       Int
+
+    item Item @relation(fields: [itemId], references: [id], onDelete: Cascade)
+    user User @relation(fields: [userId], references: [id], onDelete: Cascade)
+    
+    createdAt DateTime @default(now())
+    updatedAt DateTime @updatedAt
+    
+    @@unique([itemId, userId])
+    @@index([itemId])
+    @@index([userId])
+}
diff --git a/src/locales/da.json b/src/locales/da.json
@@ -111,6 +111,10 @@
 		"shortcut": {
 			"notFound": "Genvej blev ikke fundet"
 		},
+		"starred": {
+			"notFound": "Stjernemarkeringen blev ikke fundet",
+			"alreadyExists": "Stjernemarkeringen findes allerede"
+		},
 		"notFound": "Item ikke fundet"
 	},
 	"folder": {
@@ -163,5 +167,19 @@
 		"parentId": {
 			"type": "Parent id skal være et tal"
 		}
+	},
+	"starred": {
+		"id": {
+			"required": "id er påkrævet",
+			"type": "id skal være et tal"
+		},
+		"userId": {
+			"required": "userId er påkrævet",
+			"type": "userId skal være et tal"
+		},
+		"itemId": {
+			"required": "itemId er påkrævet",
+			"type": "itemId skal være et tal"
+		}
 	}
 }
diff --git a/src/locales/en.json b/src/locales/en.json
@@ -111,6 +111,10 @@
 		"shortcut": {
 			"notFound": "Shortcut not found"
 		},
+		"starred": {
+			"notFound": "Starred not found",
+			"alreadyExists": "Starred already exists"
+		},
 		"notFound": "Item not found"
 	},
 	"folder": {
@@ -163,5 +167,19 @@
 		"parentId": {
 			"type": "Parent id must be a number"
 		}
+	},
+	"starred": {
+		"id": {
+			"required": "id is required",
+			"type": "id must be a number"
+		},
+		"userId": {
+			"required": "userId is required",
+			"type": "userId must be a number"
+		},
+		"itemId": {
+			"required": "itemId is required",
+			"type": "itemId must be a number"
+		}
 	}
 }
diff --git a/src/modules/item/__test__/item.read.test.ts b/src/modules/item/__test__/item.read.test.ts
@@ -96,6 +96,7 @@ describe('GET /api/item/:parentId', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -107,6 +108,7 @@ describe('GET /api/item/:parentId', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -118,6 +120,7 @@ describe('GET /api/item/:parentId', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -128,6 +131,7 @@ describe('GET /api/item/:parentId', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 		]);
 	});

diff --git a/src/modules/item/__test__/item.root.test.ts b/src/modules/item/__test__/item.root.test.ts
@@ -89,6 +89,7 @@ describe('GET /api/item', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -100,6 +101,7 @@ describe('GET /api/item', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -111,6 +113,7 @@ describe('GET /api/item', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -122,6 +125,7 @@ describe('GET /api/item', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 			{
 				id: expect.any(Number),
@@ -132,6 +136,7 @@ describe('GET /api/item', () => {
 				createdAt: expect.any(String),
 				deletedAt: null,
 				updatedAt: expect.any(String),
+				isStarred: false,
 			},
 		]);
 	});

diff --git a/src/modules/item/__test__/item.service.test.ts b/src/modules/item/__test__/item.service.test.ts
@@ -82,6 +82,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -94,6 +95,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -106,6 +108,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -118,6 +121,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 			]);
 		});
@@ -247,6 +251,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -259,6 +264,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -271,6 +277,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -283,6 +290,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -295,6 +303,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -307,6 +316,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -319,6 +329,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 			];
 			const expectedSharredUser = [
@@ -333,6 +344,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -345,6 +357,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -357,6 +370,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 				{
 					id: expect.any(Number),
@@ -369,6 +383,7 @@ describe('ItemService', () => {
 					createdAt: expect.any(Date),
 					deletedAt: null,
 					updatedAt: expect.any(Date),
+					isStarred: false,
 				},
 			];