Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Better test for does_not_panic #193

Closed
1 task done
phated opened this issue May 17, 2023 · 1 comment
Closed
1 task done

Better test for does_not_panic #193

phated opened this issue May 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@phated
Copy link
Contributor

phated commented May 17, 2023

Problem

In #191, @TomAFrench noted that the does_not_panic test isn't really testing what we want to be testing.

Proposed solution

Write a better test.

Alternatives considered

No response

Additional context

No response

Submission Checklist

  • Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.
@phated phated added the enhancement New feature or request label May 17, 2023
@kobyhallx
Copy link
Contributor

Won't do since acvm-backend-barretenberg will be decomissioned.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
Noir
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@phated @kobyhallx