Stop using findByRole for region

Fixes failing tests

Author: marlo-longley

__tests__/integration/mirador/tests/annotations.test.js

@@ -13,7 +13,7 @@ describe('Annotations in Mirador', () => {
   setupIntegrationTestViewer(config);
 
   it('Loads the manifest', async () => {
-    const element = await screen.findByRole('region', { name: /Window: Self-Portrait Dedicated to Paul Gauguin/i });
+    const element = await screen.findByLabelText(/Window: Self-Portrait Dedicated to Paul Gauguin/i)
     expect(element).toBeInTheDocument();
   });
 

__tests__/integration/mirador/tests/minimalist.test.js

@@ -7,7 +7,7 @@ describe('Minimalist configuration to Mirador', () => {
   setupIntegrationTestViewer(config);
 
   it('Loads a manifest and displays it without some of the default controls', async () => {
-    expect(await screen.findByRole('region', { name: /Window: Self-Portrait Dedicated to Paul Gauguin/i })).toBeInTheDocument();
+    expect(await screen.findByLabelText(/Window: Self-Portrait Dedicated to Paul Gauguin/i)).toBeInTheDocument();
 
     const infoButton = await screen.findByRole('tab', { name: /Information/i });
     expect(infoButton).toBeInTheDocument();

__tests__/integration/mirador/tests/thumbnail-navigation.test.js

@@ -10,7 +10,8 @@ describe('Canvas navigation by clicking thumbnails', () => {
   /* eslint-disable */
   it.skip('navigates a manifest using thumbnail navigation', async (context) => {
     // Make sure we have the manifest
-    const windowElement = await screen.findByRole('region', { name: /Window: Bodleian Library MS. Ind. Inst. Misc. 22/i });
+    const windowElement = await screen.findByLabelText(/Window: Bodleian Library MS. Ind. Inst. Misc. 22/i);
+
     expect(windowElement).toBeInTheDocument();
 
     const windowId = windowElement.getAttribute('id');

__tests__/integration/mirador/tests/viewer-config.test.js

@@ -8,7 +8,7 @@ describe('initialViewerConfig', () => {
 
   describe('initialViewerConfig', () => {
     it('allows initialViewerConfig to be passed', async (context) => {
-      expect(await screen.findByRole('region', { name: /Window: Self-Portrait Dedicated to Paul Gauguin/i })).toBeInTheDocument();
+      expect(await screen.findByLabelText(/Window: Self-Portrait Dedicated to Paul Gauguin/i)).toBeInTheDocument();
 
       let viewerObject;
       await waitFor(() => {

__tests__/integration/mirador/tests/window-actions.test.js

@@ -7,7 +7,7 @@ describe('Window actions', () => {
   setupIntegrationTestViewer(config);
 
   it('Closes a Mirador window', async () => {
-    expect(await screen.findByRole('region', { name: /Window: Self-Portrait Dedicated to Paul Gauguin/i })).toBeInTheDocument();
+    expect(await screen.findByLabelText(/Window: Self-Portrait Dedicated to Paul Gauguin/i)).toBeInTheDocument();
     const closeButton = screen.getByRole('button', { name: /Close window/i });
     fireEvent.click(closeButton);
     await waitFor(() => expect(screen.queryByRole('region', { name: /Window: Self-Portrait Dedicated to Paul Gauguin/i })).not.toBeInTheDocument());