Cover RGB and RGBA color APIs

[DongjaJ]

What

Add focused unit tests for RGB and RGBA color APIs, including construction, conversions, hex parsing edge cases, arithmetic, transforms, and Bevy feature conversions.

Why

RGB/RGBA color APIs lacked direct coverage for several public conversion and parsing paths. These tests document the expected component behavior and bring rgb.rs and rgba.rs line coverage to 100% when the Bevy feature is enabled.

Closes #89

Before

After

Checklist

Required

• cargo check --all passes
• cargo fmt --all -- --check passes
• cargo clippy --workspace --all-targets -- -D warnings -A clippy::multiple-crate-versions passes
• cargo test --all passes
• I linked the related issue (for example: Closes #123)

Functional Validation

• Behavior related to this change was verified locally (if applicable)
• Rendering/backend behavior was verified when runtime code changed (if applicable)
• Algorithm behavior (pathfinding/FOV/noise/random) was verified when affected (if applicable)
• I added or updated tests for changed behavior (if applicable)

Configuration & Docs

• User-facing docs were updated (README.md, ARCHITECTURE.md, or relevant manual pages, if applicable)
• New dependencies/configuration are documented (if applicable)
• No sensitive values or credentials were introduced

If Applicable

• Security impact considered (run cargo audit locally if needed)
• Breaking behavior changes are clearly described in this PR

Verification

• cargo check --all
• cargo fmt --all -- --check
• cargo clippy --workspace --all-targets -- -D warnings -A clippy::multiple-crate-versions
• cargo test --all
• cargo test -p bracket-color --features bevy
• cargo llvm-cov -p bracket-color --features bevy --html --open

Summary by CodeRabbit

• Tests
  • Expanded test coverage for color conversions (RGB, RGBA, HSV) including edge cases and clamping behavior.
  • Improved hex parsing tests with more comprehensive validation scenarios.
  • Enhanced round-trip conversion testing to ensure data integrity across color formats.

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR expands and restructures test coverage for RGB and RGBA color types. New parameterized tests validate conversion, arithmetic, parsing, and Bevy integration behavior; interpolation assertions are refined to use exact equality checks instead of epsilon tolerance.

Changes

Color Type Test Expansion

Layer / File(s)	Summary
RGB conversion and arithmetic tests bracket-color/src/rgb.rs	New tests for from_f32 clamping, from_u8 component conversion, tuple-to-RGB construction, RGBA-to-RGB alpha dropping, HSV delegation, and parameterized component-wise arithmetic operator behavior.
RGB transformation and parsing tests bracket-color/src/rgb.rs	Parameterized HSV wrap-around case, multi-alpha to_rgba tests, XP round-trip and clamping tests, expanded hex parsing rejection cases via rstest, conditional Bevy color-channel preservation tests, and updated assertions for named BLUE, greyscale luminance, desaturate, and lerp interpolation.
RGBA conversion and arithmetic tests bracket-color/src/rgba.rs	Parameterized tests for alpha fill/preserve behavior in conversions, HSV-to-RGBA opaque alpha, and component-wise arithmetic operators (add, sub, mul) with constructor clamping.
RGBA parsing and interpolation refinements bracket-color/src/rgba.rs	Expanded hex parsing rejection tests with additional invalid-length cases and parameterized invalid-character checks, conditional Bevy as_rgba_f32 and channel preservation tests, and updated lerp and lerp_alpha assertions to use exact assert_rgba_eq equality instead of epsilon tolerance.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• Add unit tests for RGB and RGBA color APIs #89: Implements expanded parameterized unit test coverage for RGB and RGBA conversions, arithmetic, clamping, XP operations, greyscale/desaturation, and interpolation as requested.

Suggested labels

size/M

Suggested reviewers

• utilForever

Poem

🐰 A rabbit hops through tests so bright,
Colors clamped and parsed just right,
From RGB to RGBA's gleam,
Arithmetic flows like a dream!
With lerp so exact, no tolerance jest,
These specs now pass the fullest test! 🎨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title 'Cover RGB and RGBA color APIs' directly and clearly summarizes the main change: adding comprehensive test coverage for RGB and RGBA color APIs.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[utilForever]

LGTM. Thanks for your work! 🎉