Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add support for Angular 16 #26052

Merged
merged 5 commits into from
Apr 24, 2023

Conversation

jordanpowell88
Copy link
Contributor

@jordanpowell88 jordanpowell88 commented Mar 8, 2023

Additional details

This PR adds support for Angular 16 (set to release in the coming weeks).

  • also adds tests to validate signal(s) in Angular will work in CT
  • moves the standalone.component.ts tests that were included in system-tests/project angular versions 14, 15, and now 16 into project-fixtures/angular.

Steps to test

  1. change ANGULAR_MAJOR_VERSIONS in system-tests/tests/component_testing_spec.ts to just test version 16
  2. Run yarn workspace @tooling/system-tests test component_testing_spec.ts

How has the user experience changed?

The user experience should remain the same

PR Tasks

@jordanpowell88 jordanpowell88 changed the title feat(angular): add support for Angular 16 chore(angular): add support for Angular 16 Mar 8, 2023
@jordanpowell88 jordanpowell88 changed the title chore(angular): add support for Angular 16 chore: add support for Angular 16 Mar 8, 2023
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/angular-16-support branch from cf9154b to 2f74753 Compare March 27, 2023 14:28
@jordanpowell88
Copy link
Contributor Author

angular/angular#46641

@jordanpowell88
Copy link
Contributor Author

We need to add support for output() for createSpy and autoOutputSpy

@jordanpowell88
Copy link
Contributor Author

@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/angular-16-support branch from 2f74753 to 8bb5cb8 Compare April 3, 2023 17:39
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/angular-16-support branch from 8bb5cb8 to baaf717 Compare April 11, 2023 15:13
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/angular-16-support branch from baaf717 to 598e429 Compare April 21, 2023 12:17
@jordanpowell88 jordanpowell88 marked this pull request as ready for review April 21, 2023 13:31
@jordanpowell88
Copy link
Contributor Author

We will need to create a follow up issue to validate/add support for esbuild support that ships with Angular 16

@cypress
Copy link

cypress bot commented Apr 21, 2023

Passing run #45757 ↗︎

0 214 19 0 Flakiness 0

Details:

remove .only, add angular-16 to webpack-dev-server e2e tests
Project: cypress Commit: de3d8f01cf
Status: Passed Duration: 10:26 💡
Started: Apr 24, 2023 12:58 AM Ended: Apr 24, 2023 1:09 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@astone123 astone123 requested a review from a team April 21, 2023 15:08
@@ -127,7 +127,7 @@ describe(`Angular CLI major versions`, () => {
spec = `${spec},!src/app/components/standalone.component.cy.ts`
}

systemTests.it(`v${majorVersion} with mount tests`, {
systemTests.it.only(`v${majorVersion} with mount tests`, {
Copy link
Contributor

Choose a reason for hiding this comment

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

Intentional .only?

@astone123
Copy link
Contributor

@jordanpowell88 should we add the angular-16 project to the webpack-dev-server e2e tests here (probably could also update that list to be WEBPACK_ANGULAR instead, looks like copy pasta)?

const WEBPACK_REACT: ProjectFixtureDir[] = ['angular-13', 'angular-14', 'angular-15']

@lmiller1990
Copy link
Contributor

CI green! Just need one more ✅ , thanks @astone123 for the drive by fixes.

@jordanpowell88 jordanpowell88 merged commit e2439b1 into develop Apr 24, 2023
@jordanpowell88 jordanpowell88 deleted the jordanpowell88/angular-16-support branch April 24, 2023 13:57
tgriesser added a commit that referenced this pull request May 3, 2023
* feat/protocol:
  refactor: migrate from windi to tailwind (#26516)
  chore: update v8 generation vars so that from scratch depends implies updating the metafile (#26472)
  chore: Update Vite to 4.3.0 (#26553)
  fix: unify cdp approach to fix devtools in electron (#26573)
  dependency(deps): update dependency deps-ok to v1.4.1 🌟 (#26612)
  chore: update 12.11.0 release date (#26587)
  chore: 12.11.0 release (#26582)
  chore: implement experimental ESM stub/spy for Vite (#26536)
  chore: try triggering mouseleave on buttons to ensure that tooltips aren't showing (#26524)
  chore: add support for Angular 16 (#26052)
  chore: upgrade Vue to 3.2.47 (#26555)
  chore: Update v8 snapshot cache (#26537)
  chore: add missing utm parameters for cloud links to Debug page (#26556)
  chore: update stalebot to respect new labels and up process rate (#26552)
  fix: don't display run passing status if Cloud org is over run limit (#26533)
  chore: update vm2 to 3.9.17 (#26534)
  feat: display a limit warning on the run navigation component when there are 100 total runs (#26523)
  chore: Update v8 snapshot cache (#26476)
  chore: upgrade vm2 (#26495)
  fix: Treat Video compression 0 as false.  (#26503)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Component Testing Support for Angular 16
5 participants