From a3ee08968eb3af42a2193eae59a9f025027c27c1 Mon Sep 17 00:00:00 2001 From: Andrew Kushnir Date: Tue, 4 Dec 2018 20:54:42 -0800 Subject: [PATCH] fix(ivy): compiler compliance tests actualized (#27473) PR #27404 introduced additional test case to make sure we generate `elementStyling` instructions with proper set of arguments (first argument was missing in some cases). It looks like that PR was created before we updated host vars count calculation and the `allocHostVars` becomes unnecessary in the test cases introduced in PR #27404. This commit actualizes this test to get rid of unnecessary `allocHostVars` instruction. PR Close #27473 --- .../compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts | 2 +- .../test/compliance/r3_view_compiler_styling_spec.ts | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/packages/compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts b/packages/compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts index 04495bd4e5..2873ba1207 100644 --- a/packages/compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts +++ b/packages/compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts @@ -2012,7 +2012,7 @@ describe('i18n support in the view compiler', () => { const exceptions = { '3052001905251380936': 'Wrapper message generated by "ng xi18n" around ICU: " {$ICU} "' }; - verify(input, output, {exceptions, verbose: true}); + verify(input, output, {exceptions}); }); it('should handle icus in different contexts', () => { diff --git a/packages/compiler-cli/test/compliance/r3_view_compiler_styling_spec.ts b/packages/compiler-cli/test/compliance/r3_view_compiler_styling_spec.ts index 26df9d06de..36de858f73 100644 --- a/packages/compiler-cli/test/compliance/r3_view_compiler_styling_spec.ts +++ b/packages/compiler-cli/test/compliance/r3_view_compiler_styling_spec.ts @@ -906,7 +906,6 @@ describe('compiler compliance: styling', () => { … function ClassDirective_HostBindings(rf, ctx, elIndex) { if (rf & 1) { - $r3$.ɵallocHostVars(1); $r3$.ɵelementStyling(null, null, null, ctx); } if (rf & 2) {