parent 7709e00

author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283986 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283967 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283901 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283878 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283866 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283821 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283802 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283787 +0100 parent 7709e00 author lindot11 <[email protected]> 1701026896 +0100 committer Linda Nguyen <[email protected]> 1701283759 +0100 skip package upgrade added new githook script and updated guide comment out lib test hook log githook run updated run log removed logs and test from repo updated configure-git-hooks script and docs removed unnecessary readme added prettier config to client undo change non trailling commas for client and lib undo trailing commas to test code cov Revert "undo trailing commas to test code cov" This reverts commit e5bf319. test undo semiconlons set traillingcomma all undo prettierrc accept prettier lib improves codecov Corrects docker commands and updates docs (#304) - Corrects docker run commands to always restart the containers - Updates diagrams in docs to include MongoDB - Adds yarn clean command to package.json of services. uncomment prettier replaced by pre-commit framework adhere to code climate linelength markdownlint disable codeblock test test deprecated code_blocks in codeclimate shortened comment to adhere to line_length skip package upgrade Update docs and adds publication for runner (#275) - Adds package publication for runner - Updates READMEs of lib and runner microservices - Cleans up yarn commands of runner to make them cross-platform - Adds documentation for new example Fixes mistakes in the docs test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks test git hooks reset after test test git hooks test git hooks test git hooks reset after test test git hooks reset after test test git hooks test git hooks reset after test test git hooks reset after test reset after test reset after test reset after test reset after test test script for git hooks done githook test and log for documentation reset after test reset after test reset after test reset after test reset after test reset after test reset after test reset after test test git hooks reset after test test git hooks reset after test test git hooks reset after test improved test went back to original test test git hooks final test script reset after test test git hooks reset after test reset after test reset after test reset after test reset after test test and log done added new githook script and updated guide comment out lib test hook log githook run updated run log removed logs and test from repo updated configure-git-hooks script and docs updated hook to use markdownlint/markdownlint undo change non trailling commas for client and lib undo trailing commas to test code cov Revert "undo trailing commas to test code cov" This reverts commit e5bf319. undo trailling commas to test codecov undo trailling commas in lib undo single quote prettier test undo semiconlons test remove trailling commas lib set traillingcomma all undo prettierrc accept prettier lib improves codecov accept prettier client improves codecov Swap ml-workspace container and improve docs (#300) - Replaces ml-workspace container with ml-workspace-minimal container. This change potentially improves the speed of user workspace. - Adds guide book to help with non-standard installation scenarios. - Adds MongoDB installation to services.js --------- Co-authored-by: nichlaes <[email protected]> Corrects docker commands and updates docs (#304) - Corrects docker run commands to always restart the containers - Updates diagrams in docs to include MongoDB - Adds yarn clean command to package.json of services. uncomment prettier replace deprecated code_blocks uncomment markdownlint fixed table mdl error ignore url line length mdl redo mdl hook ignore table line length and use .markdownlint.yaml instead markdownlint disable codeblock test fix wrong reset action Revert "accept prettier client improves codecov" This reverts commit 924a5cd. Revert "skip package upgrade" This reverts commit ae8bea4.
INTO-CPS-Association · Nov 29, 2023 · 006bff7 · 006bff7
1 parent 7709e00
commit 006bff7
Show file tree

Hide file tree

Showing 18 changed files with 209 additions and 35 deletions.
diff --git a/client/.prettierrc b/client/.prettierrc
@@ -1,4 +1,3 @@
 {
-  "singleQuote": true,
-  "trailingComma": "all"
+  "singleQuote": true
 }
diff --git a/client/src/index.tsx b/client/src/index.tsx
@@ -57,7 +57,7 @@ const router = createBrowserRouter(
   ],
   {
     basename: `/${useURLbasename()}`,
-  },
+  }
 );
 
 const root = document.getElementById('root');
@@ -68,7 +68,7 @@ if (root) {
       <AppProvider>
         <RouterProvider router={router} />
       </AppProvider>
-    </React.StrictMode>,
+    </React.StrictMode>
   );
 } else {
   throw Error("Couldn't find root element");

diff --git a/client/test/e2e/playwright/Auth.test.ts b/client/test/e2e/playwright/Auth.test.ts
@@ -13,7 +13,7 @@ test.describe('Tests on Authentication Flow', () => {
       .getByRole('button', { name: 'GitLab logo Sign In with GitLab' })
       .click();
     await expect(
-      page.getByRole('button', { name: 'Open settings' }),
+      page.getByRole('button', { name: 'Open settings' })
     ).toBeVisible();
     await expect(page).toHaveURL(/.*Library/);
   });
@@ -23,7 +23,7 @@ test.describe('Tests on Authentication Flow', () => {
       .getByRole('button', { name: 'GitLab logo Sign In with GitLab' })
       .click();
     await expect(
-      page.getByRole('button', { name: 'Open settings' }),
+      page.getByRole('button', { name: 'Open settings' })
     ).toBeVisible();
     await expect(page).toHaveURL(/.*Library/);
 

diff --git a/client/test/e2e/playwright/Menu.test.ts b/client/test/e2e/playwright/Menu.test.ts
@@ -10,7 +10,7 @@ test.describe('Menu Links from first page (Layout)', () => {
       .getByRole('button', { name: 'GitLab logo Sign In with GitLab' })
       .click();
     await expect(
-      page.getByRole('button', { name: 'Open settings' }),
+      page.getByRole('button', { name: 'Open settings' })
     ).toBeVisible();
     await expect(page).toHaveURL(/.*Library/);
   });
@@ -19,7 +19,7 @@ test.describe('Menu Links from first page (Layout)', () => {
     await links.reduce(async (previousPromise, link) => {
       await previousPromise;
       const linkElement = await page.locator(
-        `div[role="button"]:has-text("${link.text}")`,
+        `div[role="button"]:has-text("${link.text}")`
       );
       await expect(linkElement).toBeVisible();
     }, Promise.resolve());

diff --git a/client/test/e2e/playwright/auth.setup.ts b/client/test/e2e/playwright/auth.setup.ts
@@ -42,7 +42,7 @@ setup('authenticate', async ({ page }) => {
     // 'Authorize' button did not appear within 4 seconds, so just ignore and continue.
   }
   await expect(
-    page.getByRole('button', { name: 'Open settings' }),
+    page.getByRole('button', { name: 'Open settings' })
   ).toBeVisible();
 
   const storage = await page.context().storageState();

diff --git a/client/test/integration/authRedux.test.tsx b/client/test/integration/authRedux.test.tsx
@@ -49,7 +49,7 @@ const setupTest = (authState: AuthState) => {
     <PrivateRoute>
       <Library />
     </PrivateRoute>,
-    { route: '/private', store },
+    { route: '/private', store }
   );
 };
 
@@ -75,7 +75,7 @@ describe('Redux and Authentication integration test', () => {
 
     expect(screen.getByText('Signin')).toBeInTheDocument();
     expect(authReducer(undefined, { type: 'unknown' })).toEqual(
-      initialState.auth,
+      initialState.auth
     );
     expect(store.getState().userName).toBe(undefined);
   });

diff --git a/client/test/unitTests/Components/Linkbuttons.test.tsx b/client/test/unitTests/Components/Linkbuttons.test.tsx
@@ -24,7 +24,7 @@ const getButtonIcon = (key: string) =>
 const evaluateButtonSize = (expectedSize: number) => {
   buttons.forEach((button) => {
     expect(
-      getComputedStyle(getButtonIcon(button.key)).getPropertyValue('font-size'),
+      getComputedStyle(getButtonIcon(button.key)).getPropertyValue('font-size')
     ).toBe(`${expectedSize}rem`);
   });
 };
@@ -38,7 +38,7 @@ describe('LinkButtons component default size', () => {
     buttons.forEach((button) => {
       expect(getButton(button.key).parentElement).toHaveAttribute(
         'aria-label',
-        button.link,
+        button.link
       );
       expect(getLabel(button.key).tagName).toBe('H6');
     });
@@ -66,7 +66,7 @@ describe('LinkButtons component default size', () => {
 
   it('should use name from iconLib as label when avaiable', () => {
     expect(getLabel(buttons[0].key).textContent).toBe(
-      LinkIcons[buttons[0].key].name,
+      LinkIcons[buttons[0].key].name
     );
   });
 });

diff --git a/client/test/unitTests/Components/PrivateRoute.test.tsx b/client/test/unitTests/Components/PrivateRoute.test.tsx
@@ -30,7 +30,7 @@ const setupTest = (authState: AuthState) => {
     <PrivateRoute>
       <TestComponent />
     </PrivateRoute>,
-    { route: '/private' },
+    { route: '/private' }
   );
 };
 

diff --git a/client/test/unitTests/Page/LayoutPublic.test.tsx b/client/test/unitTests/Page/LayoutPublic.test.tsx
@@ -11,7 +11,7 @@ const PublicTestComponentId = 'public-component';
 
 describe('LayoutPublic component with one element', () => {
   beforeEach(() =>
-    renderLayoutWithRouter(LayoutPublic, [PublicTestComponentId]),
+    renderLayoutWithRouter(LayoutPublic, [PublicTestComponentId])
   );
 
   basicLayoutTestsWithSingleComponent();

diff --git a/client/test/unitTests/Page/page.testUtils.tsx b/client/test/unitTests/Page/page.testUtils.tsx
@@ -7,7 +7,7 @@ export const TestComponentIdList = ['component1', 'component2', 'component3'];
 
 export function renderLayoutWithRouter(
   Layout: (props: { children: React.ReactElement[] }) => React.ReactElement,
-  Children: string[],
+  Children: string[]
 ) {
   render(<Layout>{generateTestDivs(Children)}</Layout>, {
     wrapper: BrowserRouter,

diff --git a/client/test/unitTests/Routes/Library.test.tsx b/client/test/unitTests/Routes/Library.test.tsx
@@ -19,7 +19,7 @@ describe('Library with no props', () => {
   InitRouteTests(
     <AuthProvider>
       <Library />
-    </AuthProvider>,
+    </AuthProvider>
   );
 
   itDisplaysContentOfTabs(assetType);

diff --git a/client/test/unitTests/Routes/SignIn.test.tsx b/client/test/unitTests/Routes/SignIn.test.tsx
@@ -23,19 +23,19 @@ describe('SignIn', () => {
     render(
       <MemoryRouter>
         <SignIn />
-      </MemoryRouter>,
+      </MemoryRouter>
     );
 
     expect(
-      screen.getByRole('button', { name: /Sign In With GitLab/i }),
+      screen.getByRole('button', { name: /Sign In With GitLab/i })
     ).toBeInTheDocument();
   });
 
   it('handles button click', () => {
     render(
       <MemoryRouter>
         <SignIn />
-      </MemoryRouter>,
+      </MemoryRouter>
     );
 
     const signInButton = screen.getByRole('button', {

diff --git a/client/test/unitTests/Util/Store.test.ts b/client/test/unitTests/Util/Store.test.ts
@@ -25,15 +25,15 @@ describe('reducers', () => {
   describe('menu reducer', () => {
     const itShouldHandleMenuActions = (
       actionCreator: () => { type: string },
-      expectedValue: boolean,
+      expectedValue: boolean
     ) => {
       const newState = menuReducer(initialState.menu, actionCreator());
       expect(newState.isOpen).toBe(expectedValue);
     };
 
     it('menuReducer should return the initial menu state when an unknown action type is passed with an undefined state', () => {
       expect(menuReducer(undefined, { type: 'unknown' })).toEqual(
-        initialState.menu,
+        initialState.menu
       );
     });
 
@@ -50,7 +50,7 @@ describe('reducers', () => {
   describe('auth reducer', () => {
     it('authReducer should return the initial menu state when an unknown action type is passed with an undefined state', () => {
       expect(authReducer(undefined, { type: 'unknown' })).toEqual(
-        initialState.auth,
+        initialState.auth
       );
     });
 

diff --git a/client/test/unitTests/Util/envUtil.test.ts b/client/test/unitTests/Util/envUtil.test.ts
@@ -53,10 +53,10 @@ describe('envUtil', () => {
 
   test('GetURL should return the correct enviroment variables', () => {
     expect(useURLforDT()).toBe(
-      `${testAppURL}/${testBasename}/${testUsername}/${testDT}`,
+      `${testAppURL}/${testBasename}/${testUsername}/${testDT}`
     );
     expect(useURLforLIB()).toBe(
-      `${testAppURL}/${testBasename}/${testUsername}/${testLIB}`,
+      `${testAppURL}/${testBasename}/${testUsername}/${testLIB}`
     );
     expect(useURLbasename()).toBe(testBasename);
   });
@@ -71,8 +71,8 @@ describe('envUtil', () => {
     const result = getWorkbenchLinkValues();
     expect(
       result.every(
-        (el) => typeof el.key === 'string' && typeof el.link === 'string',
-      ),
+        (el) => typeof el.key === 'string' && typeof el.link === 'string'
+      )
     ).toBe(true);
   });
 
@@ -83,8 +83,8 @@ describe('envUtil', () => {
     result.forEach((el, i) => {
       expect(el.link).toEqual(
         `${testAppURL}/${testBasename}/${testUsername}/${cleanURL(
-          testWorkbenchEndpoints[i],
-        )}`,
+          testWorkbenchEndpoints[i]
+        )}`
       );
     });
   });

diff --git a/client/test/unitTests/auth/AuthProvider.test.tsx b/client/test/unitTests/auth/AuthProvider.test.tsx
@@ -38,7 +38,7 @@ describe('AuthProvider', () => {
     const { getByText } = renderAuthProvider(<DummyComponent />);
 
     expect(
-      getByText('Authentication service unavailable...try again later'),
+      getByText('Authentication service unavailable...try again later')
     ).toBeInTheDocument();
   });
 

diff --git a/client/test/unitTests/testUtils.tsx b/client/test/unitTests/testUtils.tsx
@@ -18,15 +18,15 @@ type RouterOptions = {
 
 export const renderWithRouter = (
   ui: React.ReactElement,
-  { route = '/', store }: RouterOptions = {},
+  { route = '/', store }: RouterOptions = {}
 ) => {
   window.history.pushState({}, 'Test page', route);
 
   return store
     ? render(
         <Provider store={store}>
           <RouterComponent ui={ui} route={route} />
-        </Provider>,
+        </Provider>
       )
     : render(<RouterComponent ui={ui} route={route} />);
 };
@@ -65,7 +65,7 @@ export function InitRouteTests(component: React.ReactElement) {
 }
 
 export function itDisplaysContentOfTabs(
-  tabs: { label: string; body: string }[],
+  tabs: { label: string; body: string }[]
 ) {
   it('should render labels of all tabs', () => {
     tabs.forEach((tab) => {
@@ -117,7 +117,7 @@ export interface TabLabelURLPair {
 }
 
 export function itHasCorrectURLOfTabsWithIframe(
-  tablabelsURLpair: TabLabelURLPair[],
+  tablabelsURLpair: TabLabelURLPair[]
 ) {
   it('should render the Iframe component for the first tab with the correct title and URL', () => {
     tablabelsURLpair.forEach((tablabelURLpair) => {

diff --git a/docs/user/examples/examples.drawio b/docs/user/examples/examples.drawio
diff --git a/servers/execution/runner/README.md b/servers/execution/runner/README.md
@@ -81,12 +81,14 @@ Access to the service on network is available at `http://<ip or hostname>:3000/`
 Two REST API routes are active. The route paths and the responses given
 for these two sources are:
 
+
 | REST API Route                 | Return Value | Comment |
 | :----------------------------- | :----------- | :------ |
 | localhost:3000/phase | [ hello ] | Each invocation appends to _array_. |
 | localhost:3000/lifecycle/phase | _true_       | Always returns _true_ |
 | localhost:3000/phase | [ hello ] | array. |
 
+
 ## :balance_scale: License
 
 This software is owned by