Skip to content

Commit 40db261

Browse files
prushforprushforth
authored andcommitted
Enable CI, prettier formatting
1 parent 4d1b79d commit 40db261

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed

.github/workflows/ci-testing.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,12 @@ jobs:
1313
uses: actions/setup-node@v3
1414
with:
1515
node-version: '18.x'
16-
# - run: sudo apt-get install xvfb
17-
# - run: npm install --legacy-peer-deps
18-
# - run: npx playwright install
19-
# - run: npm install -g grunt-cli
20-
# - run: grunt default
21-
# - run: xvfb-run --auto-servernum -- npm test
16+
- run: sudo apt-get install xvfb
17+
- run: npm install --legacy-peer-deps
18+
- run: npx playwright install
19+
- run: npm install -g grunt-cli
20+
- run: grunt default
21+
- run: xvfb-run --auto-servernum -- npm test
2222
# - run: xvfb-run --auto-servernum -- npm run jest
2323
env:
2424
CI: true

test/e2e/features/mapFeature.test.js

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ test.describe('Playwright map-feature tests', () => {
3636
// expect the popup for Point 1 to have specific content
3737
await page.getByRole('heading', { name: 'Point 1' });
3838
// expect the popup to have a Zoom to here link
39-
await page
40-
.getByRole('link', { name: 'Zoom to here' })
41-
.click();
39+
await page.getByRole('link', { name: 'Zoom to here' }).click();
4240

4341
// traversing the link, expect the new map location to be centered #24,-75.6978309903406,45.42022684737822 because it is zooming to the map-feature, and it has no map-meta value set, or a map-feature zoom attribute
4442
await expect(page.getByTestId('viewer')).toHaveJSProperty(
@@ -55,10 +53,8 @@ test.describe('Playwright map-feature tests', () => {
5553
// expect the popup for Point 1 to have specific content
5654
await page.getByRole('heading', { name: 'Point 1' });
5755
// expect the popup to have a Zoom to here link
58-
await page.getByRole('link', { name: 'Zoom to here' })
56+
await page.getByRole('link', { name: 'Zoom to here' });
5957
// close the popup
60-
await page
61-
.getByRole('button', { name: 'Close popup' })
62-
.click();
58+
await page.getByRole('button', { name: 'Close popup' }).click();
6359
});
6460
});

0 commit comments

Comments
 (0)