Skip to content

test: fix and re-enable partial compliance linking tests#51620

Closed
devversion wants to merge 2 commits intoangular:mainfrom
devversion:fix-reeenable-linking-compliance-tests
Closed

test: fix and re-enable partial compliance linking tests#51620
devversion wants to merge 2 commits intoangular:mainfrom
devversion:fix-reeenable-linking-compliance-tests

Conversation

@devversion
Copy link
Member

@devversion devversion commented Sep 1, 2023

Fixes and re-enables the partial compliance tests that weren't running for a while after being disabled in:
#49914

There are two failurres-sounds like the lazy dependency functions are not properly generated/tracked with partial compilation output

Fixes and re-enables the partial compliance tests that weren't
running for a while after being disabled in:
angular#49914
@devversion devversion force-pushed the fix-reeenable-linking-compliance-tests branch from 4dd61df to 1a24a5d Compare September 1, 2023 13:48
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

The linker tests weren't running in a while-see previous commit. Due to
this the recently introduced defer compliance tests are not fully
compatible with linking. This commit disables two of these tests-
all others work.
@devversion
Copy link
Member Author

There are many source map failures: https://gist.github.com/devversion/993e2c0eee657629185dee91365bd7a2

@devversion
Copy link
Member Author

Closing as there is a tracking issue and I will enable these tests without mapper assertions in the signal inputs PR

@devversion devversion closed this Dec 17, 2023
@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 Jan 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants