Avoid spreading array (microsoft#21291)

Andy · web-flow · commit 1c9cd969662d · 2018-01-19T07:44:18.000-08:00
diff --git a/src/harness/fourslash.ts b/src/harness/fourslash.ts
@@ -1305,8 +1305,13 @@ Actual: ${stringify(fullActual)}`);
         }
 
         public verifyRangesAreRenameLocations(options?: Range[] | { findInStrings?: boolean, findInComments?: boolean, ranges?: Range[] }) {
-            const ranges = ts.isArray(options) ? options : options && options.ranges || this.getRanges();
-            this.verifyRenameLocations(ranges, { ranges, ...options });
+            if (ts.isArray(options)) {
+                this.verifyRenameLocations(options, options);
+            }
+            else {
+                const ranges = options && options.ranges || this.getRanges();
+                this.verifyRenameLocations(ranges, { ranges, ...options });
+            }
         }
 
         public verifyRenameLocations(startRanges: Range | Range[], options: Range[] | { findInStrings?: boolean, findInComments?: boolean, ranges: Range[] }) {

Original file line number	Diff line number	Diff line change
@@ -1305,8 +1305,13 @@ Actual: ${stringify(fullActual)}`);
`1305`	`1305`	`}`
`1306`	`1306`
`1307`	`1307`	`public verifyRangesAreRenameLocations(options?: Range[] \| { findInStrings?: boolean, findInComments?: boolean, ranges?: Range[] }) {`
`1308`		`- const ranges = ts.isArray(options) ? options : options && options.ranges \|\| this.getRanges();`
`1309`		`- this.verifyRenameLocations(ranges, { ranges, ...options });`
	`1308`	`+ if (ts.isArray(options)) {`
	`1309`	`+ this.verifyRenameLocations(options, options);`
	`1310`	`+ }`
	`1311`	`+ else {`
	`1312`	`+ const ranges = options && options.ranges \|\| this.getRanges();`
	`1313`	`+ this.verifyRenameLocations(ranges, { ranges, ...options });`
	`1314`	`+ }`
`1310`	`1315`	`}`
`1311`	`1316`
`1312`	`1317`	`public verifyRenameLocations(startRanges: Range \| Range[], options: Range[] \| { findInStrings?: boolean, findInComments?: boolean, ranges: Range[] }) {`