Skip to content

test(storeMemory): add valid_from round-trip tests (DAK-7424) - #213

Merged
ferhimedamine merged 1 commit into
mainfrom
sync/valid-from-dak7571
Jul 28, 2026
Merged

test(storeMemory): add valid_from round-trip tests (DAK-7424)#213
ferhimedamine merged 1 commit into
mainfrom
sync/valid-from-dak7571

Conversation

@ferhimedamine

Copy link
Copy Markdown
Contributor

Summary

Follow-up to dakera-js PR #209 which added valid_from?: number to StoreMemoryRequest in types.ts without tests.

Changes:

  • src/client.test.ts: two serialization tests for valid_from — field present in request body when set, absent when not provided

Matching pattern: same style as the existing expires_at round-trip tests.

Closes DAK-7571 (js half of the batch).

valid_from was added to StoreMemoryRequest in PR #209 (types only).
Adds two serialization tests verifying the field lands in the request
body when set and is absent when omitted — matching the existing
expires_at test pattern.

Closes DAK-7571.
@ferhimedamine ferhimedamine added the auto-merge Auto-merge when CI passes label Jul 28, 2026
@ferhimedamine
ferhimedamine merged commit 110912b into main Jul 28, 2026
4 checks passed
@ferhimedamine
ferhimedamine deleted the sync/valid-from-dak7571 branch July 28, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Auto-merge when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant