rename methods for clarity on method events

This commit is contained in:
2025-02-02 13:14:45 -08:00
parent 9dd6e945c2
commit b30e4be895
3 changed files with 7 additions and 7 deletions

View File

@@ -25,7 +25,7 @@ export interface GetterRepository {
* if ETA data is restored. * if ETA data is restored.
* @param listener * @param listener
*/ */
subscribeToEtaChanges( subscribeToEtaUpdates(
listener: (eta: IEta) => void, listener: (eta: IEta) => void,
): void; ): void;
@@ -34,7 +34,7 @@ export interface GetterRepository {
* Callback must be passed by reference. * Callback must be passed by reference.
* @param listener * @param listener
*/ */
unsubscribeFromEtaChanges(listener: (eta: IEta) => void): void; unsubscribeFromEtaUpdates(listener: (eta: IEta) => void): void;
getOrderedStopByRouteAndStopId(routeId: string, stopId: string): Promise<IOrderedStop | null>; getOrderedStopByRouteAndStopId(routeId: string, stopId: string): Promise<IOrderedStop | null>;

View File

@@ -61,11 +61,11 @@ export class UnoptimizedInMemoryRepository implements GetterSetterRepository {
return this.etas.filter(eta => eta.stopId === stopId); return this.etas.filter(eta => eta.stopId === stopId);
} }
public subscribeToEtaChanges(listener: (eta: IEta) => void) { public subscribeToEtaUpdates(listener: (eta: IEta) => void) {
this.subscribers.push(listener); this.subscribers.push(listener);
} }
public unsubscribeFromEtaChanges(listener: (eta: IEta) => void) { public unsubscribeFromEtaUpdates(listener: (eta: IEta) => void) {
const index = this.subscribers.findIndex((existingListener) => existingListener == listener); const index = this.subscribers.findIndex((existingListener) => existingListener == listener);
if (index >= 0) { if (index >= 0) {
this.subscribers.splice(index, 1); this.subscribers.splice(index, 1);

View File

@@ -228,7 +228,7 @@ describe("UnoptimizedInMemoryRepository", () => {
describe("subscribeToEtaChanges", () => { describe("subscribeToEtaChanges", () => {
test("notifies listeners if etas have been added or changed", async () => { test("notifies listeners if etas have been added or changed", async () => {
const mockCallback = jest.fn(); // Jest mock function to simulate a listener const mockCallback = jest.fn(); // Jest mock function to simulate a listener
repository.subscribeToEtaChanges(mockCallback); repository.subscribeToEtaUpdates(mockCallback);
const mockEtas = generateMockEtas(); const mockEtas = generateMockEtas();
for (const eta of mockEtas) { for (const eta of mockEtas) {
@@ -244,12 +244,12 @@ describe("UnoptimizedInMemoryRepository", () => {
describe("unsubscribeFromEtaChanges", () => { describe("unsubscribeFromEtaChanges", () => {
test("stops notifying listeners after etas have stopped changing", async () => { test("stops notifying listeners after etas have stopped changing", async () => {
const mockCallback = jest.fn(); // Jest mock function to simulate a listener const mockCallback = jest.fn(); // Jest mock function to simulate a listener
repository.subscribeToEtaChanges(mockCallback); repository.subscribeToEtaUpdates(mockCallback);
const mockEtas = generateMockEtas(); const mockEtas = generateMockEtas();
await repository.addOrUpdateEta(mockEtas[0]); await repository.addOrUpdateEta(mockEtas[0]);
repository.unsubscribeFromEtaChanges(mockCallback); repository.unsubscribeFromEtaUpdates(mockCallback);
await repository.addOrUpdateEta(mockEtas[mockEtas.length - 1]); await repository.addOrUpdateEta(mockEtas[mockEtas.length - 1]);