Skip to content

(8) misc(tests): Add type check to tests #4017

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.

Sign up for GitHub

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

Jump to bottom
Merged
krystofwoldrich merged 61 commits into from
Aug 16, 2024
Merged

(8) misc(tests): Add type check to tests #4017

krystofwoldrich merged 61 commits into from
Aug 16, 2024

Conversation

@krystofwoldrich
Copy link
Contributor

@krystofwoldrich krystofwoldrich commented Aug 12, 2024 •
edited
Loading

📢 Type of change

  • Enhancement

📜 Description

Before this PR jest would use babel to strip typings without type check. Thus there were some type issues in the test suit which are now fixed as the tests won't run until tsc passes.

💚 How did you test it?

ci, locally run yarn test

fixes: #3502

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

krystofwoldrich and others added 30 commits June 3, 2024 18:39
@krystofwoldrich
chore(tracing): extract app start to a standalone integration
c3dcfd3
@krystofwoldrich
Merge branch 'v6' into kw/ref-app-start-integration
88d6988
@krystofwoldrich
fix merge
fe53af5
@krystofwoldrich
fix spans, app start is now reported to Sentry
d4ac89f
@krystofwoldrich
fix uikit init and minimal instrumentation edge cases
7c5dcaa
@krystofwoldrich
update js docs
0b42472
@krystofwoldrich
Merge branch 'v6' into kw/ref-app-start-integration
9765eaf
@krystofwoldrich
Add App Start Integration tests
b9e9e9a
@krystofwoldrich
Remove app start test from react native tracing
b334931
@krystofwoldrich
clean up app start tests
4d16787
@krystofwoldrich
fix test affected by the app start extraction
4d84922
@krystofwoldrich
Add standalone app start
0ff2020
@krystofwoldrich
fix
b1eab51
@krystofwoldrich
ref(tracing): Extract NativeFrames as standalone integration
6d1cd70
@krystofwoldrich
Add integration handling test
5eaaad2
@krystofwoldrich
Merge branch 'kw/ref-app-start-integration' into kw/ref-native-frames…
eba6820 
…-integration
@krystofwoldrich
clean up integrations tests
91c1eb8
@krystofwoldrich
move native frames tests
db70c09
@krystofwoldrich
add changelog
e199244
@krystofwoldrich
Merge branch 'kw/ref-app-start-integration' into kw/ref-native-frames…
2956344 
…-integration
@krystofwoldrich
fix
adb53fc
@krystofwoldrich
move the app start test to tracing
699fda7
@krystofwoldrich
Merge branch 'kw/ref-app-start-integration' into kw/ref-native-frames…
7a386ab 
…-integration
@krystofwoldrich
fix tests
ad98ac0
@krystofwoldrich
add changelog
f2b9abe
@krystofwoldrich
ref(tracing): Extract Stall Tracking to a standalone integration
89b2354
@krystofwoldrich
misc(tracing): Remove ReactNativeTracing deprecated options
4f3ca7b
@krystofwoldrich
fix changelog
5c12e5c
@krystofwoldrich
ref(tracing): Extract UserInteractionTracing as standalone interaction
a111bdf
@krystofwoldrich @lucas-zimerman
Apply suggestions from code review
92e04ee 
Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
@krystofwoldrich krystofwoldrich mentioned this pull request Aug 12, 2024
Closed
lucas-zimerman
lucas-zimerman reviewed Aug 12, 2024
View reviewed changes
lucas-zimerman
lucas-zimerman reviewed Aug 13, 2024
View reviewed changes
Base automatically changed from kw/ref-navigation-integrations to v6 August 14, 2024 07:58
An error occurred while trying to automatically change base from kw/ref-navigation-integrations to v6 August 14, 2024 07:58
@github-actions
Copy link
Contributor

github-actions bot commented Aug 14, 2024 •
edited
Loading

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 482.15 ms 541.21 ms 59.06 ms
Size 17.73 MiB 20.04 MiB 2.31 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Aug 14, 2024 •
edited
Loading

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1222.85 ms 1225.82 ms 2.97 ms
Size 2.36 MiB 3.06 MiB 713.36 KiB

@github-actions
Copy link
Contributor

github-actions bot commented Aug 14, 2024 •
edited
Loading

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1236.98 ms 1236.85 ms -0.13 ms
Size 2.92 MiB 3.62 MiB 719.22 KiB

lucas-zimerman
lucas-zimerman approved these changes Aug 14, 2024
View reviewed changes
Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you

@krystofwoldrich krystofwoldrich merged commit baa0217 into v6 Aug 16, 2024
@krystofwoldrich krystofwoldrich deleted the kw/add-tsc-to-tests branch August 16, 2024 10:24
@krystofwoldrich krystofwoldrich restored the kw/add-tsc-to-tests branch August 16, 2024 10:28
@krystofwoldrich krystofwoldrich mentioned this pull request Dec 10, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucas-zimerman lucas-zimerman lucas-zimerman approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@krystofwoldrich @lucas-zimerman