From 09b27eb9da60bbe4a527221dbb64fa52c665ffee Mon Sep 17 00:00:00 2001 From: Sam Macbeth Date: Mon, 23 Oct 2023 17:46:59 +0200 Subject: [PATCH] Disable local proxying for adclick attribution tests (#2285) --- integration-test/click-attribution.spec.js | 3 --- integration-test/helpers/testPages.js | 7 +------ 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/integration-test/click-attribution.spec.js b/integration-test/click-attribution.spec.js index a40672f15b..33c4d24973 100644 --- a/integration-test/click-attribution.spec.js +++ b/integration-test/click-attribution.spec.js @@ -1,7 +1,6 @@ import { test, expect } from './helpers/playwrightHarness' import backgroundWait from './helpers/backgroundWait' import testCases from 'privacy-test-pages/adClickFlow/shared/testCases.json' -import { routeFromLocalhost } from './helpers/testPages' if (testCases.length === 0) { throw new Error('No test cases found') @@ -51,8 +50,6 @@ test.describe('Ad click blocking', () => { // Allow to filter to one test case const itMethod = testCase.only ? test.only : test itMethod(testCase.name, async ({ context }) => { - // route requests from all pages in this test to our local test server - await routeFromLocalhost(context) let page = await context.newPage() for (const step of testCase.steps) { if (step.action.type === 'navigate') { diff --git a/integration-test/helpers/testPages.js b/integration-test/helpers/testPages.js index 66711f55ff..63ebb7d2c5 100644 --- a/integration-test/helpers/testPages.js +++ b/integration-test/helpers/testPages.js @@ -2,12 +2,7 @@ const testPageHosts = new Set([ 'privacy-test-pages.site', 'broken.third-party.site', 'good.third-party.site', - 'bad.third-party.site', - 'convert.ad-company.site', - // 'www.search-company.site', - // 'www.ad-company.site', - redirects to these domains via route overriding seem to hang, so this one has to hit the real server - 'www.publisher-company.site', - 'www.payment-company.site' + 'bad.third-party.site' ]) export const TEST_SERVER_ORIGIN = 'http://127.0.0.1:3000'