From f571866f47cae625918574873f65979e2d4f3850 Mon Sep 17 00:00:00 2001
From: Gabriela Araujo Britto <t-gaara@microsoft.com>
Date: Mon, 25 Feb 2019 10:32:56 -0800
Subject: [PATCH] add new tests for bad class references

---
 ...amedParameters_classDeclarationAliasing.ts | 20 ++++++++++++++
 ...NamedParameters_classExpressionHeritage.ts | 26 +++++++++++++++++++
 2 files changed, 46 insertions(+)
 create mode 100644 tests/cases/fourslash/refactorConvertToNamedParameters_classDeclarationAliasing.ts
 create mode 100644 tests/cases/fourslash/refactorConvertToNamedParameters_classExpressionHeritage.ts

diff --git a/tests/cases/fourslash/refactorConvertToNamedParameters_classDeclarationAliasing.ts b/tests/cases/fourslash/refactorConvertToNamedParameters_classDeclarationAliasing.ts
new file mode 100644
index 00000000000..0914dedd57d
--- /dev/null
+++ b/tests/cases/fourslash/refactorConvertToNamedParameters_classDeclarationAliasing.ts
@@ -0,0 +1,20 @@
+/// <reference path='fourslash.ts' />
+
+////class Foo {
+////    /*a*/constructor/*b*/(a: number, b: number) { }
+////}
+////const fooAlias = Foo;
+////const newFoo = new fooAlias(1, 2);
+
+goTo.select("a", "b");
+// Refactor should not make changes
+edit.applyRefactor({
+    refactorName: "Convert to named parameters",
+    actionName: "Convert to named parameters",
+    actionDescription: "Convert to named parameters",
+    newContent: `class Foo {
+    constructor(a: number, b: number) { }
+}
+const fooAlias = Foo;
+const newFoo = new fooAlias(1, 2);`
+});
\ No newline at end of file
diff --git a/tests/cases/fourslash/refactorConvertToNamedParameters_classExpressionHeritage.ts b/tests/cases/fourslash/refactorConvertToNamedParameters_classExpressionHeritage.ts
new file mode 100644
index 00000000000..14b9639c2ad
--- /dev/null
+++ b/tests/cases/fourslash/refactorConvertToNamedParameters_classExpressionHeritage.ts
@@ -0,0 +1,26 @@
+/// <reference path='fourslash.ts' />
+
+////const foo = class Foo {
+////    /*a*/constructor/*b*/(a: number, b: number) { }
+////}
+////class Bar extends foo {
+////    constructor() {
+////        super(1, 2);
+////    }
+////}
+
+goTo.select("a", "b");
+// Refactor should not make changes
+edit.applyRefactor({
+    refactorName: "Convert to named parameters",
+    actionName: "Convert to named parameters",
+    actionDescription: "Convert to named parameters",
+    newContent: `const foo = class Foo {
+    constructor(a: number, b: number) { }
+}
+class Bar extends foo {
+    constructor() {
+        super(1, 2);
+    }
+}`
+});
\ No newline at end of file