Skip to content

feat(cli): Support trace file URLs#9030

Merged
dgozman merged 13 commits into
microsoft:masterfrom
sidharthv96:feat/downloadTrace
Oct 1, 2021
Merged

feat(cli): Support trace file URLs#9030
dgozman merged 13 commits into
microsoft:masterfrom
sidharthv96:feat/downloadTrace

Conversation

@sidharthv96
Copy link
Copy Markdown
Contributor

@sidharthv96 sidharthv96 commented Sep 20, 2021

CLI now supports

npx playwright show-trace https://bucket.storage.com/playwright-traces/PlaywrightTrace.979f48679fe4a110.zip

Fixes #8284

@mxschmitt mxschmitt requested a review from dgozman September 23, 2021 21:40
Copy link
Copy Markdown
Collaborator

@dgozman dgozman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR! I've left some comments, but looks nice overall.

Comment thread docs/src/trace-viewer.md Outdated
Comment thread src/utils/utils.ts Outdated
Comment thread src/server/trace/viewer/traceViewer.ts Outdated
* upstream/master: (33 commits)
  chore: migrate expect(locator).toMatchText to protocol (microsoft#9117)
  browser(webkit): fix text padding (microsoft#9114)
  test: unflake "should have correct snapshot size" (microsoft#9099)
  fix: make Serializable type any (microsoft#9059)
  feat(test runner): add some fixture debugging (microsoft#8918)
  chore(typescript): bump typescript to v4.4.3 (microsoft#9087)
  test: add failing WebKit navigation test (microsoft#9097)
  fix(firefox): use global eval in injected script (microsoft#9101)
  feat(launch): connect to process.env.SELENIUM_REMOTE_URL instead of local launch (microsoft#9056)
  test: unflake "should close the context without awaiting the failed download" (microsoft#9094)
  chore: simplify dom tasks (microsoft#9089)
  test: unflake inspector/pause.spec.ts (microsoft#9096)
  test: unflake "should fail when frame detaches" (microsoft#9095)
  test: unflake "should timeout in socket while connecting" (microsoft#9093)
  fix(junit): produce a single system-out entry (microsoft#9091)
  fix(css selector): absolutize relative CSS selectors (microsoft#9088)
  feat(fetch): support options in playwright._newRequest (microsoft#9061)
  chore(registry): refactor logic of what browsers to download to top level (microsoft#9078)
  chore(babel): rename .babelrc to babel.config.json (microsoft#9076)
  docs: improve ignorehttpserrors description (microsoft#9081)
  ...
@sidharthv96 sidharthv96 requested a review from dgozman September 25, 2021 06:11
Comment thread src/utils/utils.ts Outdated
Comment thread src/utils/download.ts Outdated
Comment thread src/utils/download.ts Outdated
Comment thread src/utils/download.ts Outdated
* upstream/master: (26 commits)
  browser(webkit): disable COOP support (microsoft#9185)
  fix(test runner): proper serial mode with beforeAll/afterAll failures (microsoft#9183)
  chore: remove browserType.connect from .net - not yet ready (microsoft#9182)
  test: canvas updates are reflected on screenshots (microsoft#9180)
  chore: dedupe return types in the dotnet api generator (microsoft#9181)
  docs: mention empty string in `userDataDir` (microsoft#9069)
  feat(expect): toContainText(array) (microsoft#9160)
  chore: enable object-curly-spacing in ESLint (microsoft#9168)
  feat(chromium): roll to r925110 (microsoft#9175)
  chore: render expect in trace viewer (microsoft#9141)
  devops: use Node.js 16 when rolling browsers
  docs(python): add docs about threading (microsoft#8829)
  browser(chromium): roll to r925110 (microsoft#9171)
  chore(test-runner): launch -> webServer (microsoft#9167)
  docs: cleanup test.describe.parallel.only doc (microsoft#9159)
  chore: remove FatalDOMError (microsoft#9119)
  feat(test runner): improve fixture typings for function fixtures (microsoft#9138)
  fix(expect): produce "waiting for selector" log, corner cases (microsoft#9140)
  fix: increase recent logs buffer (microsoft#9143)
  test: unflake a few tests (microsoft#9142)
  ...
@sidharthv96 sidharthv96 requested a review from dgozman September 28, 2021 04:19
Copy link
Copy Markdown
Collaborator

@dgozman dgozman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for being slow, I missed this yesterday.

Comment thread src/server/trace/viewer/traceViewer.ts
@sidharthv96 sidharthv96 requested a review from dgozman September 30, 2021 02:46
Copy link
Copy Markdown
Collaborator

@dgozman dgozman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thank you! However, I cannot merge right now because there are merge conflicts. Could you please rebase one more time?

* upstream/master: (21 commits)
  fix(test runner): do not write missing snapshot until the last retry (microsoft#9246)
  feat(reporters): Add error position to JSON Report (microsoft#9151)
  feat(fetch): import/export storageState (microsoft#9244)
  fix: allow binary response interception (microsoft#9236)
  test(cookies): add a test for SameSite=None cookies (microsoft#9242)
  feat(webkit): roll WebKit to 1550 (microsoft#9239)
  test: add test for downloading PDF files (microsoft#9235)
  fix: leaking route handlers when times is used in Page.route (microsoft#9234)
  chore: upgrade commander.js to version 8 (microsoft#9230)
  feat(fetch): store cookies between requests (microsoft#9221)
  fix(selenium connect): register in gracefullyCloseAll for driver cleanup (microsoft#9218)
  fix(toBeHidden): return true to missing elements (microsoft#9205)
  chore: fix PlaywrightClient disconnection logic (microsoft#9149)
  fix(expect): beautiful expect stacks (microsoft#9204)
  feat(fetch): support ignoreHTTPSErrors option (microsoft#9206)
  feat(api): introduce locator.waitFor (microsoft#9200)
  feat(fetch): send Playwright as default user-agent for global fetch (microsoft#9195)
  feat(test runner): collect test error from worker teardown (microsoft#9190)
  test: add tests for Cross-Origin-Opener-Policy navigation (microsoft#9184)
  test: get response body for COOP responses (microsoft#9196)
  ...
@sidharthv96 sidharthv96 requested a review from dgozman October 1, 2021 06:17
Copy link
Copy Markdown
Collaborator

@dgozman dgozman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful!

@dgozman dgozman merged commit 3296c21 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.

[Feature] Open trace files via URL

2 participants