Skip to content

Add unit tests for stats module - #414

Merged
sandrobonazzola merged 1 commit into
psss:mainfrom
sandrobonazzola:did_stats
Jul 8, 2025
Merged

Add unit tests for stats module#414
sandrobonazzola merged 1 commit into
psss:mainfrom
sandrobonazzola:did_stats

Conversation

@sandrobonazzola

Copy link
Copy Markdown
Collaborator

Removed useless assertions on class existance which could always be true in boolean context.
Created more useful unit tests for stats module.
Fixed typing issues while handling this.

Comment thread tests/test_stats.py Fixed
@sandrobonazzola
sandrobonazzola force-pushed the did_stats branch 2 times, most recently from c0a7e31 to b6310fc Compare June 19, 2025 11:54
@sandrobonazzola
sandrobonazzola requested a review from psss June 19, 2025 11:54
@sandrobonazzola
sandrobonazzola force-pushed the did_stats branch 2 times, most recently from 2010d37 to 1151cea Compare June 23, 2025 14:09
Removed useless assertions on class existance which
could always be true in boolean context.
Created more useful unit tests for stats module.
Fixed typing issues while handling this.

Signed-off-by: Sandro Bonazzola <sbonazzo@redhat.com>
@sandrobonazzola sandrobonazzola self-assigned this Jul 7, 2025
@sandrobonazzola
sandrobonazzola merged commit 27bbcd5 into psss:main Jul 8, 2025
13 checks passed
@sandrobonazzola
sandrobonazzola deleted the did_stats branch July 8, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants