JS: Add test case showing problem with chains going through internal …

…classes
github · Apr 5, 2024 · 200c431 · 200c431
1 parent acee521
commit 200c431
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/javascript/ql/test/library-tests/ModelGeneration/ModelGeneration.expected b/javascript/ql/test/library-tests/ModelGeneration/ModelGeneration.expected
@@ -48,6 +48,9 @@ typeModel
 | (subclass).D | subclass | Member[D] |
 | (subclass).D.prototype | (subclass).D | Instance |
 | (subclass).D.prototype.d | (subclass).D.prototype | Member[d] |
+| (subclass).ExposedMidSubClass | subclass | Member[ExposedMidSubClass] |
+| (subclass).ExposedMidSubClass.prototype | (subclass).ExposedMidSubClass | Instance |
+| (subclass).ExposedMidSubClass.prototype.m | (subclass).ExposedMidSubClass.prototype | Member[m] |
 | upstream-lib | (reexport).func | ReturnValue |
 | upstream-lib | reexport | Member[lib] |
 | upstream-lib.Type | (subclass).D.prototype |  |

diff --git a/javascript/ql/test/library-tests/ModelGeneration/subclass/subclass.js b/javascript/ql/test/library-tests/ModelGeneration/subclass/subclass.js
@@ -15,3 +15,11 @@ import * as upstream from "upstream-lib";
 export class D extends upstream.Type {
     d() {}
 }
+
+// Test case where subclass chain goes through an internal class
+// TODO: we miss the subclass chain between ExposedMidSubClass and A
+class InternalMidClass extends A {}
+
+export class ExposedMidSubClass extends InternalMidClass {
+    m() {}
+}