Add animation frame integrity tests #49

New issue

Closed

opened 2026-05-12 00:50:33 -04:00 by JackFrostbyte · 0 comments

JackFrostbyte commented

2026-05-12 00:50:33 -04:00

Owner

Context

RGB Aura now has multiple registered animations across GPU, motherboard, top Fractal fans, and Thermaltake/front fans.

Before future refactors, we should add frame integrity tests to make sure registered animations keep producing valid LED frames.

Scope

Add parametrized tests over the animation registry where practical.

Test for:

Expected frame type/shape.
No invalid color values.
Frame length matches the target zone or declared usable LED count.
Animations do not write outside their allowed indexes.
All registered animations can generate at least one valid frame.

Acceptance criteria

Registered animations are covered by integrity tests.
Tests do not require real OpenRGB hardware.
Invalid frame length or invalid RGB values would fail tests.
ruff format passes.
ruff check passes.
Test suite passes.

Branch workflow

Base branch:

develop

Suggested work branch:

test/issue-49-animation-frame-integrity

Pull request target:

develop

Do not merge directly into main. main should only receive tested release-ready changes from develop.

## Context RGB Aura now has multiple registered animations across GPU, motherboard, top Fractal fans, and Thermaltake/front fans. Before future refactors, we should add frame integrity tests to make sure registered animations keep producing valid LED frames. ## Scope Add parametrized tests over the animation registry where practical. Test for: - Expected frame type/shape. - No invalid color values. - Frame length matches the target zone or declared usable LED count. - Animations do not write outside their allowed indexes. - All registered animations can generate at least one valid frame. ## Acceptance criteria - Registered animations are covered by integrity tests. - Tests do not require real OpenRGB hardware. - Invalid frame length or invalid RGB values would fail tests. - `ruff format` passes. - `ruff check` passes. - Test suite passes. ## Branch workflow Base branch: `develop` Suggested work branch: `test/issue-49-animation-frame-integrity` Pull request target: `develop` Do not merge directly into `main`. `main` should only receive tested release-ready changes from `develop`.

JackFrostbyte added this to the v1.0.1 - Maintenance and startup reliability milestone

2026-05-12 01:02:40 -04:00