Rename to getAuthHeaders

Author: tomrf1

dotcom-rendering/src/components/SlotBodyEnd/ReaderRevenueEpic.tsx

@@ -20,7 +20,7 @@ import type {
 import { useEffect, useState } from 'react';
 import { submitComponentEvent } from '../../client/ophan/ophan';
 import {
-	buildRequestHeaders,
+	getAuthHeaders,
 	hasCmpConsentForWeeklyArticleCount,
 	hasOptedOutOfArticleCount,
 	shouldHideSupportMessaging,
@@ -107,7 +107,7 @@ export const canShowReaderRevenueEpic = async (
 		hideSupportMessagingForUser,
 	});
 
-	const headers = await buildRequestHeaders();
+	const headers = await getAuthHeaders();
 
 	const response: ModuleDataResponse<EpicProps> = await getEpic(
 		contributionsServiceUrl,

dotcom-rendering/src/lib/contributions.test.ts

@@ -4,7 +4,7 @@ import { createOrRenewCookie } from '../client/userFeatures/cookies/cookieHelper
 import { HIDE_SUPPORT_MESSAGING_COOKIE } from '../client/userFeatures/cookies/hideSupportMessaging';
 import { USER_BENEFITS_EXPIRY_COOKIE } from '../client/userFeatures/cookies/userBenefitsExpiry';
 import {
-	buildRequestHeaders,
+	getAuthHeaders,
 	getLastOneOffContributionTimestamp,
 	hasHideSupportMessagingCookie,
 	isRecentOneOffContributor,
@@ -216,23 +216,23 @@ const getAuthStatus = getAuthStatus_ as jest.MockedFunction<
 	typeof getAuthStatus_
 >;
 
-describe('buildRequestHeaders', () => {
+describe('getAuthHeaders', () => {
 	beforeEach(() => {
 		jest.clearAllMocks();
 	});
 
 	it('returns undefined when user has not consented to targeting', async () => {
 		(onConsent as jest.Mock).mockResolvedValue({ canTarget: false });
 
-		const result = await buildRequestHeaders();
+		const result = await getAuthHeaders();
 
 		expect(result).toBeUndefined();
 	});
 
 	it('returns undefined when consent check fails', async () => {
 		(onConsent as jest.Mock).mockRejectedValue(new Error('Consent error'));
 
-		const result = await buildRequestHeaders();
+		const result = await getAuthHeaders();
 
 		expect(result).toBeUndefined();
 	});
@@ -241,7 +241,7 @@ describe('buildRequestHeaders', () => {
 		(onConsent as jest.Mock).mockResolvedValue({ canTarget: true });
 		(getAuthStatus as jest.Mock).mockResolvedValue({ kind: 'SignedOut' });
 
-		const result = await buildRequestHeaders();
+		const result = await getAuthHeaders();
 
 		expect(result).toBeUndefined();
 	});
@@ -253,7 +253,7 @@ describe('buildRequestHeaders', () => {
 			accessToken: { accessToken: 'token' },
 		});
 
-		const result = await buildRequestHeaders();
+		const result = await getAuthHeaders();
 
 		expect(result).toEqual({
 			Authorization: 'Bearer token',

dotcom-rendering/src/lib/contributions.ts

@@ -232,10 +232,8 @@ const hasCanTargetConsent = (): Promise<boolean> =>
 		.then(({ canTarget }: ConsentState) => canTarget)
 		.catch(() => false);
 
-export const buildRequestHeaders = async (): Promise<
-	HeadersInit | undefined
-> => {
-	// If user has targeting consent and is signed in then we can send an Authorization header
+// Returns Auth headers only if user has targeting consent and is signed in
+export const getAuthHeaders = async (): Promise<HeadersInit | undefined> => {
 	const hasConsented = await hasCanTargetConsent();
 	if (hasConsented) {
 		const authStatus = await getAuthStatus();