Skip to content

test(compiler-cli): add unit tests for output() JIT transform#54841

Closed
devversion wants to merge 1 commit intoangular:mainfrom
devversion:output-follow-ups
Closed

test(compiler-cli): add unit tests for output() JIT transform#54841
devversion wants to merge 1 commit intoangular:mainfrom
devversion:output-follow-ups

Conversation

@devversion
Copy link
Member

We are already testing the JIT transforms via integration tests and acceptance authoring tests, but
this commit adds dedicated unit tests for the transform behavior for proper test coverage (it was a planned follow-up).

@devversion devversion added action: review The PR is still awaiting reviews from at least one requested reviewer area: core Issues related to the framework runtime target: rc This PR is targeted for the next release-candidate labels Mar 13, 2024
@devversion devversion requested a review from crisbeto March 13, 2024 15:44
@ngbot ngbot bot added this to the Backlog milestone Mar 13, 2024
@devversion devversion changed the title test(compiler-cli): add unit tests for output() JIT transform tests test(compiler-cli): add unit tests for output() JIT transform Mar 13, 2024
We are already testing the JIT transforms via integration tests, but
this commit adds dedicated unit tests for the transform behavior for
proper test coverage (planned follow-up).
@devversion devversion added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 13, 2024
@alxhub alxhub added target: major This PR is targeted for the next major release target: patch This PR is targeted for the next patch release and removed target: rc This PR is targeted for the next release-candidate target: major This PR is targeted for the next major release labels Mar 15, 2024
@alxhub
Copy link
Member

alxhub commented Mar 15, 2024

This PR was merged into the repository by commit 5a63a47.

alxhub pushed a commit that referenced this pull request Mar 15, 2024
We are already testing the JIT transforms via integration tests, but
this commit adds dedicated unit tests for the transform behavior for
proper test coverage (planned follow-up).

PR Close #54841
@alxhub alxhub closed this in 5a63a47 Mar 15, 2024
@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 Apr 15, 2024
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 area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants