Skip to content

Commit

Permalink
More reliable filePicker Test and fix eslint config error with e2e pr…
Browse files Browse the repository at this point in the history
…oject
  • Loading branch information
Polleps authored and joepio committed Apr 24, 2024
1 parent e21df66 commit 39abcdf
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
3 changes: 2 additions & 1 deletion browser/.eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@ module.exports = {
'lib/tsconfig.json',
'cli/tsconfig.json',
'react/tsconfig.json',
'data-browser/tsconfig.json'
'data-browser/tsconfig.json',
'e2e/tsconfig.json',
],
},
plugins: ['react', '@typescript-eslint', 'prettier', 'react-hooks', 'jsx-a11y'],
Expand Down
7 changes: 3 additions & 4 deletions browser/e2e/tests/filePicker.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ const createModel = async (page: Page) => {
await page.getByRole('button', { name: 'add required property' }).click();
await page
.getByPlaceholder('Search for a property or enter a URL')
.type('programming');
.fill('programming');

await page.keyboard.press('ArrowDown');
await page.keyboard.press('Enter');
Expand Down Expand Up @@ -131,7 +131,7 @@ test.describe('File Picker', () => {
).toBeVisible();

await page.getByRole('button', { name: 'Save' }).click();
await expect(page.getByText('Resource Saved')).toBeVisible();
await expect(page.getByText('New robot')).not.toBeVisible();
}

{
Expand All @@ -157,8 +157,7 @@ test.describe('File Picker', () => {
).toBeVisible();

await page.getByRole('button', { name: 'Save' }).click();

await expect(page.getByText('Resource Saved')).toBeVisible();
await expect(page.getByText('New robot')).not.toBeVisible();
await expect(page.getByText('testFile3.txt').nth(1)).toBeVisible();
await page.getByText('testFile3.txt').nth(1).click();

Expand Down

0 comments on commit 39abcdf

Please sign in to comment.