From 4b5b1f6d168e676521c3a29191e79a2b5fd8db2e Mon Sep 17 00:00:00 2001 From: Olivier Combe Date: Tue, 8 Jan 2019 14:47:30 +0100 Subject: [PATCH] test(ivy): enable i18n test for nested templates (#27984) The bug that was causing this test to fail has been resolved (see FW-903). The test now runs correctly. PR Close #27984 --- packages/core/test/i18n_integration_spec.ts | 31 ++++++++++----------- 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/packages/core/test/i18n_integration_spec.ts b/packages/core/test/i18n_integration_spec.ts index a6087e252b..36c41b49ab 100644 --- a/packages/core/test/i18n_integration_spec.ts +++ b/packages/core/test/i18n_integration_spec.ts @@ -99,22 +99,21 @@ onlyInIvy('Ivy i18n logic').describe('i18n', function() { expect(element.title).toBe('Bonjour John'); }); - fixmeIvy('FW-903: i18n attributes in nested templates throws at runtime') - .it('should correctly bind to context in nested template', () => { - const title = 'Item {{ id }}'; - const template = ` -
-
-
- `; - const fixture = getFixtureWithOverrides({template}); + it('should correctly bind to context in nested template', () => { + const title = 'Item {{ id }}'; + const template = ` +
+
+
+ `; + const fixture = getFixtureWithOverrides({template}); - const element = fixture.nativeElement; - for (let i = 0; i < element.children.length; i++) { - const child = element.children[i]; - expect((child as any).innerHTML).toBe(`
`); - } - }); + const element = fixture.nativeElement; + for (let i = 0; i < element.children.length; i++) { + const child = element.children[i]; + expect((child as any).innerHTML).toBe(`
`); + } + }); fixmeIvy('FW-904: i18n attributes placed on i18n root node don\'t work') .it('should work correctly when placed on i18n root node', () => { @@ -476,4 +475,4 @@ onlyInIvy('Ivy i18n logic').describe('i18n', function() { expect(element.innerHTML).toBe('vingt'); }); }); -}); \ No newline at end of file +});