Skip to content

Commit a3ee089

Browse files
AndrewKushnirIgorMinar
authored andcommitted
fix(ivy): compiler compliance tests actualized (angular#27473)
PR angular#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 angular#27404. This commit actualizes this test to get rid of unnecessary `allocHostVars` instruction. PR Close angular#27473
1 parent b6dbcf9 commit a3ee089

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

packages/compiler-cli/test/compliance/r3_view_compiler_i18n_spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2012,7 +2012,7 @@ describe('i18n support in the view compiler', () => {
20122012
const exceptions = {
20132013
'3052001905251380936': 'Wrapper message generated by "ng xi18n" around ICU: " {$ICU} "'
20142014
};
2015-
verify(input, output, {exceptions, verbose: true});
2015+
verify(input, output, {exceptions});
20162016
});
20172017

20182018
it('should handle icus in different contexts', () => {

packages/compiler-cli/test/compliance/r3_view_compiler_styling_spec.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -906,7 +906,6 @@ describe('compiler compliance: styling', () => {
906906
907907
function ClassDirective_HostBindings(rf, ctx, elIndex) {
908908
if (rf & 1) {
909-
$r3$.ɵallocHostVars(1);
910909
$r3$.ɵelementStyling(null, null, null, ctx);
911910
}
912911
if (rf & 2) {

0 commit comments

Comments
 (0)