refactor to remove exposing proxy server url

Author: mutdmour

packages/testing/containers/n8n-test-container-creation.ts

@@ -86,7 +86,6 @@ export interface N8NStack {
 	baseUrl: string;
 	stop: () => Promise<void>;
 	containers: StartedTestContainer[];
-	proxyServerUrl?: string;
 }
 
 /**
@@ -274,7 +273,6 @@ export async function createN8NStack(config: N8NConfig = {}): Promise<N8NStack>
 
 	return {
 		baseUrl,
-		proxyServerUrl,
 		stop: async () => {
 			await stopN8NStack(containers, network, uniqueProjectName);
 		},

packages/testing/playwright/fixtures/base.ts

@@ -164,15 +164,24 @@ export const test = base.extend<TestFixtures, WorkerFixtures>({
 
 	proxyServer: async ({ n8nContainer }, use) => {
 		// n8nContainer is "null" if running tests in "local" mode
-		// proxy server is also not initialized in local mode (only in container modes can it be supported)
-		// tests that require proxy server should have "@capability:proxy" so that they are skipped in local mode
-		if (!n8nContainer?.proxyServerUrl) {
+		if (!n8nContainer) {
 			throw new TestError(
 				'Testing with Proxy server is not supported when using N8N_BASE_URL environment variable. Remove N8N_BASE_URL to use containerized testing.',
 			);
 		}
 
-		const proxyServer = new ProxyServer(n8nContainer.proxyServerUrl);
+		const proxyServerContainer = n8nContainer.containers.find((container) =>
+			container.getName().endsWith('proxyserver'),
+		);
+
+		// proxy server is not initialized in local mode (it be only supported in container modes)
+		// tests that require proxy server should have "@capability:proxy" so that they are skipped in local mode
+		if (!proxyServerContainer) {
+			throw new TestError('Proxy server container not initialized. Cannot initialize client.');
+		}
+
+		const serverUrl = `http://${proxyServerContainer?.getHost()}:${proxyServerContainer?.getFirstMappedPort()}`;
+		const proxyServer = new ProxyServer(serverUrl);
 		await proxyServer.loadExpectations();
 
 		await use(proxyServer);