Fix remaining issues with tests, and add implementation within InMemoryParkingRepository.ts

src/entities/ParkingRepositoryEntities.ts

@@ -8,7 +8,7 @@ export interface IParkingStructure extends IEntityWithTimestamp, IEntityWithId {
   name: string;
 }
 
-export interface IParkingStructureTimestampRecord extends IEntityWithTimestamp {
+export interface IParkingStructureTimestampRecord {
   timestampMs: number;
   id: string;
   spotsAvailable: number;

src/repositories/InMemoryParkingRepository.ts

@@ -20,8 +20,33 @@ export class InMemoryParkingRepository implements ParkingGetterSetterRepository
 
   addOrUpdateParkingStructure = async (structure: IParkingStructure): Promise<void> => {
     this.structures.set(structure.id, { ...structure });
+    await this.addHistoricalDataForStructure(structure);
   };
 
+  private addHistoricalDataForStructure = async (structure: IParkingStructure): Promise<void> => {
+    const now = Date.now();
+    const lastAdded = this.dataLastAdded.get(structure.id);
+
+    function parkingLoggingIntervalExceeded() {
+      return !lastAdded || (now - lastAdded.getTime()) >= PARKING_LOGGING_INTERVAL_MS;
+    }
+
+    if (parkingLoggingIntervalExceeded()) {
+      const timestampRecord: IParkingStructureTimestampRecord = {
+        id: structure.id,
+        spotsAvailable: structure.spotsAvailable,
+        timestampMs: now,
+      };
+
+      if (!this.historicalData.has(structure.id)) {
+        this.historicalData.set(structure.id, []);
+      }
+
+      this.historicalData.get(structure.id)?.push(timestampRecord);
+      this.dataLastAdded.set(structure.id, new Date(now));
+    }
+  }
+
   clearParkingStructureData = async (): Promise<void> => {
     this.structures.clear();
   };

test/repositories/InMemoryParkingRepositoryTests.test.ts

@@ -57,8 +57,8 @@ describe("InMemoryParkingRepository", () => {
       jest.setSystemTime(now + PARKING_LOGGING_INTERVAL_MS + 60);
       await repository.addOrUpdateParkingStructure(testStructure);
 
-      expect(historicalData.get(testStructure.id)).toContain(expectedTimestampRecordMatcher);
+      expect(historicalData.get(testStructure.id)).toContainEqual(expectedTimestampRecordMatcher);
-      expect(historicalData.get(testStructure.id)).toContain({
+      expect(historicalData.get(testStructure.id)).toContainEqual({
         ...expectedTimestampRecordMatcher,
         timestampMs: now + PARKING_LOGGING_INTERVAL_MS + 60,
       });