Skip to content

test: expand symbol validation boundary coverage#425

Merged
dinahmaccodes merged 3 commits into
Navin-xmr:mainfrom
MerlinTheWhiz:test/symbol-validation-boundaries
Jun 1, 2026
Merged

test: expand symbol validation boundary coverage#425
dinahmaccodes merged 3 commits into
Navin-xmr:mainfrom
MerlinTheWhiz:test/symbol-validation-boundaries

Conversation

@MerlinTheWhiz

Copy link
Copy Markdown
Contributor

Closes #380

Changes made:

  • contracts/shipment/src/lib.rs: Registered mod test_symbol_validation so the test file is compiled
  • contracts/shipment/src/test_symbol_validation.rs: Added 13 tests — milestone/metadata boundary acceptance + overlong rejection through existing helpers

@dinahmaccodes

Copy link
Copy Markdown
Contributor

hey please fix conflicts @MerlinTheWhiz

@MerlinTheWhiz

Copy link
Copy Markdown
Contributor Author

@dinahmaccodes Sorry for the delay. Conflicts has been resolved

@dinahmaccodes

Copy link
Copy Markdown
Contributor

@MerlinTheWhiz
one more time.
just seeing this after merging something..

@MerlinTheWhiz

Copy link
Copy Markdown
Contributor Author

@dinahmaccodes Yh done

@dinahmaccodes dinahmaccodes merged commit 996cecf into Navin-xmr:main Jun 1, 2026
@dinahmaccodes

Copy link
Copy Markdown
Contributor

perfect!!

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.

[CONTRACT] Add regression tests for symbol validation helpers

2 participants