Skip to content

test: cover service metadata round-trip and slot independence - #53

Merged
mikewheeleer merged 1 commit into
Agentpay-Org:mainfrom
Jagadeeshftw:test/contracts-metadata-slot-independence
Jun 24, 2026
Merged

test: cover service metadata round-trip and slot independence#53
mikewheeleer merged 1 commit into
Agentpay-Org:mainfrom
Jagadeeshftw:test/contracts-metadata-slot-independence

Conversation

@Jagadeeshftw

Copy link
Copy Markdown
Contributor

Summary

Adds test-only coverage (in contracts/escrow/src/test.rs) proving the per-service storage slots are mutually independent:

  • set_service_metadata then get_service_metadata round-trips the exact description + owner.
  • A never-set service returns None from get_service_metadata.
  • register_service does not set the disabled flag (is_service_disabled == false).
  • set_service_disabled(true) preserves is_service_registered == true and the stored metadata.
  • unregister_service clears only the ServiceRegistered slot — metadata and the disabled flag survive.
  • A toggle-matrix test asserts is_service_registered / is_service_disabled move independently.

A module header note documents the slot model. No changes to lib.rs.

Closes #41

Test output

test result: ok. 29 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.21s

Security notes: Test-only change; no contract logic, storage layout, or auth surface is modified.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Jagadeeshftw
Jagadeeshftw force-pushed the test/contracts-metadata-slot-independence branch from 7936186 to 90f0a54 Compare June 24, 2026 02:30
@mikewheeleer

Copy link
Copy Markdown
Contributor

good — metadata round-trip and slot independence were untested. lgtm

@mikewheeleer
mikewheeleer merged commit c55fd1b into Agentpay-Org:main Jun 24, 2026
1 check failed
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.

Add tests for ServiceMetadata round-trip and registered/disabled slot independence

2 participants