Refactor tests to use array assertions instead of snapshots

Updated multiple test files to replace inline snapshot assertions with direct array comparisons for toolbar node content and console log outputs. Removed obsolete Jest snapshot file. These changes improve test clarity and maintainability.

Author: josephfusco

packages/faustwp-cli/tests/healthCheck/__snapshots__/validateFaustEnvVars.test.ts.snap

@@ -58,18 +58,22 @@ describe('healthCheck/validateFaustEnvVars', () => {
 		process.env.NEXT_PUBLIC_WORDPRESS_URL = 'https://headless.local';
 		process.env.FAUST_SECRET_KEY = 'invalid-secret-key';
 
+		const consoleLogSpy = jest.spyOn(console, 'log').mockImplementation();
+
 		fetchMock.post(
-			'https://headless.local/wp-json/faustwp/v1/validate_secret_key',
+			'https://headless.local/?rest_route=/faustwp/v1/validate_secret_key',
 			{
 				status: 401,
 			},
 		);
 
 		await validateFaustEnvVars();
 
-		return expect(Promise.resolve(validateFaustEnvVars())).toMatchSnapshot(
-			`Ensure your FAUST_SECRET_KEY environment variable matches your Secret Key in the Faust WordPress plugin settings`,
+		expect(consoleLogSpy).toHaveBeenCalledWith(
+			expect.stringContaining('Ensure your FAUST_SECRET_KEY environment variable matches your Secret Key'),
 		);
+
+		consoleLogSpy.mockRestore();
 	});
 });
 

packages/faustwp-cli/tests/healthCheck/validateFaustEnvVars.test.ts

@@ -48,9 +48,8 @@ describe('healthCheck/validateNextWordPressUrl', () => {
 
 		await validateNextWordPressUrl();
 		expect(consoleLogSpy).toHaveBeenCalledWith(
-			expect.stringContaining(
-				'Validation Failed: Your Faust front-end site URL value is misconfigured. It should NOT match the `NEXT_PUBLIC_WORDPRESS_URL.',
-			),
+			expect.stringContaining('Validation Failed, Faust is shutting down:'),
+			expect.stringContaining('Your Faust front-end site URL value is misconfigured'),
 		);
 		expect(mockExit).toHaveBeenCalledWith(1);
 		expect(fetchMock).toHaveFetched(

packages/faustwp-cli/tests/healthCheck/validateNextWordPressUrl.test.ts

@@ -129,13 +129,7 @@ test('renders a default list of nodes in the primary section if seedNode is not
 	testToolBarNode(
 		toolBars[0],
 		3,
-		`
-  Array [
-    "WordPress",
-    "",
-    "GraphiQL IDE",
-  ]
-`,
+		['WordPress', '', 'GraphiQL IDE'],
 	);
 });
 
@@ -159,13 +153,7 @@ test('renders an Edit Post Node,  in the primary section if seedNode is provided
 	testToolBarNode(
 		toolBars[0],
 		3,
-		`
-  Array [
-    "WordPress",
-    "Edit Post",
-    "GraphiQL IDE",
-  ]
-`,
+		['WordPress', 'Edit Post', 'GraphiQL IDE'],
 	);
 });
 
@@ -187,14 +175,7 @@ test('renders an Account Node in the secondary section', async () => {
 	testToolBarNode(
 		toolBars[1],
 		4,
-		`
-    Array [
-      "Howdy, Edit ProfileLog Out",
-      "",
-      "Edit Profile",
-      "Log Out",
-    ]
-  `,
+		['Howdy, Edit ProfileLog Out', '', 'Edit Profile', 'Log Out'],
 	);
 });
 
@@ -218,13 +199,7 @@ test('renders an Edit Post Node, if seedNode is not provided and is preview', as
 	testToolBarNode(
 		toolBars[0],
 		3,
-		`
-  Array [
-    "WordPress",
-    "Edit Post",
-    "GraphiQL IDE",
-  ]
-  `,
+		['WordPress', 'Edit Post', 'GraphiQL IDE'],
 	);
 });
 
@@ -244,13 +219,7 @@ test('does not render an Edit Post Node, if there is no seedNode and it is not a
 	testToolBarNode(
 		toolBars[0],
 		3,
-		`
-  Array [
-    "WordPress",
-    "",
-    "GraphiQL IDE",
-  ]
-`,
+		['WordPress', '', 'GraphiQL IDE'],
 	);
 });
 
@@ -276,18 +245,11 @@ test('Uses `toolbarNodes` hook to add nodes', async () => {
 	);
 	await waitFor(() => queryByAttribute('id', dom.container, 'wpadminbar'));
 	const toolBars = screen.getAllByRole('list', { name: /toolbar/i });
-	// Secondary Toolbar
+	// Primary Toolbar
 	testToolBarNode(
 		toolBars[0],
 		4,
-		`
-    Array [
-      "WordPress",
-      "Edit Post",
-      "GraphiQL IDE",
-      "Test Node",
-    ]
-  `,
+		['WordPress', 'Edit Post', 'GraphiQL IDE', 'Test Node'],
 	);
 });
 
@@ -318,11 +280,11 @@ class TestAddToolbarNodePlugin {
 function testToolBarNode(
 	toolBar: HTMLElement,
 	expectedLen: number,
-	expectedContent: string,
+	expectedContent: string[],
 ) {
 	const { getAllByRole } = within(toolBar);
 	const items = getAllByRole('listitem');
 	expect(items.length).toBe(expectedLen);
 	const toolBarNames = items.map((item) => item.textContent);
-	expect(toolBarNames).toMatchInlineSnapshot(expectedContent);
+	expect(toolBarNames).toEqual(expectedContent);
 }