test: Add fixture for defineProperty call with recursive property access

This currently causes a Maximum call stack size exceeded exception within TypeScript's checker#typeToString method.
SAP · Apr 17, 2024 · 878bb51 · 878bb51
1 parent 508d81d
commit 878bb51
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 0 deletions.
diff --git a/test/fixtures/linter/general/RecursiveDefineProperty.js b/test/fixtures/linter/general/RecursiveDefineProperty.js
@@ -0,0 +1,4 @@
+// See https://github.com/SAP/ui5-linter/issues/75
+Object.defineProperty(globalThis, "myProp", {
+	value: window.location.search
+});
diff --git a/test/lib/linter/snapshots/linter.ts.md b/test/lib/linter/snapshots/linter.ts.md
@@ -203,6 +203,28 @@ Generated by [AVA](https://avajs.dev).
       },
     ]
 
+## General: RecursiveDefineProperty.js
+
+> Snapshot 1
+
+    [
+      {
+        coverageInfo: [],
+        errorCount: 1,
+        fatalErrorCount: 1,
+        filePath: 'RecursiveDefineProperty.js',
+        messages: [
+          {
+            fatal: true,
+            message: 'Maximum call stack size exceeded',
+            ruleId: 'ui5-linter-parsing-error',
+            severity: 2,
+          },
+        ],
+        warningCount: 0,
+      },
+    ]
+
 ## General: TSLike.js
 
 > Snapshot 1

diff --git a/test/lib/linter/snapshots/linter.ts.snap b/test/lib/linter/snapshots/linter.ts.snap