Skip to content

Conversation

sentrivana
Copy link
Contributor

@sentrivana sentrivana commented Jul 31, 2025

Unsure if the types in tests are any good, but might as well make them into annotations

Copy link

codecov bot commented Jul 31, 2025

⚠️ Unsupported file format

Upload processing failed due to unsupported file format. Please review the parser error message:
Error parsing JUnit XML in /home/runner/work/sentry-python/sentry-python/.junitxml at 1:6217

Caused by:
RuntimeError: Error parsing XML

Caused by:
    0: ill-formed document: close tag `</testsuite>` does not match any open tag
    1: close tag `&lt;/testsuite&gt;` does not match any open tag</code></pre>

For more help, visit our troubleshooting guide.


:x: 2 Tests Failed:

Tests completed Failed Passed Skipped
18790 2 18788 1027
View the top 2 failed test(s) by shortest run time
::tests.integrations.sanic.test_sanic
Stack Traces | 0s run time
.../integrations/sanic/test_sanic.py:333: in <module>
    class TransactionTestConfig:
.../integrations/sanic/test_sanic.py:346: in TransactionTestConfig
    has_transaction_event: bool = True,
E   TypeError: 'ABCMeta' object is not subscriptable
::tests.integrations.sanic.test_sanic
Stack Traces | 0s run time
.../integrations/sanic/test_sanic.py:333: in <module>
    class TransactionTestConfig:
.../integrations/sanic/test_sanic.py:341: in TransactionTestConfig
    integration_args: Iterable[Optional[Container[int]]],
E   TypeError: 'ABCMeta' object is not subscriptable

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@sentrivana sentrivana marked this pull request as ready for review July 31, 2025 13:04
@sentrivana sentrivana requested a review from a team as a code owner July 31, 2025 13:04
@sentrivana sentrivana merged commit d913572 into potel-base Jul 31, 2025
139 of 141 checks passed
@sentrivana sentrivana deleted the ivana/potel/clean-up-type-comments-and-transactions branch July 31, 2025 14:16
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