feat(game-history): remove unnecessary updatedAt field (#366)

config/jest/jest-e2e.ts

@@ -9,6 +9,8 @@ const config: Config = {
   resetMocks: true,
   restoreMocks: true,
   clearMocks: true,
+  maxConcurrency: 1,
+  maxWorkers: 1,
   setupFilesAfterEnv: ["jest-extended/all"],
   coverageReporters: ["clover", "json", "lcov", "text", "text-summary"],
   collectCoverageFrom: [

config/jest/jest-global.ts

@@ -9,6 +9,8 @@ const config: Config = {
   resetMocks: true,
   restoreMocks: true,
   clearMocks: true,
+  maxConcurrency: 1,
+  maxWorkers: 1,
   setupFiles: ["<rootDir>/tests/global-setup.ts"],
   setupFilesAfterEnv: ["jest-extended/all"],
   coverageReporters: ["clover", "json-summary", "lcov", "text", "text-summary"],

src/modules/game/constants/game-history-record/game-history-record.constant.ts

@@ -21,7 +21,6 @@ const gameHistoryRecordFieldsSpecs = Object.freeze<Record<keyof GameHistoryRecor
   revealedPlayers: { required: false },
   deadPlayers: { required: false },
   createdAt: { required: true },
-  updatedAt: { required: true },
 });
 
 const gameHistoryRecordApiProperties = Object.freeze<Record<keyof GameHistoryRecord, ApiPropertyOptions>>({
@@ -63,10 +62,9 @@ const gameHistoryRecordApiProperties = Object.freeze<Record<keyof GameHistoryRec
     description: "When the game history record was created",
     ...gameHistoryRecordFieldsSpecs.createdAt,
   },
-  updatedAt: {
-    description: "When the game history record was updated",
-    ...gameHistoryRecordFieldsSpecs.updatedAt,
-  },
 });
 
-export { gameHistoryRecordFieldsSpecs, gameHistoryRecordApiProperties };
+export {
+  gameHistoryRecordFieldsSpecs,
+  gameHistoryRecordApiProperties,
+};

src/modules/game/schemas/game-history-record/game-history-record.schema.ts

@@ -9,7 +9,7 @@ import { PlayerSchema, Player } from "../player/player.schema";
 import { GameHistoryRecordPlay, GameHistoryRecordPlaySchema } from "./game-history-record-play/game-history-record-play.schema";
 
 @Schema({
-  timestamps: true,
+  timestamps: { createdAt: true, updatedAt: false },
   versionKey: false,
 })
 class GameHistoryRecord {
@@ -80,10 +80,6 @@ class GameHistoryRecord {
   @ApiProperty(gameHistoryRecordApiProperties.createdAt)
   @Expose()
   public createdAt: Date;
-
-  @ApiProperty(gameHistoryRecordApiProperties.updatedAt)
-  @Expose()
-  public updatedAt: Date;
 }
 
 const GameHistoryRecordSchema = SchemaFactory.createForClass(GameHistoryRecord);

src/modules/game/types/game-history-record.type.ts

@@ -1,6 +1,6 @@
 import { OmitType } from "@nestjs/swagger";
 import { GameHistoryRecord } from "../schemas/game-history-record/game-history-record.schema";
 
-class GameHistoryRecordToInsert extends OmitType(GameHistoryRecord, ["_id", "createdAt", "updatedAt"] as const) {}
+class GameHistoryRecordToInsert extends OmitType(GameHistoryRecord, ["_id", "createdAt"] as const) {}
 
 export { GameHistoryRecordToInsert };

tests/e2e/specs/modules/game/controllers/game.controller.e2e-spec.ts

@@ -809,17 +809,14 @@ describe("Game Controller", () => {
         {
           ...toJSON(gameHistoryRecords[0]),
           createdAt: expect.any(String) as Date,
-          updatedAt: expect.any(String) as Date,
         },
         {
           ...toJSON(gameHistoryRecords[1]),
           createdAt: expect.any(String) as Date,
-          updatedAt: expect.any(String) as Date,
         },
         {
           ...toJSON(gameHistoryRecords[2]),
           createdAt: expect.any(String) as Date,
-          updatedAt: expect.any(String) as Date,
         },
       ] as GameHistoryRecord[]);
     });

tests/e2e/specs/modules/game/providers/repositories/game-history-record.repository.e2e-spec.ts

@@ -125,7 +125,6 @@ describe("Game History Record Repository", () => {
         ...(toJSON(gameHistoryRecordToInsert) as GameHistoryRecordToInsert),
         _id: expect.any(String) as Types.ObjectId,
         createdAt: expect.any(String) as Date,
-        updatedAt: expect.any(String) as Date,
       });
     });
   });

tests/factories/game/schemas/game-history-record/game-history-record.schema.factory.ts

@@ -334,7 +334,6 @@ function createFakeGameHistoryRecord(gameHistoryRecord: Partial<GameHistoryRecor
     revealedPlayers: gameHistoryRecord.revealedPlayers ?? undefined,
     deadPlayers: gameHistoryRecord.deadPlayers ?? undefined,
     createdAt: gameHistoryRecord.createdAt ?? faker.date.recent(),
-    updatedAt: gameHistoryRecord.updatedAt ?? faker.date.recent(),
     ...override,
   }, plainToInstanceDefaultOptions);
 }