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

fix: Repair broken test on Node 20 #1193

Merged
merged 2 commits into from
Feb 1, 2024
Merged

Conversation

mctavish
Copy link
Contributor

Tests are currently broken on Node 20 due to a change in behaviour. A field that we expected to have present in breakpoints is no longer set. This fix removes that check in order to get the test to pass again; it is not currently worth the effort to investigate the cause of the change as it has no meaningful effect on the debug agent.

@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. labels Jan 30, 2024
Copy link

codecov bot commented Jan 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8bb6588) 61.72% compared to head (df7fcd7) 61.72%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1193   +/-   ##
=======================================
  Coverage   61.72%   61.72%           
=======================================
  Files          17       17           
  Lines        1309     1309           
  Branches      257      257           
=======================================
  Hits          808      808           
  Misses        447      447           
  Partials       54       54           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mctavish mctavish added kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Feb 1, 2024
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Feb 1, 2024
@mctavish mctavish added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 1, 2024
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Feb 1, 2024
@mctavish mctavish marked this pull request as ready for review February 1, 2024 16:45
@mctavish mctavish requested review from a team as code owners February 1, 2024 16:45
@mctavish mctavish requested a review from haroonc February 1, 2024 16:45
@mctavish mctavish merged commit a600ea8 into googleapis:main Feb 1, 2024
17 checks passed
@mctavish mctavish deleted the fix-build-20 branch February 1, 2024 16:47
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. size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants