Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

fix: surface correct error message for ambiguous sourcemap matches #971

Merged
merged 4 commits into from
Jun 14, 2021

Conversation

Louis-Ye
Copy link
Contributor

@Louis-Ye Louis-Ye commented Jun 9, 2021

Fixes #961

@Louis-Ye Louis-Ye requested review from a team as code owners June 9, 2021 21:08
@product-auto-label product-auto-label bot added the api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. label Jun 9, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 9, 2021
@codecov
Copy link

codecov bot commented Jun 9, 2021

Codecov Report

Merging #971 (521ca59) into master (87b9569) will increase coverage by 0.04%.
The diff coverage is 73.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #971      +/-   ##
==========================================
+ Coverage   66.50%   66.54%   +0.04%     
==========================================
  Files          20       20              
  Lines        1645     1656      +11     
  Branches      339      340       +1     
==========================================
+ Hits         1094     1102       +8     
- Misses        473      475       +2     
- Partials       78       79       +1     
Impacted Files Coverage Δ
src/agent/v8/legacy-debugapi.ts 0.00% <0.00%> (ø)
src/agent/v8/inspector-debugapi.ts 84.95% <80.00%> (-0.37%) ⬇️
src/agent/io/sourcemapper.ts 86.07% <100.00%> (+2.95%) ⬆️
src/agent/debuglet.ts 80.64% <0.00%> (-0.50%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87b9569...521ca59. Read the comment docs.

test/test-sourcemapper.ts Outdated Show resolved Hide resolved
@@ -26,12 +26,25 @@ const readFilep = promisify(fs.readFile);

/** @define {string} */ const MAP_EXT = '.map';

/** Represents one source map file. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you so much for this documentation!

@Louis-Ye Louis-Ye merged commit d5abfac into googleapis:master Jun 14, 2021
gcf-merge-on-green bot pushed a commit that referenced this pull request Jun 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error message is confusing when the breakpoint path matches multiple files
3 participants