Skip to content

test: rust daemon unit test coverage#734

Merged
CougarTasker merged 1 commit into
mainfrom
test/rust-daemon-unit-tests
Jun 29, 2026
Merged

test: rust daemon unit test coverage#734
CougarTasker merged 1 commit into
mainfrom
test/rust-daemon-unit-tests

Conversation

@CougarTasker

@CougarTasker CougarTasker commented Jun 29, 2026

Copy link
Copy Markdown
Member

Features

  • Adds unit-test coverage across the Rust daemon: the cloud coordinators (uploader, status, progress, registration), the recording_reaper watcher, the pipeline (dispatcher, json_writer, trace_actor), the API client's retry/auth paths, and the shared env-config layer.
  • Producer video spool is now lossless PNG, not raw RGB24. Each frame is PNG-encoded (truecolour, 8-bit, "Up" filter, deflate level 1) on the writer/drain thread before it hits disk, so the on-disk .nut — and the daemon's transcode read-back of it — is a fraction of the raw video bandwidth. Chunk rolls key off decoded-equivalent ("logical") bytes so per-chunk transcode granularity is unchanged regardless of compression ratio. Buffers and the zlib compressor are reused per writer, so a steady-state frame allocates nothing. Adds an ffmpeg-backed bit-exact round-trip test.
  • Minor test-enablement refactors: the upload-chunk backoff cap now reads the API client's configurable max_backoff (lets retry/stall tests run without real-time sleeps), and NDD_DEBUG parsing now shares the common truthy set.

Bugfixes

  • None

Items

Related PRs

@CougarTasker CougarTasker self-assigned this Jun 29, 2026
@CougarTasker CougarTasker added the version:patch non-breaking bug fixes; no input/output changes (except defaults) label Jun 29, 2026
@CougarTasker CougarTasker marked this pull request as ready for review June 29, 2026 15:10

@damonhayhurst-neuraco damonhayhurst-neuraco left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@CougarTasker CougarTasker force-pushed the test/rust-daemon-unit-tests branch from 6d98778 to f7b2eb6 Compare June 29, 2026 15:46
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch from 0448e5b to 627dce2 Compare June 29, 2026 15:46
@CougarTasker CougarTasker force-pushed the test/rust-daemon-unit-tests branch from f7b2eb6 to fed3aa0 Compare June 29, 2026 15:59
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch 2 times, most recently from 9981f19 to 747b83d Compare June 29, 2026 16:49
@CougarTasker CougarTasker force-pushed the test/rust-daemon-unit-tests branch 2 times, most recently from 633bdca to f1530b4 Compare June 29, 2026 22:56
@CougarTasker CougarTasker force-pushed the test/rust-daemon-tests branch 2 times, most recently from 4ac7ed4 to 2af91bd Compare June 29, 2026 23:09
Base automatically changed from test/rust-daemon-tests to main June 29, 2026 23:16
@CougarTasker CougarTasker force-pushed the test/rust-daemon-unit-tests branch from f1530b4 to 1881f9e Compare June 29, 2026 23:18
@CougarTasker CougarTasker merged commit 52e8ff2 into main Jun 29, 2026
20 checks passed
@CougarTasker CougarTasker deleted the test/rust-daemon-unit-tests branch June 29, 2026 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:patch non-breaking bug fixes; no input/output changes (except defaults)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants