Skip to content

Conversation

alan-agius4
Copy link
Collaborator

remapping will return a SourceMap which it's prototype is not a simple object. This causes Babel to fail when it invoked from istanbul-lib-instrument because Babel with don't know how to turn this value into a node error as Babel will not process not simple objects.

See: https://github.com/babel/babel/blob/780aa48d2a34dc55f556843074b6aed45e7eabeb/packages/babel-types/src/converters/valueToNode.ts#L115-L130

Closes #21967

@google-cla google-cla bot added the cla: yes label Oct 19, 2021
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Oct 19, 2021
… a plain object

`remapping` will return a SourceMap which it's prototype is not a simple object. This causes Babel to fail when it invoked from `istanbul-lib-instrument` because Babel with `don't know how to turn this value into a node` error as Babel will not process not simple objects.

See: https://github.com/babel/babel/blob/780aa48d2a34dc55f556843074b6aed45e7eabeb/packages/babel-types/src/converters/valueToNode.ts#L115-L130

Closes #21967
@alan-agius4 alan-agius4 requested a review from clydin October 19, 2021 11:01
@clydin clydin added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release labels Oct 19, 2021
@clydin clydin merged commit da1733c into angular:master Oct 19, 2021
@alan-agius4 alan-agius4 deleted the mapping-karma-code-coverage branch October 19, 2021 13:34
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Nov 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tests with code coverage fail when using Angular 12.2.10
2 participants