From f080c1ab5ea84d446d4c32fbaa6807478623a852 Mon Sep 17 00:00:00 2001
From: Brendan Chen <me@bchen.dev>
Date: Wed, 22 Jan 2025 19:41:31 -0800
Subject: [PATCH] update stub methods to clarify behavior

---
 src/repositories/GetterSetterRepository.ts    | 12 +++++------
 .../UnoptimizedInMemoryRepository.ts          | 21 ++++++++++++-------
 2 files changed, 20 insertions(+), 13 deletions(-)

diff --git a/src/repositories/GetterSetterRepository.ts b/src/repositories/GetterSetterRepository.ts
index 6327a20..1c914c0 100644
--- a/src/repositories/GetterSetterRepository.ts
+++ b/src/repositories/GetterSetterRepository.ts
@@ -19,12 +19,12 @@ export interface GetterSetterRepository extends GetterRepository {
   addOrUpdateOrderedStop(orderedStop: IOrderedStop): Promise<void>;
   addOrUpdateEta(eta: IEta): Promise<void>;
 
-  removeSystem(systemId: string): Promise<void>;
-  removeRoute(routeId: string): Promise<void>;
-  removeShuttle(shuttleId: string): Promise<void>;
-  removeStop(stopId: string): Promise<void>;
-  removeOrderedStop(stopId: string, routeId: string): Promise<void>;
-  removeEta(shuttleId: string, stopId: string): Promise<void>;
+  removeSystemIfExists(systemId: string): Promise<ISystem | null>;
+  removeRouteIfExists(routeId: string): Promise<IRoute | null>;
+  removeShuttleIfExists(shuttleId: string): Promise<IShuttle | null>;
+  removeStopIfExists(stopId: string): Promise<IStop | null>;
+  removeOrderedStopIfExists(stopId: string, routeId: string): Promise<IOrderedStop | null>;
+  removeEtaIfExists(shuttleId: string, stopId: string): Promise<IEta | null>;
 
   clearSystemData(): Promise<void>;
   clearRouteData(): Promise<void>;
diff --git a/src/repositories/UnoptimizedInMemoryRepository.ts b/src/repositories/UnoptimizedInMemoryRepository.ts
index 61191e8..18845ea 100644
--- a/src/repositories/UnoptimizedInMemoryRepository.ts
+++ b/src/repositories/UnoptimizedInMemoryRepository.ts
@@ -148,24 +148,30 @@ export class UnoptimizedInMemoryRepository implements GetterSetterRepository {
     }
   }
 
-
-  public async removeEta(shuttleId: string, stopId: string): Promise<void> {
+  removeSystemIfExists(systemId: string): Promise<ISystem | null> {
+    return Promise.resolve(null);
   }
 
-  public async removeOrderedStop(stopId: string, routeId: string): Promise<void> {
+  removeRouteIfExists(routeId: string): Promise<IRoute | null> {
+    return Promise.resolve(null);
   }
 
-  public async removeRoute(routeId: string): Promise<void> {
+  removeShuttleIfExists(shuttleId: string): Promise<IShuttle | null> {
+    return Promise.resolve(null);
   }
 
-  public async removeShuttle(shuttleId: string): Promise<void> {
+  removeStopIfExists(stopId: string): Promise<IStop | null> {
+    return Promise.resolve(null);
   }
 
-  public async removeStop(stopId: string): Promise<void> {
+  removeOrderedStopIfExists(stopId: string, routeId: string): Promise<IOrderedStop | null> {
+    return Promise.resolve(null);
   }
 
-  public async removeSystem(systemId: string): Promise<void> {
+  removeEtaIfExists(shuttleId: string, stopId: string): Promise<IEta | null> {
+    return Promise.resolve(null);
   }
+
   public async clearSystemData() {
     this.systems = [];
   }
@@ -189,4 +195,5 @@ export class UnoptimizedInMemoryRepository implements GetterSetterRepository {
   public async clearStopData(): Promise<void> {
     this.stops = [];
   }
+
 }
\ No newline at end of file