-
Notifications
You must be signed in to change notification settings - Fork 504
[tests] coverage dopamine #1126
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
Conversation
Codecov ReportAttention: Patch coverage is
📢 Thoughts on this report? Let us know! |
factory.py is really hard to get any higher coverage |
|
please dont hate me, i also included formatting fixes |
Added more testing to files that needed it
Also implemented some missing logic for creating spans from generator types
a lot of this testing doesn't confirm that we're doing things right, but it will warn us if we make changes in the future that effect the current functionality