Skip to content

feat(test-runner): re-use context and page when debug mode is used#9115

Merged
mxschmitt merged 6 commits into
microsoft:masterfrom
mxschmitt:feature/re-use-page-in-debugging
Oct 1, 2021
Merged

feat(test-runner): re-use context and page when debug mode is used#9115
mxschmitt merged 6 commits into
microsoft:masterfrom
mxschmitt:feature/re-use-page-in-debugging

Conversation

@mxschmitt
Copy link
Copy Markdown
Contributor

@mxschmitt mxschmitt commented Sep 23, 2021

@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch from 8c2c109 to 8a9a6f9 Compare September 24, 2021 14:23
@mxschmitt mxschmitt marked this pull request as ready for review September 24, 2021 14:24
@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch 2 times, most recently from f6d9131 to 7e9291f Compare September 24, 2021 14:36
Comment thread src/test/cli.ts
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
@pavelfeldman
Copy link
Copy Markdown
Member

I think the direction is good, we just need to be a bit more careful in the implementation.

@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch from e7dbf03 to 4a0519a Compare September 25, 2021 12:40
Comment thread src/test/index.ts Outdated
Comment thread src/test/cli.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch 2 times, most recently from b7d8a9b to 1967701 Compare September 30, 2021 13:07
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
Comment thread src/test/index.ts Outdated
@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch from 1967701 to ac359ee Compare September 30, 2021 21:17
Comment thread src/test/index.ts Outdated
@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch from 32e95af to 9b1db46 Compare September 30, 2021 22:31
@mxschmitt mxschmitt force-pushed the feature/re-use-page-in-debugging branch from 9b1db46 to aedbe9f Compare October 1, 2021 06:50
@mxschmitt mxschmitt merged commit e674d87 into microsoft:master Oct 1, 2021
sidharthv96 added a commit to sidharthv96/playwright that referenced this pull request Oct 2, 2021
…tionWithBaseChange

* upstream/master:
  chore: roll Electron to 12.2.1 (microsoft#9271)
  test: use separate Playwright instance to automate inspector (microsoft#9270)
  feat(inspector): use chrome/msedge when chromium is not available (microsoft#9269)
  fix(webkit): deduce response mime type from content-type (microsoft#9264)
  fix(fetch): use data, form and multipart for different post data (microsoft#9248)
  chore: split ContextRecorder from inspector (microsoft#9250)
  feat: add path option to `toMatchSnapshot` (microsoft#9156)
  feat(cli): Support trace file URLs (microsoft#9030)
  browser(webkit): roll to 23/09/21 (microsoft#9107)
  feat(chromium): roll to r926934 (microsoft#9259)
  browser(chromium): roll to r926934 (microsoft#9255)
  test: fix 'should fulfill with fetch result and overrides' test (microsoft#9252)
  feat(test-runner): add reuse context mode to share a single context between tests (microsoft#9115)
  fix(docs): test-runner location column type (microsoft#9222)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants