cherry-pick(release-1.9): skip stack trace playwright/src lines only under tests (#5594) (#5599)

This cherry-picks 070cfdc Co-authored-by: Max Schmitt <max@schmitt.mx>

Author: aslushnikov

src/utils/stackTrace.ts

@@ -41,7 +41,7 @@ const PW_LIB_DIRS = [
  'playwright-chromium',
  'playwright-firefox',
  'playwright-webkit',
-].map(packageName => path.join(packageName, 'lib'));
+].map(packageName => path.join('node_modules', packageName, 'lib'));
 
 export function captureStackTrace(): { stack: string, frames: StackFrame[] } {
  const stack = new Error().stack!;
@@ -56,7 +56,7 @@ export function captureStackTrace(): { stack: string, frames: StackFrame[] } {
  if (PW_LIB_DIRS.some(libDir => fileName.includes(libDir)))
  continue;
  // for tests.
- if (fileName.includes(path.join('playwright', 'src')))
+ if (isUnderTest() && fileName.includes(path.join('playwright', 'src')))
  continue;
  if (isUnderTest() && fileName.includes(path.join('playwright', 'test', 'coverage.js')))
  continue;

test/fixtures.spec.ts

@@ -19,6 +19,7 @@ import { folio, RemoteServer } from './remoteServer.fixture';
 import { execSync } from 'child_process';
 import path from 'path';
 import * as stackTrace from '../src/utils/stackTrace';
+import { setUnderTest } from '../src/utils/utils';
 
 type FixturesFixtures = {
  connectedRemoteServer: RemoteServer;
@@ -129,6 +130,7 @@ describe('fixtures', (suite, { platform, headful }) => {
 });
 
 it('caller file path', async ({}) => {
+ setUnderTest();
  const callme = require('./fixtures/callback');
  const filePath = callme(() => {
  return stackTrace.getCallerFilePath(path.join(__dirname, 'fixtures') + path.sep);