Fix imports and mocks for renamed functions in tests

Author: pyropy

bin/worker.js

@@ -1,9 +1,9 @@
-import { reportRequestMetric as defaultReportRequestMetric } from '../lib/influx.js'
+import { reportRequestMetrics as defaultReportRequestMetric } from '../lib/influx.js'
 
 export default {
-  async fetch(request, env, ctx, { reportRequestMetric = defaultReportRequestMetric } = {}) {
+  async fetch(request, env, ctx, { reportRequestMetrics = defaultReportRequestMetric } = {}) {
     const response = await fetch(request)
-    ctx.waitUntil(reportRequestMetric(request, env))
+    ctx.waitUntil(reportRequestMetrics(request, env))
     return response
   },
 }

test/influx.test.js

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
-import { createMetricsFromRequest, writeMetrics, reportRequestMetric } from '../lib/influx.js'
+import { createMetricsFromRequest, writeMetrics, reportRequestMetrics } from '../lib/influx.js'
 
-describe('reportRequestMetric', () => {
+describe('reportRequestMetrics', () => {
   const date = new Date()
 
   beforeEach(() => {
@@ -22,7 +22,7 @@ describe('reportRequestMetric', () => {
     }
     global.fetch = vi.fn().mockResolvedValue(new Response(null, { status: 204 }));
 
-    await reportRequestMetric(request, env);
+    await reportRequestMetrics(request, env);
 
     expect(global.fetch).toHaveBeenCalledWith(
       'https://influx.example.com/api/v2/write?&bucket=test_db&precision=ms',

test/worker.test.js

@@ -12,11 +12,11 @@ describe('worker.fetch', () => {
       waitUntil: vi.fn()
     };
     global.fetch = vi.fn().mockResolvedValue(new Response(null, { status: 200 }));
-    const reportRequestMetric = vi.fn()
-    const response = await worker.fetch(request, env, ctx, { reportRequestMetric });
+    const reportRequestMetrics = vi.fn()
+    const response = await worker.fetch(request, env, ctx, { reportRequestMetrics });
     expect(global.fetch).toHaveBeenCalledWith(request);
-    expect(reportRequestMetric).toHaveBeenCalledWith(request, env);
-    expect(ctx.waitUntil).toHaveBeenCalledWith(reportRequestMetric(request, env));
+    expect(reportRequestMetrics).toHaveBeenCalledWith(request, env);
+    expect(ctx.waitUntil).toHaveBeenCalledWith(reportRequestMetrics(request, env));
     expect(response.status).toBe(200);
   });
 });