Skip to content

Commit 4ece78a

Browse files
committed
skip webkit for now
1 parent a1ee9c4 commit 4ece78a

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

packages/integration-tests/suites/wasm/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ import { shouldSkipWASMTests } from '../../utils/wasmHelpers';
77

88
sentryTest(
99
'captured exception should include modified frames and debug_meta attribute',
10-
async ({ getLocalTestPath, page }) => {
11-
if (shouldSkipWASMTests()) {
10+
async ({ getLocalTestPath, page, browserName }) => {
11+
if (shouldSkipWASMTests(browserName)) {
1212
sentryTest.skip();
1313
}
1414

packages/integration-tests/utils/wasmHelpers.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,14 @@
22
* We can only test WASM tests in certain bundles/packages:
33
* - NPM (ESM, CJS)
44
* - ES6 CDN bundles
5+
* - On browsers other than WebKit
56
*
67
* @returns `true` if we should skip the replay test
78
*/
8-
export function shouldSkipWASMTests(): boolean {
9+
export function shouldSkipWASMTests(browser: string): boolean {
10+
if (browser === 'webkit') {
11+
return true;
12+
}
913
const bundle = process.env.PW_BUNDLE as string | undefined;
1014
return bundle != null && bundle.includes('es5');
1115
}

0 commit comments

Comments
 (0)