Skip to content

Commit

Permalink
OBPIH-6909 Unify manu items roles (#48)
Browse files Browse the repository at this point in the history
  • Loading branch information
awalkowiak authored Feb 17, 2025
1 parent 80c58ee commit a763e6b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
6 changes: 1 addition & 5 deletions src/components/Navbar.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,7 @@ class Navbar extends BasePageModel {
}

getNavItem(name: string) {
return this.navbar.getByRole('link', { name: name, exact: true });
}

get logoutButton() {
return this.navbar.getByRole('menuitem', { name: 'Logout' });
return this.navbar.getByRole('menuitem', { name: name, exact: true });
}

get editProfileButton() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -856,8 +856,7 @@ test.describe('Check if ward location is present in location chooser based on us
}
);

// TODO: OBPIH-6909 Fix menu clicking when impersonating users
test.skip('Assert created Ward on location chooser in impersonate mode', async ({
test('Assert created Ward on location chooser in impersonate mode', async ({
userListPage,
editUserPage,
}) => {
Expand Down
3 changes: 1 addition & 2 deletions src/tests/users/userPermission.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,7 @@ test('Add default location for user and auto-login to location', async ({
await expect(newUserPage.getByText('My Dashboard')).toBeVisible();
});

// TODO: OBPIH-6909 Fix menu clicking when impersonating users
test.skip('Impersonate created user', async ({ editUserPage }) => {
test('Impersonate created user', async ({ editUserPage }) => {
await test.step('Select role "Manager"', async () => {
await editUserPage.authorizationTabSection.defaultRoleSelect.click();
await editUserPage.authorizationTabSection.getUserRole('Manager').click();
Expand Down

0 comments on commit a763e6b

Please sign in to comment.