Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new levels of notification for dev toolbar apps #10252

Merged
merged 11 commits into from
Mar 8, 2024
Prev Previous commit
Next Next commit
Merge branch 'main' into feat/all-notifications-toolbar
  • Loading branch information
Princesseuh committed Feb 27, 2024
commit f5d2bd1abff2fe3db02953872d788468a281bf3b
14 changes: 14 additions & 0 deletions packages/astro/e2e/dev-toolbar.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -307,5 +307,19 @@ test.describe('Dev Toolbar', () => {
await expect(customAppNotification).toHaveAttribute('data-level', 'warning');

await expect(customAppNotification).toBeVisible();
});

test('can quit apps by clicking outside the window', async ({ page, astro }) => {
await page.goto(astro.resolveUrl('/'));

const toolbar = page.locator('astro-dev-toolbar');
for (const appId of ['astro:home', 'astro:audit', 'astro:xray', 'astro:settings']) {
const appButton = toolbar.locator(`button[data-app-id="${appId}"]`);
await appButton.click();

await expect(appButton).toHaveClass('item active');
await page.click('body');
await expect(appButton).not.toHaveClass('active');
}
});
});
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.