refactor(tests): use new locators to find elements

SimonGolms · Nov 9, 2022 · 5432bcf · 5432bcf
1 parent 5929188
commit 5432bcf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/Message/MessageAccessInformation.test.tsx b/src/components/Message/MessageAccessInformation.test.tsx
@@ -24,7 +24,7 @@ test.describe('MessageAccessInformation', () => {
     );
     const [newPage] = await Promise.all([
       context.waitForEvent('page'),
-      component.locator('a', { hasText: 'Microsoft Docs' }).click(), // Opens a new tab
+      component.getByRole('link', { name: 'Microsoft Docs' }).click(), // Opens a new tab
     ]);
     await newPage.waitForLoadState();
     await expect(await newPage.title()).toBeTruthy();
@@ -38,7 +38,7 @@ test.describe('MessageAccessInformation', () => {
     );
     const [newPage] = await Promise.all([
       context.waitForEvent('page'),
-      component.locator('a', { hasText: 'How to add app roles in Azure AD apps' }).click(), // Opens a new tab
+      component.getByRole('link', { name: 'How to add app roles in Azure AD apps' }).click(), // Opens a new tab
     ]);
     await newPage.waitForLoadState();
     await expect(await newPage.title()).toBeTruthy();

diff --git a/src/components/Message/MessagePermissionsInformation.test.tsx b/src/components/Message/MessagePermissionsInformation.test.tsx
@@ -24,7 +24,7 @@ test.describe('MessagePermissionsInformation', () => {
     );
     const [newPage] = await Promise.all([
       context.waitForEvent('page'),
-      component.locator('a', { hasText: 'Microsoft Docs' }).click(), // Opens a new tab
+      component.getByRole('link', { name: 'Microsoft Docs' }).click(), // Opens a new tab
     ]);
     await newPage.waitForLoadState();
     await expect(await newPage.title()).toBeTruthy();
@@ -38,7 +38,7 @@ test.describe('MessagePermissionsInformation', () => {
     );
     const [newPage] = await Promise.all([
       context.waitForEvent('page'),
-      component.locator('a', { hasText: 'Microsoft Graph' }).click(), // Opens a new tab
+      component.getByRole('link', { name: 'Microsoft Graph' }).click(), // Opens a new tab
     ]);
     await newPage.waitForLoadState();
     await expect(await newPage.title()).toBeTruthy();
@@ -52,7 +52,7 @@ test.describe('MessagePermissionsInformation', () => {
     );
     const [newPage] = await Promise.all([
       context.waitForEvent('page'),
-      component.locator('a', { hasText: 'Permissions and consent in the Microsoft identity platform' }).click(), // Opens a new tab
+      component.getByRole('link', { name: 'Permissions and consent in the Microsoft identity platform' }).click(), // Opens a new tab
     ]);
     await newPage.waitForLoadState();
     await expect(await newPage.title()).toBeTruthy();