style: linting fixes

trustpilot · May 20, 2024 · 5da2e9a · 5da2e9a
1 parent c147f6e
commit 5da2e9a
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 15 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -14,7 +14,7 @@ import {
 import { SplitTest } from './splitTest';
 import { uiFactory } from './ui';
 import { CookiePersister } from './userSessionPersister';
-import type { UserSessionPersister } from "./userSessionPersister";
+import type { UserSessionPersister } from './userSessionPersister';
 
 const ui = uiFactory(
     testsObservable,

diff --git a/src/main.ts b/src/main.ts
@@ -36,7 +36,7 @@ export function config(userConfig: Partial<Config> = {}) {
         _config.sessionPersister = userConfig.sessionPersister;
         _config.sessionPersister.saveUserSession(
             session,
-            _config.userSessionDaysToLive
+            _config.userSessionDaysToLive,
         );
     }
     configLoaded = true;
@@ -130,8 +130,10 @@ export function reset(): void {
 
 const waitUntil = (condition: () => any, checkInterval = 100) => {
     return new Promise<void>((resolve) => {
-        let interval = setInterval(() => {
-            if (!condition()) return;
+        const interval = setInterval(() => {
+            if (!condition()) {
+                return;
+            }
             clearInterval(interval);
             resolve();
         }, checkInterval);

diff --git a/src/ui.dev.ts b/src/ui.dev.ts
@@ -27,7 +27,7 @@ export const uiFactory = (
     }
 
     function renderLink(splitTest: SplitTest, variation: InternalVariation) {
-        return renderButton("currently active", () => {
+        return renderButton('currently active', () => {
             const button = document.createElement('button');
             button.value = splitTest.getVariationUrl(variation.name);
             document.body.appendChild(button);
@@ -56,7 +56,7 @@ export const uiFactory = (
     ) {
         const item = document.createElement('li');
         item.textContent = variation.name;
-        const open = renderButton("change to this variant", () => {
+        const open = renderButton('change to this variant', () => {
             setCurrentTestVariation(splitTest.name, variation.name);
         });
 
@@ -88,8 +88,8 @@ export const uiFactory = (
                     .map(
                         (key) => `
                     <div>
-                        <span class="data-label">${key}</span>
-                        <span class="data-value">${data[key]}</span>
+                        <span class='data-label'>${key}</span>
+                        <span class='data-value'>${data[key]}</span>
                     </div>
                 `,
                     )
@@ -108,29 +108,33 @@ export const uiFactory = (
                 if (currentVariation.name === variation.name) {
                     return list.appendChild(
                         renderSelectedVaraition(splitTest, variation),
-                    ).className = "ab-test-variants";
+                    ).className = 'ab-test-variants';
                 } else {
                     return list.appendChild(
                         renderUnselectedVariation(splitTest, variation),
-                    ).className = "ab-test-variants";
+                    ).className = 'ab-test-variants';
                 }
             });
 
             variations.appendChild(legend);
             variations.appendChild(list);
 
             return [test, variations];
-        } 
-        // ** Unsure what this does, when uncommented, renders a list of the same test multiple times, saying "not initialised" - perhaps the tests get mounted multiple times? **
+        }
+        /* Unsure what this does, when uncommented,
+            renders a list of the same test multiple times,
+            saying 'not initialised'
+             - perhaps the tests get mounted multiple times? */
+
         // else {
         //     const canRun = await splitTest.shouldRun(getUserAgentInfo());
         //     const test = document.createElement('div');
         //     test.className = 'test';
         //     test.innerHTML = `
-        //         <div>Test <span class="data-value">${splitTest.name}</span> is not initialized</div>
+        //         <div>Test <span class='data-value'>${splitTest.name}</span> is not initialized</div>
         //         <div>
-        //             <span class="data-label">Can run</span>
-        //             <span class="data-value">${canRun}</span>
+        //             <span class='data-label'>Can run</span>
+        //             <span class='data-value'>${canRun}</span>
         //         </div>
         //     `;