Skip to content

Stop printing in tests#722

Merged
pjonsson merged 1 commit intoopendatacube:developfrom
pjonsson:remove-prints
Feb 9, 2026
Merged

Stop printing in tests#722
pjonsson merged 1 commit intoopendatacube:developfrom
pjonsson:remove-prints

Conversation

@pjonsson
Copy link
Contributor

@pjonsson pjonsson commented Feb 6, 2026

The assert shows the output
if it fails, so stop flooding
the logs with noise.

@pjonsson
Copy link
Contributor Author

pjonsson commented Feb 6, 2026

This is failing on datacube 1.9.14 changes, so it needs to be rebased after #721 is merged.

@pjonsson pjonsson force-pushed the remove-prints branch 2 times, most recently from 9547266 to dcf9bca Compare February 9, 2026 07:20
The assert shows the output
if it fails, so stop flooding
the logs with noise.
@codecov
Copy link

codecov bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.95%. Comparing base (c3da74e) to head (0ef8853).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #722      +/-   ##
===========================================
- Coverage    76.98%   76.95%   -0.04%     
===========================================
  Files           39       39              
  Lines         3185     3172      -13     
===========================================
- Hits          2452     2441      -11     
+ Misses         733      731       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pjonsson pjonsson merged commit 4017dda into opendatacube:develop Feb 9, 2026
7 checks passed
@pjonsson pjonsson deleted the remove-prints branch February 9, 2026 12:51
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.

1 participant