Refactor Fillet Playwright Test to the new Fixture-based Approach (#4909)
* add locator * new test * remove old test * separation of setup steps * improved click reliability * fix ubuntu * ubuntu fix 2 * ubuntu fix 3 * flaky cmdbar ubuntu * ubuntu fix second yellow * ubuntu update * enable test for windows * step(Initial test setup) * extra await, just in case * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Trigger CI * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * A snapshot a day keeps the bugs away! 📷🐛 (OS: namespace-profile-ubuntu-8-cores) * Trigger CI * screenshot --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
This commit is contained in:
		@ -45,46 +45,6 @@ test.describe('Command bar tests', () => {
 | 
			
		||||
    )
 | 
			
		||||
  })
 | 
			
		||||
 | 
			
		||||
  // TODO: fix this test after the electron migration
 | 
			
		||||
  test.fixme('Fillet from command bar', async ({ page, homePage }) => {
 | 
			
		||||
    await page.addInitScript(async () => {
 | 
			
		||||
      localStorage.setItem(
 | 
			
		||||
        'persistCode',
 | 
			
		||||
        `sketch001 = startSketchOn('XY')
 | 
			
		||||
    |> startProfileAt([-5, -5], %)
 | 
			
		||||
    |> line([0, 10], %)
 | 
			
		||||
    |> line([10, 0], %)
 | 
			
		||||
    |> line([0, -10], %)
 | 
			
		||||
    |> lineTo([profileStartX(%), profileStartY(%)], %)
 | 
			
		||||
    |> close(%)
 | 
			
		||||
  extrude001 = extrude(-10, sketch001)`
 | 
			
		||||
      )
 | 
			
		||||
    })
 | 
			
		||||
 | 
			
		||||
    const u = await getUtils(page)
 | 
			
		||||
    await page.setBodyDimensions({ width: 1000, height: 500 })
 | 
			
		||||
    await homePage.goToModelingScene()
 | 
			
		||||
    await u.openDebugPanel()
 | 
			
		||||
    await u.expectCmdLog('[data-message-type="execution-done"]')
 | 
			
		||||
    await u.closeDebugPanel()
 | 
			
		||||
 | 
			
		||||
    const selectSegment = () => page.getByText(`line([0, -10], %)`).click()
 | 
			
		||||
 | 
			
		||||
    await selectSegment()
 | 
			
		||||
    await page.waitForTimeout(100)
 | 
			
		||||
    await page.getByRole('button', { name: 'Fillet' }).click()
 | 
			
		||||
    await page.waitForTimeout(100)
 | 
			
		||||
    await page.keyboard.press('Enter') // skip selection
 | 
			
		||||
    await page.waitForTimeout(100)
 | 
			
		||||
    await page.keyboard.press('Enter') // accept default radius
 | 
			
		||||
    await page.waitForTimeout(100)
 | 
			
		||||
    await page.keyboard.press('Enter') // submit
 | 
			
		||||
    await page.waitForTimeout(100)
 | 
			
		||||
    await expect(page.locator('.cm-activeLine')).toContainText(
 | 
			
		||||
      `fillet({ radius = ${KCL_DEFAULT_LENGTH}, tags = [seg01] }, %)`
 | 
			
		||||
    )
 | 
			
		||||
  })
 | 
			
		||||
 | 
			
		||||
  test('Command bar can change a setting, and switch back and forth between arguments', async ({
 | 
			
		||||
    page,
 | 
			
		||||
    homePage,
 | 
			
		||||
 | 
			
		||||
		Reference in New Issue
	
	Block a user