Remove pre-authenticated Playwright tests (#13606)

* Remove signed in playwright tests

Author: DanielCliftonGuardian

dotcom-rendering/playwright/lib/load-page.ts

@@ -98,30 +98,6 @@ const loadPageWithOverrides = async (
 	await loadPage({ page, path, queryParamsOn: false });
 };
 
-/**
- * Allows us to continue using cookies for signed in features
- * until we figure out how to use Okta in e2e testing.
- * See https://github.com/guardian/dotcom-rendering/issues/8758
- */
-const loadPageNoOkta = async (
-	page: Page,
-	article: FEArticleType,
-	overrides?: {
-		configOverrides?: Record<string, unknown>;
-		switchOverrides?: Record<string, unknown>;
-	},
-): Promise<void> => {
-	await loadPageWithOverrides(page, article, {
-		configOverrides: overrides?.configOverrides,
-		switchOverrides: {
-			...overrides?.switchOverrides,
-			okta: false,
-			idCookieRefresh: false,
-			userFeaturesDcr: true,
-		},
-	});
-};
-
 /**
  * Fetch the page json from PROD then load it as a POST with overrides
  */
@@ -148,6 +124,5 @@ export {
 	BASE_URL,
 	fetchAndloadPageWithOverrides,
 	loadPage,
-	loadPageNoOkta,
 	loadPageWithOverrides,
 };

dotcom-rendering/playwright/tests/braze.e2e.spec.ts

@@ -84,20 +84,6 @@ test.describe('Sign In Gate Tests', () => {
 			await expectToNotExist(page, SIGN_IN_GATE_MAIN_SELECTOR);
 		});
 
-		test('do NOT load the sign in gate if the user is signed in', async ({
-			context,
-			page,
-		}) => {
-			// Fake login
-			await addCookie(context, { name: 'GU_U', value: 'true' });
-
-			await loadPageWithOverrides(page, standardArticle);
-
-			await scrollToGateForLazyLoading(page);
-
-			await expectToNotExist(page, SIGN_IN_GATE_MAIN_SELECTOR);
-		});
-
 		test('do NOT load the sign in gate if the user has already dismissed the gate', async ({
 			page,
 		}) => {

dotcom-rendering/playwright/tests/sign-in-gate.e2e.spec.ts

@@ -1,125 +1,16 @@
 import { expect, test } from '@playwright/test';
 import { Standard as standardArticle } from '../../fixtures/generated/fe-articles/Standard';
 import { disableCMP } from '../lib/cmp';
-import { addCookie } from '../lib/cookies';
 import { waitForIsland } from '../lib/islands';
-import { loadPageNoOkta } from '../lib/load-page';
-import { stubResponse } from '../lib/network';
-
-const profileResponse = {
-	status: 'ok',
-	userProfile: {
-		userId: '102309223',
-		displayName: 'Guardian User',
-		webUrl: 'https://profile.theguardian.com/user/id/102309223',
-		apiUrl: 'https://discussion.guardianapis.com/discussion-api/profile/102309223',
-		avatar: 'https://avatar.guim.co.uk/user/102309223',
-		secureAvatarUrl: 'https://avatar.guim.co.uk/user/102309223',
-		badge: [],
-		privateFields: {
-			canPostComment: true,
-			isPremoderated: false,
-			hasCommented: false,
-		},
-	},
-};
-
-const idapiIdentifiersResponse = {
-	id: '000000000',
-	brazeUuid: 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa',
-	puzzleUuid: 'aaaaaaaaaaaa',
-	googleTagId: 'aaaaaaaaaaaa',
-};
+import { loadPageWithOverrides } from '../lib/load-page';
 
 test.describe('Signed in readers', () => {
-	test('should display the correct page state for signed in users', async ({
-		context,
-		page,
-	}) => {
-		// Fake login
-		await addCookie(context, { name: 'GU_U', value: 'true' });
-
-		// Mock call to '/user/me/identifiers'
-		const idapiReponseProfile = stubResponse(
-			page,
-			'**/user/me/identifiers',
-			{
-				json: idapiIdentifiersResponse,
-			},
-		);
-
-		// Mock call to 'profile/me'
-		const profileResponsePromise = stubResponse(
-			page,
-			'**/profile/me?strict_sanctions_check=false',
-			{
-				json: profileResponse,
-			},
-		);
-
-		await disableCMP(context);
-		await loadPageNoOkta(page, standardArticle);
-
-		await idapiReponseProfile;
-		await profileResponsePromise;
-
-		// This text is shown in the header for signed in users
-		await expect(page.getByText('My account')).toBeVisible();
-	});
-
-	test('should have the correct urls for the header links', async ({
-		context,
-		page,
-	}) => {
-		// Fake login
-		await addCookie(context, { name: 'GU_U', value: 'true' });
-
-		await addCookie(context, {
-			name: 'gu_hide_support_messaging',
-			value: 'true',
-		});
-
-		// Mock call to 'profile/me'
-		const profileResponsePromise = stubResponse(
-			page,
-			'**/profile/me?strict_sanctions_check=false',
-			{
-				json: profileResponse,
-			},
-		);
-
-		await disableCMP(context);
-		await loadPageNoOkta(page, standardArticle);
-
-		await profileResponsePromise;
-
-		expect(
-			await page
-				.locator('a[data-link-name="header : topbar : printsubs"]')
-				.getAttribute('href'),
-		).toContain('support.theguardian.com/subscribe');
-
-		expect(
-			await page
-				.locator('a[data-link-name="header : topbar : job-cta"]')
-				.getAttribute('href'),
-		).toContain('https://jobs.theguardian.com');
-
-		expect(
-			await page
-				.locator(
-					'button[data-link-name="header : topbar : my account"]',
-				)
-				.textContent(),
-		).toContain('My account');
-	});
-
 	test('should not display signed in texts when users are not signed in', async ({
 		context,
 		page,
 	}) => {
 		await disableCMP(context);
-		await loadPageNoOkta(page, standardArticle);
+		await loadPageWithOverrides(page, standardArticle);
 
 		await waitForIsland(page, 'DiscussionWeb');