Skip to content

Add coverage for backend edge cases - #35

Merged
tripledoublev merged 4 commits into
mainfrom
codex/coverage-gaps
May 31, 2026
Merged

tripledoublev merged 4 commits into
mainfrom
codex/coverage-gaps

Conversation

@tripledoublev

Copy link
Copy Markdown
Collaborator

Summary

  • add hermetic unit coverage for malformed URLs, DHT parsing, ciphertext envelopes, permissions, repo-slot collision handling, and RPC response parsing
  • harden URL/key/hash parsing and encrypted payload handling so bad inputs return errors instead of panicking
  • make group repo advertisement/load logic sparse-slot aware and keep RPC list/remove group IDs roundtrippable

Validation

  • cargo fmt
  • cargo test --lib group::tests
  • cargo test --lib repo::tests
  • cargo test --lib parse_remove_group_id
  • cargo test --lib listed_group_id_roundtrips_through_remove_parser
  • timeout 120 cargo test --lib refresh_group_reloads_group_after_cache_invalidation -- --nocapture
  • cargo test --lib --no-run
  • cargo check
  • git diff --check

Full cargo test --lib was not run because the existing suite is network-heavy and slow.

- Error instead of panic on malformed join-URL keys and short ciphertext
- Use the repo record's own owner secret as its write credential
- Move nextest.toml to .config/ and use slow-timeout so it actually loads
@tripledoublev
tripledoublev force-pushed the codex/coverage-gaps branch from e256ff0 to 89883a8 Compare May 31, 2026 16:07
# Conflicts:
#	src/backend.rs
#	src/common.rs
@tripledoublev
tripledoublev merged commit 1ad0b3b into main May 31, 2026
1 check 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.

1 participant