test: improved meter test coverage & removed redundant code

src/meter/MeterState.ts

@@ -3,8 +3,8 @@ import {
   useSealedState,
 } from "reakit-utils/useSealedState";
 
-import { isFunction, valueToPercent } from "../utils";
-import { getDefaultOptimumValue, calculateStatus, clamp } from "./helpers";
+import { isFunction, valueToPercent, clampValue } from "../utils";
+import { getDefaultOptimumValue, calculateStatus } from "./helpers";
 
 type Status = "safe" | "caution" | "danger" | undefined;
 
@@ -83,10 +83,10 @@ export const useMeterState = (
   const initialOptimum =
     sealed.optimum ?? getDefaultOptimumValue(initialLow, initialHigh);
 
-  const value = clamp(initialValue, min, max);
-  const optimum = clamp(initialOptimum, min, max);
-  let low = clamp(initialLow, min, max);
-  let high = clamp(initialHigh, min, max);
+  const value = clampValue(initialValue, min, max);
+  const optimum = clampValue(initialOptimum, min, max);
+  let low = clampValue(initialLow, min, max);
+  let high = clampValue(initialHigh, min, max);
 
   // More inequalities handled
   //  low ≤ high (if both low and high are specified)

src/meter/__tests__/Meter.test.tsx

@@ -124,4 +124,18 @@ describe("Meter", () => {
     },
     data,
   );
+
+  test("useMeterState: low >= high", function () {
+    const { current } = renderMeterStateHook({ low: 1, high: 0.2 });
+
+    expect(current.low).toBe(0.2);
+    expect(current.high).toBe(0.2);
+  });
+
+  test("useMeterState: high >= low", function () {
+    const { current } = renderMeterStateHook({ high: 1, low: 0.2 });
+
+    expect(current.low).toBe(0.2);
+    expect(current.high).toBe(1);
+  });
 });

src/meter/helpers.ts

@@ -8,22 +8,6 @@ export function getDefaultOptimumValue(min: number, max: number) {
   return max < min ? min : min + (max - min) / 2;
 }
 
-/**
- * Handle Inequalities with received values
- *
- * minimum ≤ value ≤ maximum
- * minimum ≤ low ≤ maximum (if low is specified)
- * minimum ≤ high ≤ maximum (if high is specified)
- * minimum ≤ optimum ≤ maximum (if optimum is specified)
- *
- * @see https://html.spec.whatwg.org/multipage/form-elements.html#the-meter-element:attr-meter-max-3:~:text=following%20inequalities%20must%20hold
- */
-export function clamp(value: number, min: number, max: number) {
-  if (value == null) return 0;
-
-  return Math.min(Math.max(value, min), max);
-}
-
 type CalculateStatusProps = {
   value: number;
   optimum: number;