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.

Sign up for GitHub

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

Jump to bottom

feat: add support for angular 17 #28152

Merged
merged 11 commits into from
Nov 7, 2023
Merged

feat: add support for angular 17 #28152

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
bd44311
feat(webpack-dev-server): add support for angular 17
jordanpowell88 Oct 26, 2023
d1279e6
update changelog
jordanpowell88 Oct 26, 2023
1944869
fix broken spec pattern
jordanpowell88 Oct 26, 2023
f5b0c94
update to rc 2 for cli
jordanpowell88 Oct 26, 2023
9e7d20a
remove = from dependency minVersion lists
jordanpowell88 Nov 1, 2023
b1044e1
Merge branch 'develop' into jordanpowell88/angular-17
jordanpowell88 Nov 1, 2023
a27fd7b
update angular 17 rc version and update dep test
jordanpowell88 Nov 1, 2023
2630c7c
add projectDirFolder to fixtures for angular-17
jordanpowell88 Nov 1, 2023
6cb89d3
resolve broken system tests
jordanpowell88 Nov 3, 2023
d9e0b46
Merge branch 'develop' into jordanpowell88/angular-17
jordanpowell88 Nov 3, 2023
5ac72a5
update ct project dir angular version
jordanpowell88 Nov 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 4 additions & 0 deletions cli/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@

_Released 11/7/2023 (PENDING)_

**Features:**

- Adds Component Testing support for Angular version 17. Addresses [#28153](https://github.com/cypress-io/cypress/issues/28153).

Comment on lines +6 to +9
Copy link
Contributor

Choose a reason for hiding this comment

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

Since this is a feature we should bump the next version number to 13.5.0, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yup

Copy link
Member

Choose a reason for hiding this comment

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

This check should have ensured this was bumped before merging...

**Bugfixes:**

- Fixed a regression in [`13.3.2`](https://docs.cypress.io/guides/references/changelog/13.3.2) where loading a service worker and immediately reloading the page can cause a crash. Fixes [#28141](https://github.com/cypress-io/cypress/issues/28141).
Expand Down
2 changes: 1 addition & 1 deletion npm/webpack-dev-server/cypress/e2e/angular.cy.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
/// <reference path="../support/e2e.ts" />
import type { ProjectFixtureDir } from '@tooling/system-tests/lib/fixtureDirs'

const WEBPACK_ANGULAR: ProjectFixtureDir[] = ['angular-13', 'angular-14', 'angular-15', 'angular-16', 'angular-16.1']
const WEBPACK_ANGULAR: ProjectFixtureDir[] = ['angular-13', 'angular-14', 'angular-15', 'angular-16', 'angular-16.1', 'angular-17']

// Add to this list to focus on a particular permutation
const ONLY_PROJECTS: ProjectFixtureDir[] = []
Expand Down
10 changes: 5 additions & 5 deletions packages/scaffold-config/src/dependencies.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ export const WIZARD_DEPENDENCY_ANGULAR_CLI = {
package: '@angular/cli',
installer: '@angular/cli',
description: 'CLI tool that you use to initialize, develop, scaffold, and maintain Angular applications.',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0 || ^=17.0.0',
jordanpowell88 marked this conversation as resolved.
Show resolved Hide resolved
} as const

export const WIZARD_DEPENDENCY_ANGULAR_DEVKIT_BUILD_ANGULAR = {
Expand All @@ -112,7 +112,7 @@ export const WIZARD_DEPENDENCY_ANGULAR_DEVKIT_BUILD_ANGULAR = {
package: '@angular-devkit/build-angular',
installer: '@angular-devkit/build-angular',
description: 'Angular Webpack build facade',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0 || ^=17.0.0',
jordanpowell88 marked this conversation as resolved.
Show resolved Hide resolved
} as const

export const WIZARD_DEPENDENCY_ANGULAR_CORE = {
Expand All @@ -121,7 +121,7 @@ export const WIZARD_DEPENDENCY_ANGULAR_CORE = {
package: '@angular/core',
installer: '@angular/core',
description: 'The core of the Angular framework',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0 || ^=17.0.0',
jordanpowell88 marked this conversation as resolved.
Show resolved Hide resolved
} as const

export const WIZARD_DEPENDENCY_ANGULAR_COMMON = {
Expand All @@ -130,7 +130,7 @@ export const WIZARD_DEPENDENCY_ANGULAR_COMMON = {
package: '@angular/common',
installer: '@angular/common',
description: 'Commonly needed Angular directives and services',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0 || ^=17.0.0',
jordanpowell88 marked this conversation as resolved.
Show resolved Hide resolved
} as const

export const WIZARD_DEPENDENCY_ANGULAR_PLATFORM_BROWSER_DYNAMIC = {
Expand All @@ -139,7 +139,7 @@ export const WIZARD_DEPENDENCY_ANGULAR_PLATFORM_BROWSER_DYNAMIC = {
package: '@angular/platform-browser-dynamic',
installer: '@angular/platform-browser-dynamic',
description: 'Library for using Angular in a web browser with JIT compilation',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0',
minVersion: '^=13.0.0 || ^=14.0.0 || ^=15.0.0 || ^=16.0.0 || ^=17.0.0',
jordanpowell88 marked this conversation as resolved.
Show resolved Hide resolved
} as const

export const WIZARD_DEPENDENCY_SVELTE: Cypress.CypressComponentDependency = {
Expand Down
0 ...rc/app/components/signals.component.cy.ts → ...rc/app/components/signals.component.cy.ts
View file
Open in desktop
File renamed without changes.
0 ...1/src/app/components/signals.component.ts → ...r/src/app/components/signals.component.ts
View file
Open in desktop
File renamed without changes.
14 changes: 0 additions & 14 deletions system-tests/projects/angular-16/src/app/components/signals.component.cy.ts
View file
Open in desktop

This file was deleted.

14 changes: 0 additions & 14 deletions system-tests/projects/angular-16/src/app/components/signals.component.ts
View file
Open in desktop

This file was deleted.

42 changes: 42 additions & 0 deletions system-tests/projects/angular-17/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
{
"name": "angular-17",
"version": "0.0.0",
"private": true,
"scripts": {
"ng": "ng",
"start": "ng serve",
"build": "ng build",
"watch": "ng build --watch --configuration development",
"test": "ng test"
},
"dependencies": {
"@angular/animations": "^17.0.0-rc.0",
"@angular/common": "^17.0.0-rc.0",
"@angular/compiler": "^17.0.0-rc.0",
"@angular/core": "^17.0.0-rc.0",
"@angular/forms": "^17.0.0-rc.0",
"@angular/platform-browser": "^17.0.0-rc.0",
"@angular/platform-browser-dynamic": "^17.0.0-rc.0",
"@angular/router": "^17.0.0-rc.0",
"@angular/ssr": "^17.0.0-rc.2",
"rxjs": "~7.8.0",
"tslib": "^2.3.0",
"zone.js": "~0.14.0"
},
"devDependencies": {
"@angular-devkit/build-angular": "^17.0.0-rc.2",
"@angular/cli": "^17.0.0-rc.2",
"@angular/compiler-cli": "^17.0.0-rc.0",
"@types/jasmine": "~4.3.0",
"cypress": "^13.3.2",
"jasmine-core": "~4.6.0",
"jest": "^29.7.0",
"jest-environment-jsdom": "^29.7.0",
"karma": "~6.4.0",
"karma-chrome-launcher": "~3.2.0",
"karma-coverage": "~2.2.0",
"karma-jasmine": "~5.1.0",
"karma-jasmine-html-reporter": "~2.1.0",
"typescript": "~5.2.2"
}
}