Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(blob): store playwright user agent #26337

Merged
merged 4 commits into from
Aug 8, 2023
Merged

Conversation

yury-s
Copy link
Member

@yury-s yury-s commented Aug 8, 2023

No description provided.

packages/playwright-test/src/reporters/blob.ts Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Test results for "tests 1"

7 flaky
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [firefox] › page/page-goto.spec.ts:81:3 › should work with Cross-Origin-Opener-Policy
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [chromium] › library/tracing.spec.ts:395:14 › should produce screencast frames fit
⚠️ [webkit] › library/browsercontext-fetch.spec.ts:1039:3 › should abort requests when browser context closes
⚠️ [webkit] › library/inspector/cli-codegen-pytest.spec.ts:56:5 › should save the codegen output to a file if specified
⚠️ [playwright-test] › ui-mode-trace.spec.ts:22:5 › should merge trace events

25042 passed, 583 skipped
✔️✔️✔️

Merge workflow run.

@yury-s yury-s changed the title chore(blob): store playwright version chore(blob): store playwright user agent Aug 8, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Test results for "tests 1"

6 failed
❌ [playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version
❌ [playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version
❌ [playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version
❌ [playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version
❌ [playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version
❌ [playwright-test] › test-step.spec.ts:1090:5 › should show final toPass error

10 flaky
⚠️ [chromium] › page/page-event-request.spec.ts:101:3 › should report navigation requests and responses handled by service worker
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [chromium] › library/tracing.spec.ts:395:14 › should produce screencast frames fit
⚠️ [playwright-test] › ui-mode-trace.spec.ts:22:5 › should merge trace events
⚠️ [playwright-test] › ui-mode-trace.spec.ts:53:5 › should merge web assertion events
⚠️ [playwright-test] › ui-mode-trace.spec.ts:126:5 › should show snapshots for sync assertions
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [webkit] › library/inspector/cli-codegen-python.spec.ts:74:5 › should save the codegen output to a file if specified
⚠️ [playwright-test] › ui-mode-test-watch.spec.ts:184:5 › should watch new file

25033 passed, 583 skipped
✔️✔️✔️

[playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version

Error: expect(received).toBe(expected) // Object.is equality

Expected: "Playwright/1.37.0-next (x64; macOS 12.6) node/16.20 CI/1"
Received: "Playwright/1.37.0-next (x64; macOS 12.6) node/16.20"

    at /Users/runner/work/playwright/playwright/tests/playwright-test/reporter-blob.spec.ts:1395:42

[playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version

Error: expect(received).toBe(expected) // Object.is equality

Expected: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/16.20 CI/1"
Received: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/16.20"

    at /home/runner/work/playwright/playwright/tests/playwright-test/reporter-blob.spec.ts:1395:42

[playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version

Error: expect(received).toBe(expected) // Object.is equality

Expected: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/18.17 CI/1"
Received: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/18.17"

    at /home/runner/work/playwright/playwright/tests/playwright-test/reporter-blob.spec.ts:1395:42

[playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version

Error: expect(received).toBe(expected) // Object.is equality

Expected: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/20.5 CI/1"
Received: "Playwright/1.37.0-next (x64; ubuntu 22.04) node/20.5"

    at /home/runner/work/playwright/playwright/tests/playwright-test/reporter-blob.spec.ts:1395:42

[playwright-test] › reporter-blob.spec.ts:1191:1 › blob report should include version

Error: expect(received).toBe(expected) // Object.is equality

Expected: "Playwright/1.37.0-next (x64; windows 10.0) node/16.20 CI/1"
Received: "Playwright/1.37.0-next (x64; windows 10.0) node/16.20"

    at D:\a\playwright\playwright\tests\playwright-test\reporter-blob.spec.ts:1395:42

[playwright-test] › test-step.spec.ts:1090:5 › should show final toPass error

Error: expect(received).toEqual(expected) // deep equality

- Expected  - 12
+ Received  +  0

@@ -9,22 +9,10 @@
      "location": Object {
        "column": "number",
        "file": "a.test.ts",
        "line": "number",
      },
-     "steps": Array [
-       Object {
-         "category": "expect",
-         "error": "<error>",
-         "location": Object {
-           "column": "number",
-           "file": "a.test.ts",
-           "line": "number",
-         },
-         "title": "expect.toBe",
-       },
-     ],
      "title": "expect.toPass",
    },
    Object {
      "category": "hook",
      "title": "After Hooks",

  1104 |   expect(result.exitCode).toBe(1);
  1105 |   const objects = result.outputLines.map(line => JSON.parse(line));
> 1106 |   expect(objects).toEqual([
       |                   ^
  1107 |     {
  1108 |       title: 'Before Hooks',
  1109 |       category: 'hook',

    at D:\a\playwright\playwright\tests\playwright-test\test-step.spec.ts:1106:19

Merge workflow run.

@yury-s yury-s merged commit 8fcb638 into microsoft:main Aug 8, 2023
@yury-s yury-s deleted the blob-pw-version branch August 8, 2023 17:39
@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2023

Test results for "tests 1"

1 failed
❌ [playwright-test] › watch.spec.ts:124:5 › should run tests on Enter

11 flaky
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [chromium] › page/page-event-request.spec.ts:130:3 › should report navigation requests and responses handled by service worker with routing
⚠️ [firefox] › page/page-event-request.spec.ts:162:3 › should return response body when Cross-Origin-Opener-Policy is set
⚠️ [firefox] › page/page-goto.spec.ts:81:3 › should work with Cross-Origin-Opener-Policy
⚠️ [playwright-test] › ui-mode-trace.spec.ts:22:5 › should merge trace events
⚠️ [playwright-test] › ui-mode-trace.spec.ts:53:5 › should merge web assertion events
⚠️ [playwright-test] › ui-mode-trace.spec.ts:126:5 › should show snapshots for sync assertions
⚠️ [chromium] › library/proxy.spec.ts:308:3 › should use SOCKS proxy for websocket requests
⚠️ [chromium] › library/tracing.spec.ts:239:5 › should not include trace resources from the previous chunks
⚠️ [playwright-test] › ui-mode-trace.spec.ts:22:5 › should merge trace events
⚠️ [webkit] › library/inspector/cli-codegen-pytest.spec.ts:32:5 › should print the correct context options when using a device and lang

25037 passed, 583 skipped
✔️✔️✔️

[playwright-test] › watch.spec.ts:124:5 › should run tests on Enter

Test timeout of 30000ms exceeded.

Merge workflow run.

Germandrummer92 pushed a commit to OctoMind-dev/playwright that referenced this pull request Oct 27, 2023
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