fix: events index & flaky page locator test (#2279)

* fix(playwright): update main content visibility check to use first match

* test: remove query parameter from test request context

Author: Onibenjo

tests/playwright/helpers/navigation-helpers.ts

@@ -192,5 +192,6 @@ export const verifyPageLoad = async (page: Page, expectedUrl?: string): Promise<
   }
 
   // Check that main content is visible
-  await expect(page.locator('main, .main-content, body')).toBeVisible();
+  // Grabs the highest match and checks if it's visible
+  await expect(page.locator('#main-content, body').first()).toBeVisible();
 };

tests/test_views.py

@@ -218,7 +218,7 @@ def test_events_index_filters_past_events(self, mock_render_template):
 
         events_index = build_events_index(mock_engage_docs)
 
-        with self.app.test_request_context("/?q=test"):
+        with self.app.test_request_context():
             events_index()
             call_kwargs = mock_render_template.call_args[1]
             metadata = call_kwargs["metadata"]