Schematics for Component spec should use test host component when created with `-cd OnPush`

Offen

#10,482 erstellt am 27.4.2018 2 Kommentare 5 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: angular, typescript
Bereich: testing, tooling

Rechercherichtung

Führe ng generate component test -cd OnPush aus und untersuche die generierten Component-Dateien und die .spec-Datei. Konzentriere dich dabei darauf, wie das Angular component schematic Tests einrichtet. Verwende die verlinkten Angular-Testhinweise als Kontext und überprüfe anschließend, dass der generierte Test das Muster der Test-Host-Komponente verwendet und dass wiederholte Aufrufe von fixture.detectChanges() die Change Detection auslösen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

area: @schematics/angular feature feature: insufficient votes

Versions

Angular CLI: 1.7.3
Node: 8.9.1
OS: win32 x64
Angular: 5.2.1
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, platform-server, router

@angular/cli: 1.7.3
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.3.2
@angular-devkit/schematics: 0.3.2
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.2
@schematics/angular: 0.3.2
@schematics/package-update: 0.3.2
typescript: 2.4.2
webpack: 3.11.0

Repro steps

ng generate component test -cd OnPush
Edit the test.component.html to be

{{ checkMe() }}

Add the checkMe method to test.component.ts:

checkMe() {
      console.log('change detection was invoked!');
}

Edit the generated .spec file to include the following test:

fit('runs change detection', () => {
    fixture.detectChanges();
    fixture.detectChanges();
    fixture.detectChanges();
});

Run unit test

Observed behavior

'change detection was invoked!' is logged only once (the initial check on creation of the component) rather than 4 times. That is to say, invoking fixture.detectChanges() does not actually run change detection. This is a known issue and is discussed in https://github.com/angular/angular/issues/12313

LOG: 'change detection was invoked!'
Chrome 65.0.3325 (Windows 10 0.0.0): Executed 1 of 478 SUCCESS (0 secs / 0.138 secs)

Desired behavior

As discussed in the thread linked above, the solution for testing OnPush components is to use the "test host component" pattern as described in the angular.io testing docs.

Therefore, the Angular CLI should use a schematic which uses the test host component when the -cd OnPush flag has been used. Otherwise we get a default test setup which is not fit for purpose (and will cause many wasted hours for devs like me 😓)

Vorherrschende Sprache: TypeScript
Sterne: 27k
Forks: 11.8k
Ø Merge: 17 Std. 25 Min.
Gemergte PRs (30 T.): 183

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus angular/angular-cli

Can't use an array of hostnames in --allowedHosts cli parameter in @angular/build:dev-server Offen

area: @angular/build gemini-triaged

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100

angular/angular-cli#33955 · Erstellt am 28.8.2026
Allow tests to pass without any test files (e.g. via Vitest `passWithNoTests`) Offen

area: @angular/cli gemini-triaged

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

angular/angular-cli#33055 · 1 Kommentar · 3 Reaktionen · Erstellt am 26.4.2026
library builder: tslib is not added to the output package.json Offen

angular/build:library area: @angular/build gemini-triaged

angular/angular-cli#34131 · 1 zugewiesene Person · Erstellt am 20.9.2026
library builder: a name re-exported through two "export *" paths is dropped from the bundle Offen

angular/build:library area: @angular/build gemini-triaged

angular/angular-cli#34130 · 1 zugewiesene Person · Erstellt am 20.9.2026
library builder: entry points a/b and a-b are built into the same bundle Offen

angular/build:library area: @angular/build gemini-triaged

angular/angular-cli#34128 · 1 zugewiesene Person · Erstellt am 19.9.2026

Alle Issues in angular/angular-cli