Skip to content
New issue

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

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

Already on GitHub? # to your account

Add tracing tests #72

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

igniteflow
Copy link

@igniteflow igniteflow commented May 14, 2023

Adds tests for xocto.tracing

Before

> pytest --cov=xocto

Name                          Stmts   Miss  Cover
-------------------------------------------------
...
xocto/tracing.py                 16     16     0%
...

After

> pytest --cov=xocto

Name                          Stmts   Miss  Cover
-------------------------------------------------
...
xocto/tracing.py                 16     0     100%
...

Note that I haven't used pytest much before. I feel like the first two lines of each test could be encapsulated in the pytest equivalent for setUp() - perhaps setup_method?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant