Remove <Playground> from PUI frontend (#8276)

* Remove <Playground> from PUI frontend - Components are now mature enough with sufficient examples in the codebase * Remove dead code
inventree · Oct 13, 2024 · 7d3eb43 · 7d3eb43
1 parent 8aade76
commit 7d3eb43
Show file tree

Hide file tree

Showing 6 changed files with 1 addition and 349 deletions.
diff --git a/src/frontend/src/defaults/links.tsx b/src/frontend/src/defaults/links.tsx
@@ -37,10 +37,6 @@ export const navTabs = [
   { text: <Trans>Sales</Trans>, name: 'sales', role: UserRoles.sales_order }
 ];
 
-if (IS_DEV_OR_DEMO) {
-  navTabs.push({ text: <Trans>Playground</Trans>, name: 'playground' });
-}
-
 export const docLinks = {
   app: 'https://docs.inventree.org/app/',
   getting_started: 'https://docs.inventree.org/en/latest/getting_started/',

diff --git a/src/frontend/src/defaults/menuItems.tsx b/src/frontend/src/defaults/menuItems.tsx
@@ -64,12 +64,3 @@ export const menuItems: menuItemsCollection = {
     link: '/settings/admin'
   }
 };
-
-if (IS_DEV_OR_DEMO) {
-  menuItems['playground'] = {
-    id: 'playground',
-    text: <Trans>Playground</Trans>,
-    link: '/playground',
-    highlight: true
-  };
-}
diff --git a/src/frontend/src/pages/Index/Playground.tsx b/src/frontend/src/pages/Index/Playground.tsx
diff --git a/src/frontend/src/router.tsx b/src/frontend/src/router.tsx
@@ -8,9 +8,6 @@ export const LayoutComponent = Loadable(
   lazy(() => import('./components/nav/Layout'))
 );
 export const Home = Loadable(lazy(() => import('./pages/Index/Home')));
-export const Playground = Loadable(
-  lazy(() => import('./pages/Index/Playground'))
-);
 
 export const CompanyDetail = Loadable(
   lazy(() => import('./pages/company/CompanyDetail'))
@@ -125,7 +122,6 @@ export const routes = (
       <Route path="home/" element={<Home />} />,
       <Route path="dashboard/" element={<Dashboard />} />,
       <Route path="notifications/*" element={<Notifications />} />,
-      <Route path="playground/" element={<Playground />} />,
       <Route path="scan/" element={<Scan />} />,
       <Route path="settings/">
         <Route index element={<Navigate to="admin/" />} />

diff --git a/src/frontend/tests/pages/pui_index.spec.ts b/src/frontend/tests/pages/pui_index.spec.ts
@@ -1,78 +1,6 @@
 import { test } from '../baseFixtures.js';
 import { doQuickLogin } from '../login.js';
 
-const newPartName = 'UITESTIN123';
-
-test('Pages - Index - Playground', async ({ page }) => {
-  await doQuickLogin(page);
-
-  await page.goto('./');
-  // Playground
-  await page.getByRole('tab', { name: 'Playground' }).click();
-  await page.getByRole('button', { name: 'API Forms' }).click();
-
-  // New Part
-  await page.getByRole('button', { name: 'Create New Part' }).click();
-  await page.getByLabel('related-field-category').fill('category 0');
-
-  await page
-    .getByRole('option', { name: 'Category 0' })
-    .locator('div')
-    .first()
-    .click();
-
-  // Set the "name"
-  await page.getByLabel('text-field-name').fill(newPartName);
-  await page.getByLabel('number-field-initial_stock').fill('1');
-
-  await page
-    .getByLabel('Create Part')
-    .getByRole('button', { name: 'Cancel' })
-    .click();
-
-  // Edit Part
-  await page.getByRole('button', { name: 'Edit Part' }).click();
-  await page.getByLabel('IPN').click();
-  await page.getByLabel('IPN').fill(newPartName);
-  await page
-    .getByLabel('Edit Part')
-    .getByRole('button', { name: 'Cancel' })
-    .click();
-
-  // Create Stock Item
-  await page.getByRole('button', { name: 'Create Stock Item' }).click();
-  await page.getByLabel('related-field-part').fill('R_1K_0402_1');
-  await page.getByText('R_1K_0402_1%').click();
-  await page
-    .getByLabel('Add Stock Item')
-    .getByRole('button', { name: 'Cancel' })
-    .click();
-
-  // EditCategory
-  await page.getByRole('button', { name: 'Edit Category' }).click();
-  await page.getByLabel('related-field-parent').click();
-  await page.getByText('Category 0').click();
-  await page
-    .getByLabel('Edit Category')
-    .getByRole('button', { name: 'Cancel' })
-    .click();
-
-  // Create Part new Modal
-  await page.getByRole('button', { name: 'Create Part new Modal' }).click();
-  await page.getByLabel('related-field-category').click();
-  await page.getByText('Category 0').click();
-  await page
-    .getByLabel('Create part')
-    .getByRole('button', { name: 'Cancel' })
-    .click();
-
-  // Status Label
-  await page.getByRole('button', { name: 'Status labels' }).click();
-  await page.getByRole('textbox').dblclick();
-  await page.getByRole('textbox').fill('50');
-  await page.getByText('Attention needed').waitFor();
-});
-
 test('Pages - Index - Dashboard', async ({ page }) => {
   await doQuickLogin(page);
 

diff --git a/src/frontend/tests/pui_command.spec.ts b/src/frontend/tests/pui_command.spec.ts
@@ -97,24 +97,5 @@ test('Quick Command - No Keys', async ({ page }) => {
     .click();
   await page.waitForURL('https://docs.inventree.org/**');
 
-  // Test addition of new actions
-  await page.goto(`${baseUrl}/playground`);
-  await page.locator('p').filter({ hasText: 'Playground' }).waitFor();
-  await page.getByRole('button', { name: 'Spotlight actions' }).click();
-  await page.getByRole('button', { name: 'Register extra actions' }).click();
-  /*
-  await page.getByPlaceholder('Search...').fill('secret');
-  await page.getByRole('button', { name: 'Secret action It was' }).click();
-  await page.getByLabel('open-spotlight').click();
-  await page.getByPlaceholder('Search...').fill('Another secret action');
-  await page
-    .getByRole('button', {
-      name: 'Another secret action You can register multiple actions with just one command'
-    })
-    .click();
-  await page.getByRole('tab', { name: 'Home' }).click();
-  await page.getByLabel('open-spotlight').click();
-  */
-  await page.getByPlaceholder('Search...').fill('secret');
-  await page.getByText('Nothing found...').click();
+  // TODO: Test addition of new actions
 });