-
Notifications
You must be signed in to change notification settings - Fork 406
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
feat: one step test debugging #2804
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2804 +/- ##
===========================================
+ Coverage 76.26% 76.34% +0.07%
===========================================
Files 274 277 +3
Lines 10496 10611 +115
Branches 1197 1216 +19
===========================================
+ Hits 8005 8101 +96
- Misses 2152 2162 +10
- Partials 339 348 +9
Continue to review full report at Codecov.
|
|
||
export const LibraryCommandletExecutor = | ||
sfdxCoreExports.LibraryCommandletExecutor; | ||
export const notificationService = sfdxCoreExports.notificationService; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets use the new notification service and workspace context on salesforcedx-utils-vscode module https://github.com/forcedotcom/salesforcedx-vscode/tree/develop/packages/salesforcedx-utils-vscode/src/context & https://github.com/forcedotcom/salesforcedx-vscode/tree/develop/packages/salesforcedx-utils-vscode/src/commands
@@ -62,5 +62,12 @@ export const messages = { | |||
'Retrieved line breakpoint info from language server', | |||
language_client_not_ready: | |||
'Unable to retrieve breakpoint info from language server, language server is not ready', | |||
unable_to_retrieve_org_info: 'Unable to retrieve OrgInfo' | |||
unable_to_retrieve_org_info: 'Unable to retrieve OrgInfo', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @ekapner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lcampos @rcoringrato-sfdc - where does this message show up? Would like to rewrite it; potentially provide information to recover.
@@ -24,6 +24,9 @@ | |||
"Debuggers" | |||
], | |||
"dependencies": { | |||
"@salesforce/apex-node": "0.1.4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's also make sure to update the apex-node version to the one that includes the auth issue fix so we don't run into that while running tests here
What does this PR do?
This PR adds actions to test classes and test methods in the Apex test view that start a test run with trace flags enabled. The resultant debug log from the run is retrieved and a debug session started using the log. Subsequent PRs will add in breakpoint/checkpoint verification, and a codelens to allow initiation of the session from an apex test class context.
What issues does this PR fix or reference?
@W-7748704@