Skip to content

Commit 151023c

Browse files
author
Andy Hanson
committed
Fix and consolidate tsx rename tests
1 parent d99a46e commit 151023c

File tree

8 files changed

+10
-187
lines changed

8 files changed

+10
-187
lines changed

tests/cases/fourslash/tsxRename10.ts

Lines changed: 0 additions & 40 deletions
This file was deleted.

tests/cases/fourslash/tsxRename11.ts

Lines changed: 0 additions & 39 deletions
This file was deleted.

tests/cases/fourslash/tsxRename12.ts

Lines changed: 0 additions & 39 deletions
This file was deleted.

tests/cases/fourslash/tsxRename13.ts

Lines changed: 0 additions & 39 deletions
This file was deleted.

tests/cases/fourslash/tsxRename6.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,4 @@
2222
//// let opt3 = <[|Opt|] wrong />;
2323
//// let opt4 = <[|Opt|] propx={100} propString="hi" />;
2424

25-
let ranges = test.ranges();
26-
verify.assertHasRanges(ranges);
27-
for (let range of ranges) {
28-
goTo.position(range.start);
29-
verify.renameLocations(/*findInStrings*/ false, /*findInComments*/ false);
30-
}
25+
verify.rangesAreRenameLocations();

tests/cases/fourslash/tsxRename7.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,4 @@
2121
//// let opt2 = <Opt [|propx|]={100} optional/>;
2222
//// let opt3 = <Opt wrong />;
2323

24-
let ranges = test.ranges();
25-
verify.assertHasRanges(ranges);
26-
for (let range of ranges) {
27-
goTo.position(range.start);
28-
verify.renameLocations(/*findInStrings*/ false, /*findInComments*/ false);
29-
}
24+
verify.rangesAreRenameLocations();

tests/cases/fourslash/tsxRename8.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,4 @@
2323
//// let opt3 = <Opt [|wrong|] />;
2424
//// let opt4 = <Opt propx={100} propString="hi" />;
2525

26-
let ranges = test.ranges();
27-
verify.assertHasRanges(ranges);
28-
for (let range of ranges) {
29-
goTo.position(range.start);
30-
verify.renameLocations(/*findInStrings*/ false, /*findInComments*/ false);
31-
}
26+
verify.rangesAreRenameLocations();

tests/cases/fourslash/tsxRename9.ts

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,24 +15,19 @@
1515
//// className?: string;
1616
//// }
1717
//// interface ButtonProps extends ClickableProps {
18-
//// onClick(event?: React.MouseEvent<HTMLButtonElement>): void;
18+
//// [|onClick|](event?: React.MouseEvent<HTMLButtonElement>): void;
1919
//// }
2020
//// interface LinkProps extends ClickableProps {
21-
//// goTo: string;
21+
//// [|goTo|]: string;
2222
//// }
2323
//// declare function [|MainButton|](buttonProps: ButtonProps): JSX.Element;
2424
//// declare function [|MainButton|](linkProps: LinkProps): JSX.Element;
2525
//// declare function [|MainButton|](props: ButtonProps | LinkProps): JSX.Element;
2626
//// let opt = <[|MainButton|] />;
2727
//// let opt = <[|MainButton|] children="chidlren" />;
28-
//// let opt = <[|MainButton|] onClick={()=>{}} />;
29-
//// let opt = <[|MainButton|] onClick={()=>{}} ignore-prop />;
30-
//// let opt = <[|MainButton|] goTo="goTo" />;
31-
//// let opt = <[|MainButton|] wrong />;
28+
//// let opt = <[|MainButton|] [|onClick|]={()=>{}} />;
29+
//// let opt = <[|MainButton|] [|onClick|]={()=>{}} [|ignore-prop|] />;
30+
//// let opt = <[|MainButton|] [|goTo|]="goTo" />;
31+
//// let opt = <[|MainButton|] [|wrong|] />;
3232

33-
let ranges = test.ranges();
34-
verify.assertHasRanges(ranges);
35-
for (let range of ranges) {
36-
goTo.position(range.start);
37-
verify.renameLocations(/*findInStrings*/ false, /*findInComments*/ false);
38-
}
33+
verify.rangesWithSameTextAreRenameLocations();

0 commit comments

Comments
 (0)