Skip to content

Increase colour_transform coverage#241

Merged
antond-weta merged 1 commit intoAcademySoftwareFoundation:mainfrom
soswow:colour-transform-coverage
Jan 28, 2026
Merged

Increase colour_transform coverage#241
antond-weta merged 1 commit intoAcademySoftwareFoundation:mainfrom
soswow:colour-transform-coverage

Conversation

@soswow
Copy link
Contributor

@soswow soswow commented Jan 17, 2026

more tests

@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.21%. Comparing base (61c40f8) to head (3b731ee).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
+ Coverage   88.86%   89.21%   +0.35%     
==========================================
  Files          15       15              
  Lines        2532     2532              
  Branches      381      381              
==========================================
+ Hits         2250     2259       +9     
+ Misses        282      273       -9     
Files with missing lines Coverage Δ
src/rawtoaces_util/cache_base.h 100.00% <ø> (ø)
src/rawtoaces_util/transform_cache.cpp 100.00% <ø> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61c40f8...3b731ee. Read the comment docs.

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

@soswow soswow force-pushed the colour-transform-coverage branch from 8404bbe to 0f9eec8 Compare January 17, 2026 13:07
@@ -14,7 +14,7 @@ std::ostream &
operator<<( std::ostream &os, const rta::core::Metadata &data [[maybe_unused]] )
{
return os << std::string( "<Metadata>" );
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return os << std::string( "<Metadata>" );
os << std::string( "<Metadata>" );
return os;

would this help?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

=(

Image

Signed-off-by: Aleksandr Motsjonov <[email protected]>
@antond-weta antond-weta force-pushed the colour-transform-coverage branch from 0f9eec8 to 3b731ee Compare January 28, 2026 07:53
@antond-weta antond-weta merged commit 730fa59 into AcademySoftwareFoundation:main Jan 28, 2026
18 checks passed
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.

3 participants