Skip to content

Commit 1762114

Browse files
committed
remove optimizations from other calls
1 parent 4b72d5b commit 1762114

File tree

3 files changed

+13
-11
lines changed

3 files changed

+13
-11
lines changed

Diff for: FindServiceWorker/index.ts

+6-5
Original file line numberDiff line numberDiff line change
@@ -61,12 +61,13 @@ const httpTrigger: AzureFunction = async function (
6161

6262
try {
6363
page.on('request', (req) => {
64-
if(SKIP_RESOURCES.some((type) => req.resourceType() == type)){
65-
req.abort();
66-
}
67-
else {
64+
// commented because it doesn't work on Azure environment
65+
// if(SKIP_RESOURCES.some((type) => req.resourceType() == type)){
66+
// req.abort();
67+
// }
68+
// else {
6869
req.continue();
69-
}
70+
// }
7071
});
7172

7273
try {

Diff for: FindWebManifest/index.ts

+6-5
Original file line numberDiff line numberDiff line change
@@ -156,12 +156,13 @@ async function puppeteerAttempt(site: string, context?: Context): Promise<{error
156156
try {
157157
// speed up loading
158158
page.on('request', (req) => {
159-
if(SKIP_RESOURCES.some((type) => req.resourceType() == type)){
160-
req.abort();
161-
}
162-
else {
159+
// commented because it doesn't work on Azure environment
160+
// if(SKIP_RESOURCES.some((type) => req.resourceType() == type)){
161+
// req.abort();
162+
// }
163+
// else {
163164
req.continue();
164-
}
165+
// }
165166
});
166167

167168
// waiting for manifest request or until full page loads

Diff for: playwright.config.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ export default defineConfig({
2626
/* Fail the build on CI if you accidentally left test.only in the source code. */
2727
forbidOnly: !!process.env.CI,
2828
/* Retry on CI only */
29-
retries: process.env.CI ? 1 : 0,
29+
retries: process.env.CI ? 2 : 2,
3030
/* Opt out of parallel tests on CI. */
3131
workers: process.env.CI ? 1 : undefined,
3232
/* Reporter to use. See https://playwright.dev/docs/test-reporters */

0 commit comments

Comments
 (0)