Fix temporary meter

autometrics-dev · Sep 6, 2023 · 9296f10 · 9296f10
1 parent 745bda3
commit 9296f10
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/packages/autometrics/src/temporaryMeter.ts b/packages/autometrics/src/temporaryMeter.ts
@@ -70,7 +70,7 @@ export class TemporaryMeter implements Meter {
     }
 
     const counter = new TemporaryCounter(name, options);
-    this._createdUpDownCounters.push(counter);
+    this._createdCounters.push(counter);
     return counter;
   }
 
@@ -220,7 +220,7 @@ export class TemporaryMeter implements Meter {
 
     for (const counter of this._createdObservableUpDownCounters) {
       counter.forward(
-        meter.createObservableGauge(counter.name, counter.options),
+        meter.createObservableUpDownCounter(counter.name, counter.options),
       );
     }
 

diff --git a/packages/autometrics/tests/temporaryMeter.test.ts b/packages/autometrics/tests/temporaryMeter.test.ts
@@ -4,7 +4,9 @@ import { TemporaryMeter } from "../src/temporaryMeter";
 
 describe("temporaryMeter test", () => {
   test("it disables itself after the timeout expires", async () => {
-    const meter = new TemporaryMeter({ timeout: 50 });
+    const timeout = 50;
+
+    const meter = new TemporaryMeter({ timeout });
 
     const counter = meter.createCounter("test.counter");
     counter.add(1);
@@ -14,7 +16,7 @@ describe("temporaryMeter test", () => {
     // @ts-ignore
     expect(counter._addedValues.length).toBe(1);
 
-    await new Promise((resolve) => setTimeout(resolve, 50));
+    await new Promise((resolve) => setTimeout(resolve, timeout));
 
     // @ts-ignore
     expect(meter._createdCounters.length).toBe(0);