Skip to content

feat(codelens): trigger debugger on 'Debug test' command (#3)#100

Merged
asterite merged 1 commit intonoir-lang:masterfrom
manastech:feat/5208-debug-test-codelens
Apr 22, 2025
Merged

feat(codelens): trigger debugger on 'Debug test' command (#3)#100
asterite merged 1 commit intonoir-lang:masterfrom
manastech:feat/5208-debug-test-codelens

Conversation

@anaPerezGhiglia
Copy link
Copy Markdown
Contributor

Description

  • trigger dap debugger when receiving nargo.debug.test command
  • add new testName and oracleResolver fields to launch configuration
dapDebugger.mov

Problem*

related to noir-lang/noir#5208

PR Checklist*

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

* trigger `startDebugging` when receiving 'nargo.debug.test' command
anaPerezGhiglia added a commit to manastech/noir that referenced this pull request Mar 25, 2025
 - Add the possibility to compile and debug a test function
 - Add a new config parameter to configure an external oracle resolver when debugging
 - REPL: execute the debugger in a separate thread than the repl cli
 - Add new `Debug test` codelens request. Related vscode-noir PR (noir-lang/vscode-noir#100)

---------

Co-authored-by: Martin Verzilli <martin.verzilli@gmail.com>
@asterite
Copy link
Copy Markdown
Contributor

asterite commented Apr 8, 2025

Thank you! I tried this locally and I don't see the "Debug test" thing. Do I need to configure something?

@anaPerezGhiglia
Copy link
Copy Markdown
Contributor Author

The codelens Debug test is displayed by the lsp, so this won't ve visible until these changes in noir's code_lens_request.rs are merged (see noir-lang/noir#7958)

Copy link
Copy Markdown
Contributor

@asterite asterite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you!

@asterite asterite merged commit fca148c into noir-lang:master Apr 22, 2025
4 checks passed
@github-actions github-actions bot mentioned this pull request Apr 22, 2025
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.

2 participants