Skip to content

Commit 899fa4d

Browse files
Clean up RefTestCase methods after moving them around
1 parent ce82545 commit 899fa4d

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed

test/util.dart

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -65,34 +65,34 @@ class RefTestCase {
6565
RefTestCase._({@required this.ref, @required this.verifyRefWasUpdated});
6666

6767
static RefTestCase untypedCallbackRefCase() {
68-
final untypedCallbackRefCalls = [];
68+
final calls = [];
6969
return RefTestCase._(
70-
ref: (value) => untypedCallbackRefCalls.add(value),
71-
verifyRefWasUpdated: (actualValue) => expect(untypedCallbackRefCalls, [same(actualValue)]),
70+
ref: (value) => calls.add(value),
71+
verifyRefWasUpdated: (actualValue) => expect(calls, [same(actualValue)]),
7272
);
7373
}
7474

7575
static RefTestCase typedCallbackRefCase<T>() {
76-
final typedCallbackRefCalls = [];
76+
final calls = [];
7777
return RefTestCase._(
78-
ref: (T value) => typedCallbackRefCalls.add(value),
79-
verifyRefWasUpdated: (actualValue) => expect(typedCallbackRefCalls, [same(actualValue)]),
78+
ref: (T value) => calls.add(value),
79+
verifyRefWasUpdated: (actualValue) => expect(calls, [same(actualValue)]),
8080
);
8181
}
8282

8383
static RefTestCase refObjectCase<T>() {
84-
final refObjectRef = createRef<T>();
84+
final ref = createRef<T>();
8585
return RefTestCase._(
86-
ref: refObjectRef,
87-
verifyRefWasUpdated: (actualValue) => expect(refObjectRef.current, same(actualValue)),
86+
ref: ref,
87+
verifyRefWasUpdated: (actualValue) => expect(ref.current, same(actualValue)),
8888
);
8989
}
9090

91-
static RefTestCase jsRefObjectCase<T>() {
92-
final jsRefObjectRef = React.createRef();
91+
static RefTestCase jsRefObjectCase() {
92+
final ref = React.createRef();
9393
return RefTestCase._(
94-
ref: jsRefObjectRef,
95-
verifyRefWasUpdated: (actualValue) => expect(jsRefObjectRef.current, same(actualValue)),
94+
ref: ref,
95+
verifyRefWasUpdated: (actualValue) => expect(ref.current, same(actualValue)),
9696
);
9797
}
9898

@@ -106,6 +106,6 @@ class RefTestCase {
106106
untypedCallbackRefCase(),
107107
typedCallbackRefCase<T>(),
108108
refObjectCase<T>(),
109-
jsRefObjectCase<T>(),
109+
jsRefObjectCase(),
110110
];
111111
}

0 commit comments

Comments
 (0)