Skip to content

Conversation

@thomasrockhu-codecov
Copy link
Collaborator

@thomasrockhu-codecov thomasrockhu-codecov commented Oct 22, 2025

The ordering was wrong

@codecov
Copy link

codecov bot commented Oct 22, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
695 1 694 1
View the full list of 1 ❄️ flaky test(s)
api.temp.calculator.test_calculator::test_divide

Flake rate in main: 99.80% (Passed 12 times, Failed 5900 times)

Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

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

Copy link

@adrianviquez adrianviquez left a comment

Choose a reason for hiding this comment

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

Lgmt; a test here could be nice to avoid order mismatch with other arguments

@thomasrockhu-codecov thomasrockhu-codecov merged commit ac2b57a into main Oct 22, 2025
26 checks passed
@thomasrockhu-codecov thomasrockhu-codecov deleted the th/fix-collector-ordering branch October 22, 2025 17:27
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.

5 participants