Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented May 6, 2022

A new E2E test has been added that updates a newly generated application to use a standalone component that
is bootstrapped with the newly introduced bootstrapApplication API. This test is intended to check that
the minimal functionality for a standalone-based application functions with the Angular CLI. More expansive
tests will be added as standalone features and capabilities are introduced within the Angular CLI.

Closes #23044

@clydin clydin added the target: rc This PR is targeted for the next release-candidate label May 6, 2022
@clydin clydin added this to the v14 milestone May 6, 2022
@clydin clydin force-pushed the e2e/standalone-test branch from bd816c8 to 9141c84 Compare May 6, 2022 22:22
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 6, 2022
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

Not familiar with the CLI testing infrastructure but the scenario / test itself LGTM

dgp1130
dgp1130 previously approved these changes May 10, 2022
await ng('build');

// Perform the default E2E tests
await ng('e2e', 'test-project');
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should also consider adding a unit test for standalone and verifying that that works, but we can follow up with and expand this test later, so not a blocking issue for this PR.

A new E2E test has been added that updates a newly generated application to use a standalone component that
is bootstrapped with the newly introduced `bootstrapApplication` API. This test is intended to check that
the minimal functionality for a standalone-based application functions with the Angular CLI. More expansive
tests will be added as standalone features and capabilities are introduced within the Angular CLI.
@clydin clydin dismissed stale reviews from dgp1130 and pkozlowski-opensource via c2c137f May 10, 2022 18:22
@clydin clydin force-pushed the e2e/standalone-test branch from 9141c84 to c2c137f Compare May 10, 2022 18:22
@clydin clydin requested a review from dgp1130 May 10, 2022 18:24
@clydin clydin 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 May 10, 2022
@dgp1130 dgp1130 merged commit d51e63b into angular:main May 10, 2022
@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 Jun 10, 2022
@clydin clydin deleted the e2e/standalone-test branch September 21, 2022 14:03
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.

Standalone components end-to-end test
3 participants