Skip to content

Commit

Permalink
Assert data-navbar-state attribute
Browse files Browse the repository at this point in the history
  • Loading branch information
joel-jeremy committed Dec 10, 2024
1 parent 6abe078 commit 41e771d
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions packages/desktop-client/e2e/page-models/mobile-navigation.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
import { MobileAccountPage } from './mobile-account-page';

Check warning on line 1 in packages/desktop-client/e2e/page-models/mobile-navigation.js

View workflow job for this annotation

GitHub Actions / lint

There should be at least one empty line between import groups
import { expect } from '@playwright/test';

Check warning on line 2 in packages/desktop-client/e2e/page-models/mobile-navigation.js

View workflow job for this annotation

GitHub Actions / lint

`@playwright/test` import should occur before import of `./mobile-account-page`

import { MobileAccountsPage } from './mobile-accounts-page';
import { MobileBudgetPage } from './mobile-budget-page';
import { MobileReportsPage } from './mobile-reports-page';
Expand Down Expand Up @@ -69,6 +71,8 @@ export class MobileNavigation {
return false;
}

await expect(this.navbar).toHaveAttribute('data-navbar-state');

const dataNavbarState = await this.navbar.getAttribute('data-navbar-state');
return states.includes(dataNavbarState);
}
Expand Down

0 comments on commit 41e771d

Please sign in to comment.