cargdev/angular
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(compiler-cli): make typescript_ast_factory_spec tests resilient to line-endings (#38925)

Browse Source 
The tests were assuming that newlines were `\n` characters but this is not
the case on Windows.

PR Close #38925
This commit is contained in:
Pete Bacon Darwin 2020-09-21 21:51:40 +01:00 committed by Misko Hevery
parent 297b123151
commit 3082f7378b
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
packages/compiler-cli/src/ngtsc/translator/test/typescript_ast_factory_spec.ts
View File

@ -6,6 +6,7 @@
* found in the LICENSE file at https://angular.io/license * found in the LICENSE file at https://angular.io/license
*/ */
import {leadingComment} from '@angular/compiler'; import {leadingComment} from '@angular/compiler';
import {EOL} from 'os';
import * as ts from 'typescript'; import * as ts from 'typescript';
import {TypeScriptAstFactory} from '../src/typescript_ast_factory'; import {TypeScriptAstFactory} from '../src/typescript_ast_factory';
@ -24,7 +25,7 @@ describe('TypeScriptAstFactory', () => {
'/* comment 1 */', '/* comment 1 */',
'//comment 2', '//comment 2',
'x = 10;', 'x = 10;',
].join('\n')); ].join(EOL));
}); });
}); });
@ -61,7 +62,7 @@ describe('TypeScriptAstFactory', () => {
' x = 10;', ' x = 10;',
' y = 20;', ' y = 20;',
'}', '}',
].join('\n')); ].join(EOL));
}); });
}); });
@ -152,7 +153,7 @@ describe('TypeScriptAstFactory', () => {
' x = 10;', ' x = 10;',
'else', 'else',
' x = 42;', ' x = 42;',
].join('\n')); ].join(EOL));
}); });
it('should create an if statement if the else expression is null', () => { it('should create an if statement if the else expression is null', () => {
@ -162,7 +163,7 @@ describe('TypeScriptAstFactory', () => {
expect(generate(ifStmt)).toEqual([ expect(generate(ifStmt)).toEqual([
'if (!test)', 'if (!test)',
' x = 10;', ' x = 10;',
].join('\n')); ].join(EOL));
}); });
}); });