Skip to content
This repository was archived by the owner on Feb 22, 2026. It is now read-only.

Update CodeQL workflow and add unit test suite#1

Merged
theusmarshall merged 1 commit intomainfrom
fork-customizations
Feb 18, 2026
Merged

Update CodeQL workflow and add unit test suite#1
theusmarshall merged 1 commit intomainfrom
fork-customizations

Conversation

@theusmarshall
Copy link
Owner

Summary

  • CodeQL workflow: Updated all GitHub Actions from end-of-life v1/v2 to current v3/v4, fixed branch references from master to main, added required permissions block, and removed obsolete manual PR checkout workaround.
  • Unit tests: Added 55 tests using node:test covering proxy cache (parseCacheControl, generateKey, setFilteredHeaders, calculateHeuristicMaxAge, cache state transitions), playlist reader (mp3 filtering, integration), and radar utils (coordinate conversions, color remapping with mock canvas).

Test plan

  • All 55 unit tests pass: node --test tests/unit/*.test.mjs
  • Verify CodeQL workflow runs correctly on push to main
  • Verify app starts and loads normally: npm start then open http://localhost:8080

Made with Cursor

- Update CodeQL workflow: actions v1->v3, checkout v2->v4, branch refs
  master->main, add required permissions block
- Add unit tests (node:test) for proxy/cache.mjs: parseCacheControl,
  generateKey, setFilteredHeaders, calculateHeuristicMaxAge, and
  cache state transitions (fresh/stale/miss)
- Add unit tests for playlist-reader: mp3 filtering and integration
- Add unit tests for radar-utils: coordinate conversion functions
  and removeDopplerRadarImageNoise color remapping with mock canvas

Co-authored-by: Cursor <cursoragent@cursor.com>
@theusmarshall theusmarshall merged commit 7e426b4 into main Feb 18, 2026
3 checks passed
@theusmarshall theusmarshall deleted the fork-customizations branch February 18, 2026 05:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant