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

Flaky App Start Test #4197

Open
krystofwoldrich opened this issue Oct 21, 2024 · 1 comment · May be fixed by #4299
Open

Flaky App Start Test #4197

krystofwoldrich opened this issue Oct 21, 2024 · 1 comment · May be fixed by #4299

Comments

@krystofwoldrich
Copy link
Member

Description

This test is flaky. (One in 10 run, maybe less, but it is)

**FAIL  test/tracing/integrations/appStart.test.ts (10.966 s)
    ● App Start Integration › Standalone App Start › adds ui kit init start mark as a child of the main app start span
  
      expect(received).toEqual(expected) // deep equality
  
      - Expected  - 3
      + Received  + 9
  
      - ObjectContaining {
      -   "data": ObjectContaining {
      + Object {
      +   "data": Object {
      +     "sentry.op": "app.start.cold",
            "sentry.origin": "auto.app.start",
          },
          "description": "UIKit Init to JS Exec Start",
      +   "op": "app.start.cold",
          "origin": "auto.app.start",
      +   "parent_span_id": "8c54b0acc4d08d8b",
      +   "span_id": "9275b85b7ed89654",
          "start_timestamp": 1729519015.859,
      -   "timestamp": NumberCloseTo 1729519015.909 (2 digits),
      +   "status": "ok",
      +   "timestamp": 1729519015.9[26](https://github.com/getsentry/sentry-react-native/actions/runs/11441700199/job/31830365871#step:6:27),
      +   "trace_id": "2ec172e61a3143f5b5ae84182fee7d19",
        }
  
        [31](https://github.com/getsentry/sentry-react-native/actions/runs/11441700199/job/31830365871#step:6:32)1 |
        312 |       expect(nativeRuntimeInitSpan).toBeDefined();
      > 313 |       expect(nativeRuntimeInitSpan).toEqual(
            |                                     ^
        314 |         expect.objectContaining(<Partial<SpanJSON>>{
        315 |           description: 'UIKit Init to JS Exec Start',
        316 |           start_timestamp: (timeOriginMilliseconds - 100) / 1000,
  
        at Object.<anonymous> (test/tracing/integrations/appStart.test.ts:313:[37](https://github.com/getsentry/sentry-react-native/actions/runs/11441700199/job/31830365871#step:6:38))
        at fulfilled (node_modules/tslib/tslib.js:166:62)
**
@krystofwoldrich
Copy link
Member Author

@krystofwoldrich krystofwoldrich moved this from Needs Discussion to Backlog in Mobile & Cross Platform SDK Nov 8, 2024
@antonis antonis linked a pull request Nov 21, 2024 that will close this issue
10 tasks
@antonis antonis self-assigned this Nov 21, 2024
@antonis antonis moved this from Backlog to Needs Review in Mobile & Cross Platform SDK Nov 21, 2024
@antonis antonis moved this from Needs Review to In Progress in Mobile & Cross Platform SDK Nov 21, 2024
@antonis antonis moved this from In Progress to Needs Review in Mobile & Cross Platform SDK Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Review
Development

Successfully merging a pull request may close this issue.

2 participants