From f47b53c5b9867221cfdc517f9b035b7efde5ff66 Mon Sep 17 00:00:00 2001
From: Simon Larsen <simonlarsen@oneuptime.com>
Date: Thu, 25 Apr 2024 22:03:12 +0100
Subject: [PATCH] Fix formatting issues in code blocks

---
 .../Components/CategoryCheckbox/Category.tsx  |  2 +-
 .../RadioButtons/GroupRadioButtons.tsx        |  1 -
 CommonUI/src/Components/TextArea/TextArea.tsx |  2 +-
 Dashboard/src/Pages/Onboarding/Welcome.tsx    |  2 +-
 E2E/Config.ts                                 | 10 +--
 E2E/Tests/Accounts/Register.spec.ts           | 68 +++++++++++--------
 E2E/Tests/App/StatusCheck.spec.ts             | 34 ++++++----
 E2E/Tests/Home/Landing.spec.ts                |  4 +-
 E2E/Tests/Home/SignUp.spec.ts                 |  4 +-
 E2E/Tests/Ingestor/StatusCheck.spec.ts        | 33 ++++++---
 .../Public/oneuptime/templates/e2e-cron.yml   |  6 ++
 11 files changed, 103 insertions(+), 63 deletions(-)

diff --git a/CommonUI/src/Components/CategoryCheckbox/Category.tsx b/CommonUI/src/Components/CategoryCheckbox/Category.tsx
index 57b241bdb7..4e14cc1421 100644
--- a/CommonUI/src/Components/CategoryCheckbox/Category.tsx
+++ b/CommonUI/src/Components/CategoryCheckbox/Category.tsx
@@ -14,7 +14,7 @@ export interface CategoryProps {
     onChange: (value: Array<CategoryCheckboxValue>) => void;
     initialValue?: undefined | Array<CategoryCheckboxValue>;
     isLastCategory: boolean;
-    dataTestId?: string | undefined;    
+    dataTestId?: string | undefined;
 }
 
 enum CategoryCheckboxValueState {
diff --git a/CommonUI/src/Components/RadioButtons/GroupRadioButtons.tsx b/CommonUI/src/Components/RadioButtons/GroupRadioButtons.tsx
index 8eca1d05de..bb70a50fe0 100644
--- a/CommonUI/src/Components/RadioButtons/GroupRadioButtons.tsx
+++ b/CommonUI/src/Components/RadioButtons/GroupRadioButtons.tsx
@@ -12,7 +12,6 @@ export interface RadioButton {
     value: string;
     sideTitle?: string | undefined;
     sideDescription?: string | undefined;
-
 }
 
 export interface ComponentProps {
diff --git a/CommonUI/src/Components/TextArea/TextArea.tsx b/CommonUI/src/Components/TextArea/TextArea.tsx
index 3b85b62e49..d1465f32f1 100644
--- a/CommonUI/src/Components/TextArea/TextArea.tsx
+++ b/CommonUI/src/Components/TextArea/TextArea.tsx
@@ -59,7 +59,7 @@ const TextArea: FunctionComponent<ComponentProps> = (
                 <textarea
                     autoFocus={props.autoFocus}
                     placeholder={props.placeholder}
-                    data-testid={props.dataTestId}  
+                    data-testid={props.dataTestId}
                     className={`${className || ''}`}
                     value={text}
                     onChange={(e: React.ChangeEvent<HTMLTextAreaElement>) => {
diff --git a/Dashboard/src/Pages/Onboarding/Welcome.tsx b/Dashboard/src/Pages/Onboarding/Welcome.tsx
index 369d799a57..f0b19a8665 100644
--- a/Dashboard/src/Pages/Onboarding/Welcome.tsx
+++ b/Dashboard/src/Pages/Onboarding/Welcome.tsx
@@ -35,7 +35,7 @@ const Welcome: FunctionComponent<ComponentProps> = (
                         onClick={() => {
                             props.onClickShowProjectModal();
                         }}
-                        dataTestId='create-new-project-button'
+                        dataTestId="create-new-project-button"
                     />
                 }
             />
diff --git a/E2E/Config.ts b/E2E/Config.ts
index 31c9f11226..ce5abf26c5 100644
--- a/E2E/Config.ts
+++ b/E2E/Config.ts
@@ -14,7 +14,9 @@ export const HTTP_PROTOCOL: Protocol =
 
 export const BASE_URL: URL = URL.fromString(`${HTTP_PROTOCOL}${HOST}`);
 
-export const IS_USER_REGISTERED: boolean = env('E2E_TEST_IS_USER_REGISTERED') === 'true';
-export const REGISTERED_USER_EMAIL: string = env('E2E_TEST_REGISTERED_USER_EMAIL') || '';
-export const REGISTERED_USER_PASSWORD: string = env('E2E_TEST_REGISTERED_USER_PASSWORD') || '';
-
+export const IS_USER_REGISTERED: boolean =
+    env('E2E_TEST_IS_USER_REGISTERED') === 'true';
+export const REGISTERED_USER_EMAIL: string =
+    env('E2E_TEST_REGISTERED_USER_EMAIL') || '';
+export const REGISTERED_USER_PASSWORD: string =
+    env('E2E_TEST_REGISTERED_USER_PASSWORD') || '';
diff --git a/E2E/Tests/Accounts/Register.spec.ts b/E2E/Tests/Accounts/Register.spec.ts
index 9d176074d9..0972918c65 100644
--- a/E2E/Tests/Accounts/Register.spec.ts
+++ b/E2E/Tests/Accounts/Register.spec.ts
@@ -1,37 +1,45 @@
-import { test, expect } from '@playwright/test';
+import { test, expect, Page } from '@playwright/test';
 import { BASE_URL, IS_USER_REGISTERED } from '../../Config';
 import Faker from 'Common/Utils/Faker';
 import URL from 'Common/Types/API/URL';
 test.describe('Account Registration', () => {
-  test('should register a new account', async ({ page }) => {
+    test('should register a new account', async ({ page }: { page: Page }) => {
+        if (IS_USER_REGISTERED) {
+            // pass this test if user is already registered
+            return;
+        }
 
-    if(IS_USER_REGISTERED) {
-        // pass this test if user is already registered
-        return;
-    }
+        await page.goto(
+            URL.fromString(BASE_URL.toString())
+                .addRoute('/accounts/register')
+                .toString()
+        );
+        await page.getByTestId('email').click();
+        await page.getByTestId('email').fill(Faker.generateEmail().toString());
+        await page.getByTestId('email').press('Tab');
+        await page.getByTestId('name').fill('sample');
+        await page.getByTestId('name').press('Tab');
+        await page.getByTestId('companyName').fill('sample');
+        await page.getByTestId('companyName').press('Tab');
+        await page.getByTestId('companyPhoneNumber').fill('+15853641376');
+        await page.getByTestId('companyPhoneNumber').press('Tab');
+        await page.getByTestId('password').fill('sample');
+        await page.getByTestId('password').press('Tab');
+        await page.getByTestId('confirmPassword').fill('sample');
+        await page.getByTestId('Sign Up').click();
 
+        // wait for navigation with base url
+        await page.waitForURL(
+            URL.fromString(BASE_URL.toString())
+                .addRoute('/dashboard/welcome')
+                .toString()
+        );
+        expect(page.url()).toBe(
+            URL.fromString(BASE_URL.toString())
+                .addRoute('/dashboard/welcome')
+                .toString()
+        );
 
-    await page.goto(URL.fromString(BASE_URL.toString()).addRoute('/accounts/register').toString());
-    await page.getByTestId('email').click();
-    await page.getByTestId('email').fill(Faker.generateEmail().toString());
-    await page.getByTestId('email').press('Tab');
-    await page.getByTestId('name').fill('sample');
-    await page.getByTestId('name').press('Tab');
-    await page.getByTestId('companyName').fill('sample');
-    await page.getByTestId('companyName').press('Tab');
-    await page.getByTestId('companyPhoneNumber').fill('+15853641376');
-    await page.getByTestId('companyPhoneNumber').press('Tab');
-    await page.getByTestId('password').fill('sample');
-    await page.getByTestId('password').press('Tab');
-    await page.getByTestId('confirmPassword').fill('sample');
-    await page.getByTestId('Sign Up').click();
-
-    // wait for navigation with base url
-    await page.waitForURL(URL.fromString(BASE_URL.toString()).addRoute('/dashboard/welcome').toString());
-    expect(page.url()).toBe(URL.fromString(BASE_URL.toString()).addRoute('/dashboard/welcome').toString());
-
-    await page.getByTestId('create-new-project-button').click();
-
-  });
-
-})
\ No newline at end of file
+        await page.getByTestId('create-new-project-button').click();
+    });
+});
diff --git a/E2E/Tests/App/StatusCheck.spec.ts b/E2E/Tests/App/StatusCheck.spec.ts
index 40cd2f2528..8a7dd31c62 100644
--- a/E2E/Tests/App/StatusCheck.spec.ts
+++ b/E2E/Tests/App/StatusCheck.spec.ts
@@ -1,25 +1,35 @@
-import { test, expect } from '@playwright/test';
+import { test, expect, Page } from '@playwright/test';
 import { BASE_URL } from '../../Config';
 import URL from 'Common/Types/API/URL';
 
-
 test.describe('check live and health check of the app', () => {
-    
-    test('check if app status is ok', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/status").toString()}`);
-        const content = await page.content();
+    test('check if app status is ok', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/status')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 
-    test('check if app is ready', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/status/ready").toString()}`);
-        const content = await page.content();
+    test('check if app is ready', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/status/ready')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 
-    test('check if app is live', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/status/live").toString()}`);
-        const content = await page.content();
+    test('check if app is live', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/status/live')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 });
diff --git a/E2E/Tests/Home/Landing.spec.ts b/E2E/Tests/Home/Landing.spec.ts
index 47081c00a9..47c63d8fb9 100644
--- a/E2E/Tests/Home/Landing.spec.ts
+++ b/E2E/Tests/Home/Landing.spec.ts
@@ -20,6 +20,8 @@ test.describe('check if pages loades with its title', () => {
             .getByRole('link', { name: 'OneUptime', exact: true })
             .click();
 
-        await expect(page).toHaveURL(URL.fromString(BASE_URL.toString()).toString());
+        await expect(page).toHaveURL(
+            URL.fromString(BASE_URL.toString()).toString()
+        );
     });
 });
diff --git a/E2E/Tests/Home/SignUp.spec.ts b/E2E/Tests/Home/SignUp.spec.ts
index 84d9ca5d90..58d4f08232 100644
--- a/E2E/Tests/Home/SignUp.spec.ts
+++ b/E2E/Tests/Home/SignUp.spec.ts
@@ -8,6 +8,8 @@ test.beforeEach(async ({ page }: { page: Page }) => {
 test('sign up button', async ({ page }: { page: Page }) => {
     await page.getByTestId('Sign-up').click();
     await expect(page).toHaveURL(
-        URL.fromString(BASE_URL.toString()).addRoute('/accounts/register').toString()
+        URL.fromString(BASE_URL.toString())
+            .addRoute('/accounts/register')
+            .toString()
     );
 });
diff --git a/E2E/Tests/Ingestor/StatusCheck.spec.ts b/E2E/Tests/Ingestor/StatusCheck.spec.ts
index 1d953d6736..c062d34fa0 100644
--- a/E2E/Tests/Ingestor/StatusCheck.spec.ts
+++ b/E2E/Tests/Ingestor/StatusCheck.spec.ts
@@ -1,24 +1,35 @@
-import { test, expect } from '@playwright/test';
+import { test, expect, Page } from '@playwright/test';
 import { BASE_URL } from '../../Config';
 import URL from 'Common/Types/API/URL';
 
 test.describe('check live and health check of the app', () => {
-    
-    test('check if app status is ok', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/ingestor/status").toString()}`);
-        const content = await page.content();
+    test('check if app status is ok', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/ingestor/status')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 
-    test('check if app is ready', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/ingestor/status/ready").toString()}`);
-        const content = await page.content();
+    test('check if app is ready', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/ingestor/status/ready')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 
-    test('check if app is live', async ({ page }) => {
-        await page.goto(`${URL.fromString(BASE_URL.toString()).addRoute("/ingestor/status/live").toString()}`);
-        const content = await page.content();
+    test('check if app is live', async ({ page }: { page: Page }) => {
+        await page.goto(
+            `${URL.fromString(BASE_URL.toString())
+                .addRoute('/ingestor/status/live')
+                .toString()}`
+        );
+        const content: string = await page.content();
         expect(content).toContain('{"status":"ok"}');
     });
 });
diff --git a/HelmChart/Public/oneuptime/templates/e2e-cron.yml b/HelmChart/Public/oneuptime/templates/e2e-cron.yml
index 9112a73c17..36108cbe5e 100644
--- a/HelmChart/Public/oneuptime/templates/e2e-cron.yml
+++ b/HelmChart/Public/oneuptime/templates/e2e-cron.yml
@@ -19,6 +19,12 @@ spec:
             image: {{ printf "%s/%s/%s:%s" $.Values.image.registry $.Values.image.repository "e2e" $.Values.image.tag }}
             env:
               {{- include "oneuptime.env.common" . | nindent 14 }}
+              - name: E2E_TEST_IS_USER_REGISTERED
+                value: {{ $.Values.cronJobs.e2e.isUserRegistered }}
+              - name: E2E_TEST_REGISERED_USER_EMAIL
+                value: {{ $.Values.cronJobs.e2e.registeredUserEmail }}
+              - name: E2E_TEST_REGISERED_USER_PASSWORD
+                value: {{ $.Values.cronJobs.e2e.registeredUserPassword }}
           restartPolicy: Never
 
 {{- end }}
\ No newline at end of file