fix(processing/script/transformers/location): handle class property d…

…efinitions (#3033)

The transformers generates wrong (and invalid) JavaScript if a class contains a
property named `locations`.

The generated code currently is `class C{ __get$Loc(location) }`.

src/processing/script/transformers/location-get.ts

@@ -68,6 +68,11 @@ const transformer: Transformer<Identifier> = {
         if (parent.type === Syntax.MethodDefinition)
             return false;
 
+        // Skip: class X { location }
+        // @ts-ignore
+        if (parent.type === Syntax.PropertyDefinition)
+            return false;
+
         // Skip: class location { x () {} }
         if (parent.type === Syntax.ClassDeclaration)
             return false;

test/server/script-processor-test.js

@@ -288,6 +288,11 @@ describe('Script processor', () => {
                           '? function(){var _hh$temp0 = newLocation; return __set$Loc(location,_hh$temp0)||(location=_hh$temp0);}.call(this)' +
                           ': function(){var _hh$temp1 = "#123"; return __set$Loc(location,_hh$temp1)||(location=_hh$temp1);}.call(this);',
             },
+            {
+                src: 'class C { location }',
+
+                expected: 'class C { location }',
+            },
             {
                 src: 'if (location) { location = newLocation; } else location = "#123";',