Uuid fix (#94)

* used randomUUID to get uuid; added unit test to check uniqueness

* added separator symbol to element id

package.json

@@ -1,6 +1,6 @@
 {
   "name": "powerbi-visuals-utils-testutils",
-  "version": "2.5.0",
+  "version": "2.6.0",
   "description": "powerbi-visuals-utils-testutils",
   "main": "lib/index.js",
   "module": "lib/index.js",

src/helpers/helpers.ts

@@ -36,18 +36,17 @@ function each(element: JQuery | HTMLElement, fn: (i: number, el: HTMLElement) =>
     }
 }
 
-function uuid() {
-    const buf = new Uint8Array(1);
-    window.crypto.getRandomValues(buf);
-    return buf[0];
+export function getUuid() {
+    const uuid = (window.crypto as any).randomUUID();
+    return uuid;
 }
 
 
 export function testDom(height: number | string, width: number | string): HTMLElement {
     let element: HTMLElement = document.createElement("div"),
         heightWithUnits: string = isFinite( Number(height) ) ? `${Number(height)}px` : String(height),
         widthWithUnits: string = isFinite( Number(width) ) ? `${Number(width)}px` : String(width),
-        id = "item" + uuid();
+        id = "item_" + getUuid();
 
     element.id = id;
     element.style.height = heightWithUnits;

test/helpers/helpersTest.ts

@@ -24,7 +24,7 @@
  *  THE SOFTWARE.
  */
 
-import { testDom, getRandomNumber, getRandomNumbers } from "../../src/helpers/helpers";
+import { testDom, getRandomNumber, getRandomNumbers, getUuid } from "../../src/helpers/helpers";
 
 describe("testDom", () => {
     it("should return an element", () => {
@@ -34,6 +34,21 @@ describe("testDom", () => {
     });
 });
 
+describe("uuid", () => {
+    it("should be unique across multiple tests", () => {
+        const uuids = [];
+        const testsAmount = 1000;
+
+        for (let i = 0; i < testsAmount; i++) {
+            uuids.push(getUuid());
+        }
+
+        const uniqueUUids = [...new Set(uuids)];
+
+        expect(uuids.length).toEqual(uniqueUUids.length);
+    });
+});
+
 describe("getRandomNumber", () => {
     it("should return a number between min and max", () => {
         const min: number = 150,