Test for 3 export paths (#3596)

test for 3 export paths #3582

e2e/playwright/projects.spec.ts

@@ -130,98 +130,104 @@ test(
   }
 )
 
-test(
-  'Can export from electron app',
-  { tag: '@electron' },
-  async ({ browserName }, testInfo) => {
-    test.skip(
-      process.platform === 'win32',
-      'TODO: remove this skip https://github.com/KittyCAD/modeling-app/issues/3557'
-    )
-    const { electronApp, page } = await setupElectron({
-      testInfo,
-      folderSetupFn: async (dir) => {
-        await fsp.mkdir(`${dir}/bracket`, { recursive: true })
-        await fsp.copyFile(
-          'src/wasm-lib/tests/executor/inputs/focusrite_scarlett_mounting_braket.kcl',
-          `${dir}/bracket/main.kcl`
+test.describe('Can export from electron app', () => {
+  const exportMethods = ['sidebarButton', 'commandBar'] as const
+
+  for (const method of exportMethods) {
+    test(
+      `Can export using ${method}`,
+      { tag: '@electron' },
+      async ({ browserName }, testInfo) => {
+        test.skip(
+          process.platform === 'win32',
+          'TODO: remove this skip https://github.com/KittyCAD/modeling-app/issues/3557'
         )
-      },
-    })
-
-    await page.setViewportSize({ width: 1200, height: 500 })
-    const u = await getUtils(page)
-
-    page.on('console', console.log)
-    await electronApp.context().addInitScript(async () => {
-      ;(window as any).playwrightSkipFilePicker = true
-    })
-
-    const pointOnModel = { x: 630, y: 280 }
-
-    await test.step('Opening the bracket project should load the stream', async () => {
-      // expect to see the text bracket
-      await expect(page.getByText('bracket')).toBeVisible()
-
-      await page.getByText('bracket').click()
-
-      await expect(page.getByTestId('loading')).toBeAttached()
-      await expect(page.getByTestId('loading')).not.toBeAttached({
-        timeout: 20_000,
-      })
-
-      await expect(
-        page.getByRole('button', { name: 'Start Sketch' })
-      ).toBeEnabled({
-        timeout: 20_000,
-      })
-
-      // gray at this pixel means the stream has loaded in the most
-      // user way we can verify it (pixel color)
-      await expect
-        .poll(() => u.getGreatestPixDiff(pointOnModel, [75, 75, 75]), {
-          timeout: 10_000,
+        const { electronApp, page } = await setupElectron({
+          testInfo,
+          folderSetupFn: async (dir) => {
+            await fsp.mkdir(`${dir}/bracket`, { recursive: true })
+            await fsp.copyFile(
+              'src/wasm-lib/tests/executor/inputs/focusrite_scarlett_mounting_braket.kcl',
+              `${dir}/bracket/main.kcl`
+            )
+          },
         })
-        .toBeLessThan(10)
-    })
 
-    const exportLocations: Array<Paths> = []
-    await test.step('export the model as a glTF', async () => {
-      exportLocations.push(
-        await doExport(
-          {
-            type: 'gltf',
-            storage: 'embedded',
-            presentation: 'pretty',
-          },
-          page,
-          true
-        )
-      )
-    })
+        await page.setViewportSize({ width: 1200, height: 500 })
+        const u = await getUtils(page)
 
-    await test.step('Check the export size', async () => {
-      await expect
-        .poll(
-          async () => {
-            try {
-              const outputGltf = await fsp.readFile('output.gltf')
-              return outputGltf.byteLength
-            } catch (e) {
-              return 0
-            }
-          },
-          { timeout: 15_000 }
-        )
-        .toBe(477327)
+        page.on('console', console.log)
+        await electronApp.context().addInitScript(async () => {
+          ;(window as any).playwrightSkipFilePicker = true
+        })
 
-      // clean up output.gltf
-      await fsp.rm('output.gltf')
-    })
+        const pointOnModel = { x: 630, y: 280 }
 
-    await electronApp.close()
+        await test.step('Opening the bracket project should load the stream', async () => {
+          // expect to see the text bracket
+          await expect(page.getByText('bracket')).toBeVisible()
+
+          await page.getByText('bracket').click()
+
+          await expect(page.getByTestId('loading')).toBeAttached()
+          await expect(page.getByTestId('loading')).not.toBeAttached({
+            timeout: 20_000,
+          })
+
+          await expect(
+            page.getByRole('button', { name: 'Start Sketch' })
+          ).toBeEnabled({
+            timeout: 20_000,
+          })
+
+          // gray at this pixel means the stream has loaded in the most
+          // user way we can verify it (pixel color)
+          await expect
+            .poll(() => u.getGreatestPixDiff(pointOnModel, [75, 75, 75]), {
+              timeout: 10_000,
+            })
+            .toBeLessThan(10)
+        })
+
+        const exportLocations: Array<Paths> = []
+        await test.step(`export the model as a glTF using ${method}`, async () => {
+          exportLocations.push(
+            await doExport(
+              {
+                type: 'gltf',
+                storage: 'embedded',
+                presentation: 'pretty',
+              },
+              page,
+              method
+            )
+          )
+        })
+
+        await test.step('Check the export size', async () => {
+          await expect
+            .poll(
+              async () => {
+                try {
+                  const outputGltf = await fsp.readFile('output.gltf')
+                  return outputGltf.byteLength
+                } catch (e) {
+                  return 0
+                }
+              },
+              { timeout: 15_000 }
+            )
+            .toBe(477327)
+
+          // clean up output.gltf
+          await fsp.rm('output.gltf')
+        })
+
+        await electronApp.close()
+      }
+    )
   }
-)
+})
 test(
   'Rename and delete projects, also spam arrow keys when renaming',
   { tag: '@electron' },