* Add back stream idle mode
* Shut up codespell
* Correct serialization; only expose at user level
* cargo fmt
* tsc lint fmt
* Move engineStreamMachine as a global actor; tons of more work
* Fix up everything after bumping kittycad/lib
* Remove camera sync
* Use pause/play iconology
* Add back better ping indicator
* wip
* Fix streamIdleMode checkbox being wonky
* yarn fmt
* Massive extinction event for waitForExecutionDone; try to stop projects view switching from crashing
* Clear diagnostics when unmounting code editor!
* wip
* Rework initial root projects dir + deflake many projects tests
* More e2e fixes
* Deflake revolve some revolve tests
* Fix the rest of the mfing tests
* yarn fmt
* yarn lint
* yarn tsc
* Fix tsc after rebase
* wip
* less flaky point and click
* wip
* Fixup after rebase
* Fix more tests
* Fix 2 more
* Fix up named-views tests
* yarn fmt lint tsc
* Fix up new changes
* Get rid of 1 cyclic dependency
* Fix another cyclic mfer!
* fmt
* fmt tsc
* Fix zoom to fit being frigged
* a new list of circular deps
* Remove NetworkHealthIndicator test that was shit
* Fix the bad reload repeat issue kevin started on
* Fix zoom to fit at the right moments...
* Fix cache count numbers in editor test
* Remove a test race - poll window info.
* Qualify fail function
* Try something
* Use scene.connectionEstablished
* Hopefully fix snapshots at least
* Add app console.log
* Fix native menu tests more
* tsc lint
* Fix camera failure
* Try again
* Test attempt number 15345203, action!
* Add back old window detection heuristic
* Remove firstWindow to complete the work of 2342d04fe2
* Tweak some tests for MacOS
* Tweak "set appearance" test for MacOS
Revert this if it messes up any other platform's color checks!
* Are you serious? This was all that needed formatting?
* More color tweaks
Local MacOS and CI MacOS don't agree
* Fixes on apperance e2e test for stream idle branch (#6168)
pierremtb/stream-idle-revamp-appearance-fixes
* Another apperance fix
* Skip one native menu test to make stream idle green (#6169)
* pierremtb/stream-idle-revamp-more-fixes
* Fix lint
* Update snapshot for test_generate_settings_docs
---------
Co-authored-by: lee-at-zoo-corp <lee@zoo.dev>
Co-authored-by: Frank Noirot <frankjohnson1993@gmail.com>
Co-authored-by: Pierre Jacquier <pierrejacquier39@gmail.com>
Co-authored-by: Pierre Jacquier <pierre@zoo.dev>
		
	
		
			
				
	
	
		
			345 lines
		
	
	
		
			11 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
			
		
		
	
	
			345 lines
		
	
	
		
			11 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
import { bracket } from '@src/lib/exampleKcl'
 | 
						|
import fsp from 'fs/promises'
 | 
						|
import { join } from 'path'
 | 
						|
 | 
						|
import { TEST_CODE_LONG_WITH_ERROR_OUT_OF_VIEW } from '@e2e/playwright/storageStates'
 | 
						|
import {
 | 
						|
  executorInputPath,
 | 
						|
  getUtils,
 | 
						|
  orRunWhenFullSuiteEnabled,
 | 
						|
} from '@e2e/playwright/test-utils'
 | 
						|
import { expect, test } from '@e2e/playwright/zoo-test'
 | 
						|
 | 
						|
test.describe('Code pane and errors', { tag: ['@skipWin'] }, () => {
 | 
						|
  test('Typing KCL errors induces a badge on the code pane button', async ({
 | 
						|
    page,
 | 
						|
    homePage,
 | 
						|
    scene,
 | 
						|
    cmdBar,
 | 
						|
  }) => {
 | 
						|
    const u = await getUtils(page)
 | 
						|
 | 
						|
    // Load the app with the working starter code
 | 
						|
    await page.addInitScript(() => {
 | 
						|
      localStorage.setItem(
 | 
						|
        'persistCode',
 | 
						|
        `@settings(defaultLengthUnit = in)
 | 
						|
// Extruded Triangle
 | 
						|
sketch001 = startSketchOn(XZ)
 | 
						|
  |> startProfileAt([0, 0], %)
 | 
						|
  |> line(end = [10, 0])
 | 
						|
  |> line(end = [-5, 10])
 | 
						|
  |> line(endAbsolute = [profileStartX(%), profileStartY(%)])
 | 
						|
  |> close()
 | 
						|
extrude001 = extrude(sketch001, length = 5)`
 | 
						|
      )
 | 
						|
    })
 | 
						|
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 500 })
 | 
						|
    await homePage.goToModelingScene()
 | 
						|
    await scene.settled(cmdBar)
 | 
						|
 | 
						|
    // Ensure no badge is present
 | 
						|
    const codePaneButtonHolder = page.locator('#code-button-holder')
 | 
						|
    await expect(codePaneButtonHolder).not.toContainText('notification')
 | 
						|
 | 
						|
    // Delete a character to break the KCL
 | 
						|
    await u.openKclCodePanel()
 | 
						|
    await page.getByText('extrude(').click()
 | 
						|
    await page.keyboard.press('Backspace')
 | 
						|
 | 
						|
    // Ensure that a badge appears on the button
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
  })
 | 
						|
 | 
						|
  test('Opening and closing the code pane will consistently show error diagnostics', async ({
 | 
						|
    page,
 | 
						|
    homePage,
 | 
						|
    editor,
 | 
						|
  }) => {
 | 
						|
    test.fixme(orRunWhenFullSuiteEnabled())
 | 
						|
    const u = await getUtils(page)
 | 
						|
 | 
						|
    // Load the app with the working starter code
 | 
						|
    await page.addInitScript((code) => {
 | 
						|
      localStorage.setItem('persistCode', code)
 | 
						|
    }, bracket)
 | 
						|
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 900 })
 | 
						|
    await homePage.goToModelingScene()
 | 
						|
 | 
						|
    // wait for execution done
 | 
						|
    await u.openDebugPanel()
 | 
						|
    await u.expectCmdLog('[data-message-type="execution-done"]')
 | 
						|
    await u.closeDebugPanel()
 | 
						|
 | 
						|
    // Ensure we have no errors in the gutter.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).not.toBeVisible()
 | 
						|
 | 
						|
    // Ensure no badge is present
 | 
						|
    const codePaneButton = page.getByRole('button', { name: 'KCL Code pane' })
 | 
						|
    const codePaneButtonHolder = page.locator('#code-button-holder')
 | 
						|
    await expect(codePaneButtonHolder).not.toContainText('notification')
 | 
						|
 | 
						|
    // Delete a character to break the KCL
 | 
						|
    await editor.openPane()
 | 
						|
    await editor.scrollToText('bracketLeg1Sketch, length = thickness)')
 | 
						|
    await page
 | 
						|
      .getByText('extrude(bracketLeg1Sketch, length = thickness)')
 | 
						|
      .click()
 | 
						|
    await page.keyboard.press('Backspace')
 | 
						|
 | 
						|
    // Ensure that a badge appears on the button
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
 | 
						|
    // Ensure we have an error diagnostic.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).toBeVisible()
 | 
						|
 | 
						|
    // error text on hover
 | 
						|
    await page.hover('.cm-lint-marker-error')
 | 
						|
    await expect(page.locator('.cm-tooltip').first()).toBeVisible()
 | 
						|
 | 
						|
    // Close the code pane
 | 
						|
    await codePaneButton.click()
 | 
						|
 | 
						|
    await page.waitForTimeout(500)
 | 
						|
 | 
						|
    // Ensure that a badge appears on the button
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
    // Ensure we have no errors in the gutter.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).not.toBeVisible()
 | 
						|
 | 
						|
    // Open the code pane
 | 
						|
    await editor.openPane()
 | 
						|
 | 
						|
    // Go to our problematic code again (missing closing paren!)
 | 
						|
    await editor.scrollToText('extrude(bracketLeg1Sketch, length = thickness')
 | 
						|
 | 
						|
    // Ensure that a badge appears on the button
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
 | 
						|
    // Ensure we have an error diagnostic.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).toBeVisible()
 | 
						|
 | 
						|
    // error text on hover
 | 
						|
    await page.hover('.cm-lint-marker-error')
 | 
						|
    await expect(page.locator('.cm-tooltip').first()).toBeVisible()
 | 
						|
  })
 | 
						|
 | 
						|
  test('When error is not in view you can click the badge to scroll to it', async ({
 | 
						|
    page,
 | 
						|
    homePage,
 | 
						|
    context,
 | 
						|
  }) => {
 | 
						|
    test.fixme(orRunWhenFullSuiteEnabled())
 | 
						|
    // Load the app with the working starter code
 | 
						|
    await context.addInitScript((code) => {
 | 
						|
      localStorage.setItem('persistCode', code)
 | 
						|
    }, TEST_CODE_LONG_WITH_ERROR_OUT_OF_VIEW)
 | 
						|
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 500 })
 | 
						|
    await homePage.goToModelingScene()
 | 
						|
 | 
						|
    await page.waitForTimeout(1000)
 | 
						|
 | 
						|
    // Ensure badge is present
 | 
						|
    const codePaneButtonHolder = page.locator('#code-button-holder')
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
 | 
						|
    // Ensure we have no errors in the gutter, since error out of view.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).not.toBeVisible()
 | 
						|
 | 
						|
    // Click the badge.
 | 
						|
    const badge = page.locator('#code-badge')
 | 
						|
    await expect(badge).toBeVisible()
 | 
						|
    await badge.click()
 | 
						|
 | 
						|
    // Ensure we have an error diagnostic.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error').first()).toBeVisible()
 | 
						|
 | 
						|
    // Hover over the error to see the error message
 | 
						|
    await page.hover('.cm-lint-marker-error')
 | 
						|
    await expect(
 | 
						|
      page
 | 
						|
        .getByText(
 | 
						|
          'Modeling command failed: [ApiError { error_code: InternalEngine, message: "Solid3D revolve failed:  sketch profile must lie entirely on one side of the revolution axis" }]'
 | 
						|
        )
 | 
						|
        .first()
 | 
						|
    ).toBeVisible()
 | 
						|
  })
 | 
						|
 | 
						|
  test('When error is not in view WITH LINTS you can click the badge to scroll to it', async ({
 | 
						|
    context,
 | 
						|
    page,
 | 
						|
    homePage,
 | 
						|
    scene,
 | 
						|
    cmdBar,
 | 
						|
  }) => {
 | 
						|
    // Load the app with the working starter code
 | 
						|
    await context.addInitScript((code) => {
 | 
						|
      localStorage.setItem('persistCode', code)
 | 
						|
    }, TEST_CODE_LONG_WITH_ERROR_OUT_OF_VIEW)
 | 
						|
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 500 })
 | 
						|
    await homePage.goToModelingScene()
 | 
						|
 | 
						|
    await scene.settled(cmdBar)
 | 
						|
 | 
						|
    // Ensure badge is present
 | 
						|
    const codePaneButtonHolder = page.locator('#code-button-holder')
 | 
						|
    await expect(codePaneButtonHolder).toContainText('notification')
 | 
						|
 | 
						|
    // Ensure we have no errors in the gutter, since error out of view.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error')).not.toBeVisible()
 | 
						|
 | 
						|
    // click in the editor to focus it
 | 
						|
    await page.locator('.cm-content').click()
 | 
						|
 | 
						|
    await page.waitForTimeout(2000)
 | 
						|
 | 
						|
    // go to the start of the editor and enter more text which will trigger
 | 
						|
    // a lint error.
 | 
						|
    // GO to the start of the editor.
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('ArrowUp')
 | 
						|
    await page.keyboard.press('Home')
 | 
						|
    await page.keyboard.type('foo_bar = 1')
 | 
						|
    await page.waitForTimeout(2000)
 | 
						|
    await page.keyboard.press('Enter')
 | 
						|
    await page.waitForTimeout(2000)
 | 
						|
 | 
						|
    // ensure we have a lint error
 | 
						|
    await expect(page.locator('.cm-lint-marker-info').first()).toBeVisible()
 | 
						|
 | 
						|
    // Click the badge.
 | 
						|
    const badge = page.locator('#code-badge')
 | 
						|
    await expect(badge).toBeVisible()
 | 
						|
    await badge.click()
 | 
						|
 | 
						|
    // Ensure we have an error diagnostic.
 | 
						|
    await expect(page.locator('.cm-lint-marker-error').first()).toBeVisible()
 | 
						|
 | 
						|
    // Hover over the error to see the error message
 | 
						|
    await page.hover('.cm-lint-marker-error')
 | 
						|
    await expect(
 | 
						|
      page
 | 
						|
        .getByText(
 | 
						|
          'sketch profile must lie entirely on one side of the revolution axis'
 | 
						|
        )
 | 
						|
        .first()
 | 
						|
    ).toBeVisible()
 | 
						|
  })
 | 
						|
})
 | 
						|
 | 
						|
test(
 | 
						|
  'Opening multiple panes persists when switching projects',
 | 
						|
  { tag: '@electron' },
 | 
						|
  async ({ context, page }, testInfo) => {
 | 
						|
    // Setup multiple projects.
 | 
						|
    await context.folderSetupFn(async (dir) => {
 | 
						|
      const routerTemplateDir = join(dir, 'router-template-slate')
 | 
						|
      const bracketDir = join(dir, 'bracket')
 | 
						|
      await Promise.all([
 | 
						|
        fsp.mkdir(routerTemplateDir, { recursive: true }),
 | 
						|
        fsp.mkdir(bracketDir, { recursive: true }),
 | 
						|
      ])
 | 
						|
      await Promise.all([
 | 
						|
        fsp.copyFile(
 | 
						|
          executorInputPath('cylinder-inches.kcl'),
 | 
						|
          join(routerTemplateDir, 'main.kcl')
 | 
						|
        ),
 | 
						|
        fsp.copyFile(
 | 
						|
          executorInputPath('e2e-can-sketch-on-chamfer.kcl'),
 | 
						|
          join(bracketDir, 'main.kcl')
 | 
						|
        ),
 | 
						|
      ])
 | 
						|
    })
 | 
						|
 | 
						|
    const u = await getUtils(page)
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 500 })
 | 
						|
 | 
						|
    await test.step('Opening the bracket project should load', async () => {
 | 
						|
      await expect(page.getByText('bracket')).toBeVisible()
 | 
						|
 | 
						|
      await page.getByText('bracket').click()
 | 
						|
 | 
						|
      await u.waitForPageLoad()
 | 
						|
    })
 | 
						|
 | 
						|
    // If they're open by default, we're not actually testing anything.
 | 
						|
    await test.step('Pre-condition: panes are not already visible', async () => {
 | 
						|
      await expect(page.locator('#variables-pane')).not.toBeVisible()
 | 
						|
      await expect(page.locator('#logs-pane')).not.toBeVisible()
 | 
						|
    })
 | 
						|
 | 
						|
    await test.step('Open multiple panes', async () => {
 | 
						|
      await u.openKclCodePanel()
 | 
						|
      await u.openVariablesPane()
 | 
						|
      await u.openLogsPane()
 | 
						|
    })
 | 
						|
 | 
						|
    await test.step('Clicking the logo takes us back to the projects page / home', async () => {
 | 
						|
      await page.getByTestId('app-logo').click()
 | 
						|
 | 
						|
      await expect(page.getByRole('link', { name: 'bracket' })).toBeVisible()
 | 
						|
      await expect(page.getByText('router-template-slate')).toBeVisible()
 | 
						|
      await expect(page.getByText('Create project')).toBeVisible()
 | 
						|
    })
 | 
						|
 | 
						|
    await test.step('Opening the router-template project should load', async () => {
 | 
						|
      await expect(page.getByText('router-template-slate')).toBeVisible()
 | 
						|
 | 
						|
      await page.getByText('router-template-slate').click()
 | 
						|
 | 
						|
      await u.waitForPageLoad()
 | 
						|
    })
 | 
						|
 | 
						|
    await test.step('All panes opened before should be visible', async () => {
 | 
						|
      await expect(page.locator('#code-pane')).toBeVisible()
 | 
						|
      await expect(page.locator('#variables-pane')).toBeVisible()
 | 
						|
      await expect(page.locator('#logs-pane')).toBeVisible()
 | 
						|
    })
 | 
						|
  }
 | 
						|
)
 | 
						|
 | 
						|
test(
 | 
						|
  'external change of file contents are reflected in editor',
 | 
						|
  { tag: '@electron' },
 | 
						|
  async ({ context, page }, testInfo) => {
 | 
						|
    const PROJECT_DIR_NAME = 'lee-was-here'
 | 
						|
    const { dir: projectsDir } = await context.folderSetupFn(async (dir) => {
 | 
						|
      const aProjectDir = join(dir, PROJECT_DIR_NAME)
 | 
						|
      await fsp.mkdir(aProjectDir, { recursive: true })
 | 
						|
    })
 | 
						|
 | 
						|
    const u = await getUtils(page)
 | 
						|
    await page.setBodyDimensions({ width: 1200, height: 500 })
 | 
						|
 | 
						|
    await test.step('Open the project', async () => {
 | 
						|
      await expect(page.getByText(PROJECT_DIR_NAME)).toBeVisible()
 | 
						|
      await page.getByText(PROJECT_DIR_NAME).click()
 | 
						|
      await u.waitForPageLoad()
 | 
						|
    })
 | 
						|
 | 
						|
    await u.openFilePanel()
 | 
						|
    await u.openKclCodePanel()
 | 
						|
 | 
						|
    await test.step('Write to file externally and check for changed content', async () => {
 | 
						|
      const content = 'ha he ho ho ha blap scap be dap'
 | 
						|
      await fsp.writeFile(
 | 
						|
        join(projectsDir, PROJECT_DIR_NAME, 'main.kcl'),
 | 
						|
        content
 | 
						|
      )
 | 
						|
      await u.editorTextMatches(content)
 | 
						|
    })
 | 
						|
  }
 | 
						|
)
 |