Skip to content

Add extension attribute schema tests and CI feature matrix#94

Merged
gvonness-apolitical merged 1 commit intomainfrom
feat/extension-schema-tests-ci-matrix
Feb 16, 2026
Merged

Add extension attribute schema tests and CI feature matrix#94
gvonness-apolitical merged 1 commit intomainfrom
feat/extension-schema-tests-ci-matrix

Conversation

@gvonness-apolitical
Copy link
Collaborator

Summary

  • Add cdx-core/tests/extension_schema.rs with table-driven tests verifying field names and types for all 15 ExtensionMark convenience constructors (citation, entity, glossary, index, equation_ref, algorithm_ref, theorem_ref, highlight, etc.)
  • Add backward-compatibility deserialization tests for citation marks (both "refs" array and legacy "ref" string formats)
  • Add Citation struct schema tests verifying serialization emits "refs" (not "ref") and deserialization accepts both
  • Add citation mark archive roundtrip test (single-ref and multi-ref marks survive save/reopen via CdxWriter/CdxReader)
  • Add CI feature matrix job testing 5 feature combinations: no-features, zstd, signatures, zstd+signatures, signatures+encryption

Test plan

  • cargo test --workspace --all-features passes (1052 tests, 0 failures)
  • cargo clippy --workspace --all-features -- -D warnings clean
  • cargo build -p cdx-core --no-default-features builds successfully
  • CI feature matrix job runs all 5 combinations without failure

Prevent field-level spec divergences with table-driven tests that verify
field names and types for every ExtensionMark convenience constructor.
Also adds a citation mark archive roundtrip test and a CI job that tests
important feature combinations (no-features, zstd, signatures, etc.).
@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 94.26230% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cdx-core/tests/extension_schema.rs 95.14% 10 Missing ⚠️
cdx-core/tests/integration.rs 89.47% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@gvonness-apolitical gvonness-apolitical merged commit 88c4c83 into main Feb 16, 2026
16 checks passed
@gvonness-apolitical gvonness-apolitical deleted the feat/extension-schema-tests-ci-matrix branch February 16, 2026 18:08
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