refactor: rename store methods

src/measurement/route/get-measurement.ts

@@ -12,7 +12,7 @@ const handle = async (ctx: ParameterizedContext<DefaultState, DefaultContext & R
 		return;
 	}
 
-	const result = await store.getMeasurement(id);
+	const result = await store.getMeasurementString(id);
 
 	if (!result) {
 		ctx.status = 404;

src/measurement/store.ts

@@ -44,11 +44,11 @@ const substractObjects = (obj1: Record<string, unknown>, obj2: Record<string, un
 export class MeasurementStore {
 	constructor (private readonly redis: RedisClient) {}
 
-	async getMeasurement (id: string): Promise<string> {
+	async getMeasurementString (id: string): Promise<string> {
 		return this.redis.sendCommand([ 'JSON.GET', getMeasurementKey(id) ]);
 	}
 
-	async getMeasurementJson (id: string): Promise<MeasurementRecord> {
+	async getMeasurement (id: string): Promise<MeasurementRecord> {
 		return await this.redis.json.get(getMeasurementKey(id)) as MeasurementRecord;
 	}
 

src/probe/router.ts

@@ -103,7 +103,7 @@ export class ProbeRouter {
 				allProbes.push(connectedProbe);
 			} else {
 				if (!prevMeasurement) {
-					prevMeasurement = await this.store.getMeasurementJson(measurementId);
+					prevMeasurement = await this.store.getMeasurement(measurementId);
 
 					if (!prevMeasurement) {
 						return emptyResult;

test/tests/unit/probe/router.test.ts

@@ -31,7 +31,7 @@ describe('probe router', () => {
 	const getRegionMock = sinon.stub();
 	const store = {
 		getIpsByMeasurementId: sinon.stub(),
-		getMeasurementJson: sinon.stub(),
+		getMeasurement: sinon.stub(),
 	} as unknown as MeasurementStore;
 	const router = new ProbeRouter(fetchSocketsMock, store);
 	let buildProbe: (socket: RemoteProbeSocket) => Promise<Probe>;