Fix randomRange

src/index.ts

@@ -11,6 +11,7 @@ export { clamp } from "./math/clamp"
 export { modulo } from "./math/modulo"
 export { normalizeValue } from "./math/normalizeValue"
 export { originalValueFromNormalized } from "./math/normalizeValue"
+export { randomRange } from "./math/randomRange"
 // promise
 export { deferredPromise } from "./promise/deferredPromise"
 export type { TDeferredPromise } from "./promise/deferredPromise"

src/math/randomRange.ts

@@ -2,9 +2,9 @@
  * Get random value between min and max (include)
  * @param min
  * @param max
- * @param decimal
+ * @param decimalNumber 1 = 0.1, 2 = 0.01, 3 = 0.001
  */
-export function randomRange(min: number, max: number, decimal = 0): number {
-  const power = Math.pow(10, decimal)
-  return Math.floor(Math.random() * (max - min + 1) + min * power) / power
+export function randomRange(min: number, max: number, decimalNumber = 0): number {
+  if (decimalNumber < 0) return
+  return parseFloat((Math.random() * (max - min) + min).toFixed(decimalNumber))
 }

tests/math/randomRange.test.ts

@@ -1,6 +1,5 @@
 import { describe, expect, it } from "vitest"
-import { randomRange } from "../../src/math/randomRange"
-
+import { randomRange } from "../../src"
 
 describe("randomRange", () => {
   it("should return random value between min and max", () => {
@@ -15,4 +14,23 @@ describe("randomRange", () => {
       }
     }
   })
+
+  it("should generate a random integer within the specified range", () => {
+    const result = randomRange(1, 10)
+    expect(result).toBeTypeOf("number")
+    expect(result).toBeGreaterThanOrEqual(1)
+    expect(result).toBeLessThanOrEqual(10)
+  })
+
+  it("should generate a random float with the specified number of decimal places", () => {
+    const result = randomRange(1.5, 5.5, 2)
+    expect(result).toBeTypeOf("number")
+    expect(result).toBeGreaterThanOrEqual(1.5)
+    expect(result).toBeLessThanOrEqual(5.5)
+    expect(result.toFixed(2)).toBe(result.toString())
+  })
+
+  it("should return undefined if there is no valid decimal set as param", () => {
+    expect(randomRange(0, 1, -1)).toBeUndefined()
+  })
 })