noir icon indicating copy to clipboard operation
noir copied to clipboard

chore(test): add panicking tests for 'defunctionalize'

Open michaeljklein opened this issue 6 months ago • 3 comments

Description

Problem*

Resolves https://github.com/noir-lang/noir/issues/8553

Summary*

This PR contains defunctionalize SSA pass tests that I found using ssa_afl_fuzzer

Additional Context

Documentation*

Check one:

  • [x] No documentation needed.
  • [ ] Documentation included in this PR.
  • [ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • [x] I have tested the changes locally.
  • [x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

michaeljklein avatar May 14 '25 14:05 michaeljklein

@michaeljklein Do we have an issue for this? I'm good to merge this once we have one + a link to it.

TomAFrench avatar May 15 '25 12:05 TomAFrench

@TomAFrench I added one, but still need to add the other tests and update docs

michaeljklein avatar May 16 '25 15:05 michaeljklein

Following up on this. @michaeljklein can you add any necessary details surrounding these tests when you're back tomorrow?

TomAFrench avatar May 27 '25 16:05 TomAFrench