Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: time range module test #1379

Draft
wants to merge 1 commit into
base: linna/test-snapshots
Choose a base branch
from

Conversation

linnall
Copy link
Collaborator

@linnall linnall commented Feb 20, 2025

Pull Request Checklist


PR-Codex overview

This PR focuses on modifying a test case for the time range module in the client.test.ts file. It changes the entityId, updates the valid time range, and adjusts the way a transaction is sent outside the valid time range.

Detailed summary

  • Changed the test to use .only for focused execution.
  • Updated signerEntity from { entityId: 1 } to { entityId: 2 }.
  • Modified validAfter and validUntil timestamps to 1722043948 and 1722043949, respectively.
  • Adjusted the transaction sending logic to capture and log the result instead of expecting an error.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Feb 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 20, 2025 11:38pm
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 20, 2025 11:38pm

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