Fix Improper Error Catching and Incorrect Uniqueness Check

Author: Nephelite

backend/question-service/models/HistoryEntry.ts

@@ -15,7 +15,7 @@ export interface HistoryEntry extends mongoose.Document {
 const historyEntrySchema: Schema = new Schema<HistoryEntry>({
   userId: { type: String, required: true },
   question: { type: Schema.Types.ObjectId, ref: 'question', required: true },
-  roomId: { type: String, required: true },
+  roomId: { type: String, required: true, unique: false },
   attemptStartedAt: { type: Date, required: true, default: Date.now() },
   attemptCompletedAt: { type: Date, required: true, default: Date.now() },
   collaboratorId: { type: String, required: true },

frontend/src/data/repositories/HistoryRemoteDataSource.ts

@@ -18,7 +18,9 @@ export class HistoryRemoteDataSource extends BaseApi {
     }
 
     async deleteSelectedHistories(selectedHistoryIds: string[]): Promise<void> {
+        console.log("Delete")
         selectedHistoryIds.forEach(async (id) => {
+            console.log(id);
             await this.delete<void>(`/user/${id}`);
         });
     }

frontend/src/domain/usecases/HistoryUseCases.ts

@@ -38,7 +38,7 @@ export class HistoryUseCases {
      * @throws Error if selectedHistoryIds is of length 0 or contains an empty _id.
      */
     async deleteSelectedUserHistories(selectedHistoryIds: string[]): Promise<void> {
-        if (selectedHistoryIds.length === 0 || !selectedHistoryIds.every((_id) => (!_id || _id.trim() === ""))) {
+        if (selectedHistoryIds.length === 0 || selectedHistoryIds.find((_id) => (!_id || _id.trim() === ""))) {
             throw new Error("History ID must be provided");
         }
         await this.historyRepository.deleteSelectedUserHistories(selectedHistoryIds);