test: minor testing harness cleanups (#3129)

Author: mxschmitt

src/Playwright.TestingHarnessTest/tests/baseTest.ts

@@ -63,7 +63,7 @@ export const test = base.extend<{
       if (process.env.PWTEST_DEBUG) {
         cp.stdout.pipe(process.stdout);
         cp.stderr.pipe(process.stderr);
-        console.log(`Running: ${command}`);
+        console.log(`Running: TEST_MODE=${testMode} PWTEST_TEST_DIR=${testDir} ${command}`);
       }
       let [rawStdout, rawStderr] = ['', ''];
       cp.stdout.on('data', (data: Buffer) => rawStdout += data.toString());

src/Playwright.TestingHarnessTest/tests/mstest/basic.spec.ts

@@ -22,7 +22,6 @@
  * SOFTWARE.
  */
 
-import http from 'http';
 import { test, expect } from '../baseTest';
 
 test.use({ testMode: 'mstest' });

src/Playwright.TestingHarnessTest/tests/xunit/basic.spec.ts

@@ -61,10 +61,7 @@ test('should be able to forward DEBUG=pw:api env var', async ({ runTest }) => {
           </EnvironmentVariables>
         </RunConfiguration>
       </RunSettings>`,
-    }, 'dotnet test --settings=.runsettings', {
-      // Workaround until we can enable https://github.com/xunit/xunit/issues/1730#issuecomment-2330825763 assembly wide.
-      PWAPI_TO_STDOUT: '1',
-    });
+    }, 'dotnet test --settings=.runsettings');
   expect(result.passed).toBe(1);
   expect(result.failed).toBe(0);
   expect(result.total).toBe(1);