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

Implement UnsafeSignEOTS for e2e tests #187

Closed
Lazar955 opened this issue Dec 4, 2024 · 0 comments · Fixed by #193
Closed

Implement UnsafeSignEOTS for e2e tests #187

Lazar955 opened this issue Dec 4, 2024 · 0 comments · Fixed by #193

Comments

@Lazar955
Copy link
Member

Lazar955 commented Dec 4, 2024

          > Which is expected. Should we now expose an UnsafeSignEOTS in which we would retain the old logic or mock EOTS for this e2e?

Good point. For this PR, we can add a case where the error is expected to make CI pass. A subsequent PR can implement UnsafeSignEOTS for old logic

Originally posted by @gitferry in #184 (comment)

@Lazar955 Lazar955 linked a pull request Dec 6, 2024 that will close this issue
@Lazar955 Lazar955 closed this as completed Dec 6, 2024
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 a pull request may close this issue.

1 participant