fix(949): fix unmatched closing tags in JSX with referenced identifiers (#1056)

Author: a-tarasyuk

internal/transformers/utilities.go

@@ -120,6 +120,8 @@ func isIdentifierReference(name *ast.IdentifierNode, parent *ast.Node) bool {
 		return parent.AsImportAttribute().Value == name
 	case ast.KindJsxOpeningElement:
 		return parent.AsJsxOpeningElement().TagName == name
+	case ast.KindJsxClosingElement:
+		return parent.AsJsxClosingElement().TagName == name
 	default:
 		return false
 	}

testdata/baselines/reference/submodule/conformance/inlineJsxFactoryDeclarations.js

@@ -49,7 +49,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.prerendered = void 0;
 /** @jsx h */
 const renderer_1 = require("./renderer");
-exports.prerendered = <renderer_1.dom></h>;
+exports.prerendered = <renderer_1.dom></renderer_1.dom>;
 //// [othernoalias.js]
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });

testdata/baselines/reference/submodule/conformance/inlineJsxFactoryDeclarations.js.diff

@@ -16,7 +16,7 @@
 -var renderer_1 = require("./renderer");
 -exports.prerendered = (0, renderer_1.dom)("h", null);
 +const renderer_1 = require("./renderer");
-+exports.prerendered = <renderer_1.dom></h>;
++exports.prerendered = <renderer_1.dom></renderer_1.dom>;
  //// [othernoalias.js]
  "use strict";
  Object.defineProperty(exports, "__esModule", { value: true });

testdata/baselines/reference/submodule/conformance/inlineJsxFactoryDeclarationsLocalTypes.js

@@ -109,7 +109,7 @@ class MyClass {
     }
 }
 exports.MyClass = MyClass;
-exports.tree = <exports.MySFC x={1} y={2}><MyClass x={3} y={4}/><MyClass x={5} y={6}/></MySFC>;
+exports.tree = <exports.MySFC x={1} y={2}><MyClass x={3} y={4}/><MyClass x={5} y={6}/></exports.MySFC>;
 exports.default = <h></h>;
 //// [index.js]
 "use strict";
@@ -133,6 +133,6 @@ class DOMClass {
 // Should work, everything is a DOM element
 const _tree = <DOMSFC x={1} y={2}><DOMClass x={3} y={4}/><DOMClass x={5} y={6}/></DOMSFC>;
 // Should fail, no dom elements
-const _brokenTree = <component_1.MySFC x={1} y={2}><component_1.MyClass x={3} y={4}/><component_1.MyClass x={5} y={6}/></MySFC>;
+const _brokenTree = <component_1.MySFC x={1} y={2}><component_1.MyClass x={3} y={4}/><component_1.MyClass x={5} y={6}/></component_1.MySFC>;
 // Should fail, nondom isn't allowed as children of dom
 const _brokenTree2 = <DOMSFC x={1} y={2}>{component_1.tree}{component_1.tree}</DOMSFC>;

testdata/baselines/reference/submodule/conformance/inlineJsxFactoryDeclarationsLocalTypes.js.diff

@@ -55,7 +55,7 @@
 -    (0, renderer2_1.predom)(MyClass, { x: 3, y: 4 }),
 -    (0, renderer2_1.predom)(MyClass, { x: 5, y: 6 }));
 -exports.default = (0, renderer2_1.predom)("h", null);
-+exports.tree = <exports.MySFC x={1} y={2}><MyClass x={3} y={4}/><MyClass x={5} y={6}/></MySFC>;
++exports.tree = <exports.MySFC x={1} y={2}><MyClass x={3} y={4}/><MyClass x={5} y={6}/></exports.MySFC>;
 +exports.default = <h></h>;
  //// [index.js]
  "use strict";
@@ -116,7 +116,7 @@
 -var _brokenTree = (0, renderer_1.dom)(component_1.MySFC, { x: 1, y: 2 },
 -    (0, renderer_1.dom)(component_1.MyClass, { x: 3, y: 4 }),
 -    (0, renderer_1.dom)(component_1.MyClass, { x: 5, y: 6 }));
-+const _brokenTree = <component_1.MySFC x={1} y={2}><component_1.MyClass x={3} y={4}/><component_1.MyClass x={5} y={6}/></MySFC>;
++const _brokenTree = <component_1.MySFC x={1} y={2}><component_1.MyClass x={3} y={4}/><component_1.MyClass x={5} y={6}/></component_1.MySFC>;
  // Should fail, nondom isn't allowed as children of dom
 -var _brokenTree2 = (0, renderer_1.dom)(DOMSFC, { x: 1, y: 2 },
 -    component_1.tree,