From 83a98cccf34334e225dad25b43f701fe4c54f2e0 Mon Sep 17 00:00:00 2001
From: Max Schmitt <max@schmitt.mx>
Date: Fri, 28 Jul 2023 18:48:34 +0200
Subject: [PATCH] test: skip tracing test on trace mode (#24483)

Thats like we do it here:
https://github.com/microsoft/playwright/blob/c8f2fc392a8c9a6c38c491dbd7134e86443e6a83/tests/library/tracing.spec.ts#L26
---
 tests/library/browsercontext-reuse.spec.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/library/browsercontext-reuse.spec.ts b/tests/library/browsercontext-reuse.spec.ts
index dce50ad997..fb994f4eea 100644
--- a/tests/library/browsercontext-reuse.spec.ts
+++ b/tests/library/browsercontext-reuse.spec.ts
@@ -235,7 +235,9 @@ test('should reset mouse position', async ({ reusedContext, browserName, platfor
   await expect(page.locator('#two')).toHaveCSS('background-color', 'rgb(0, 0, 255)');
 });
 
-test('should reset tracing', async ({ reusedContext }, testInfo) => {
+test('should reset tracing', async ({ reusedContext, trace }, testInfo) => {
+  test.skip(trace === 'on');
+
   let context = await reusedContext();
   await context.tracing.start();