Skip to content
Permalink
Browse files

fix(compiler): fix Elements not making a new ParseSourceSpan (#31190)

Change the Element constructor in r3_ast to create a new ParseSourceSpan when regenerating it rather than extending an object, which does not contain the overloaded toString().

PR Close #31190
  • Loading branch information...
ayazhafiz authored and kara committed Jun 21, 2019
1 parent 9c06af2 commit 7035f225ad8d5a1fff018b5ab2d60952a4f40885
@@ -72,7 +72,7 @@ export class Element implements Node {
public endSourceSpan: ParseSourceSpan|null, public i18n?: I18nAST) {
// If the element is empty then the source span should include any closing tag
if (children.length === 0 && startSourceSpan && endSourceSpan) {
this.sourceSpan = {...sourceSpan, end: endSourceSpan.end};
this.sourceSpan = new ParseSourceSpan(sourceSpan.start, endSourceSpan.end);
}
}
visit<Result>(visitor: Visitor<Result>): Result { return visitor.visitElement(this); }
@@ -95,7 +95,17 @@ function expectFromR3Nodes(nodes: t.Node[]) {
return expect(humanizer.result);
}

function expectSpanFromHtml(html: string) {
const {nodes} = parse(html);
return expect(nodes[0] !.sourceSpan.toString());
}

describe('R3 template transform', () => {
describe('ParseSpan on nodes toString', () => {
it('should create valid text span on Element with adjacent start and end tags',
() => { expectSpanFromHtml('<div></div>').toBe('<div></div>'); });
});

describe('Nodes without binding', () => {
it('should parse text nodes', () => {
expectFromHtml('a').toEqual([

0 comments on commit 7035f22

Please sign in to comment.
You can’t perform that action at this time.