Skip to content
This repository was archived by the owner on Dec 6, 2023. It is now read-only.

Commit 299aa25

Browse files
Fix up beta and old channel tests (#1077)
* Fix up `beta` tests * Fix up `old` channel as well
1 parent 7e3ed29 commit 299aa25

File tree

4 files changed

+12
-32
lines changed

4 files changed

+12
-32
lines changed

test/analysis_server_test.dart

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -104,13 +104,8 @@ void defineTests() {
104104
final issue = results.issues[0];
105105
expect(issue.line, 2);
106106
expect(issue.kind, 'info');
107-
if (!sdk.oldChannel) {
108-
expect(issue.message,
109-
'An uninitialized variable should have an explicit type annotation.');
110-
} else {
111-
expect(
112-
issue.message, 'Prefer typing uninitialized variables and fields.');
113-
}
107+
expect(issue.message,
108+
'An uninitialized variable should have an explicit type annotation.');
114109
});
115110

116111
test('repro #126 - completions polluted on second request', () async {
@@ -287,13 +282,8 @@ void defineTests() {
287282
final issue = results.issues[0];
288283
expect(issue.line, 2);
289284
expect(issue.kind, 'info');
290-
if (!sdk.oldChannel) {
291-
expect(issue.message,
292-
'An uninitialized variable should have an explicit type annotation.');
293-
} else {
294-
expect(
295-
issue.message, 'Prefer typing uninitialized variables and fields.');
296-
}
285+
expect(issue.message,
286+
'An uninitialized variable should have an explicit type annotation.');
297287
});
298288

299289
test('files={} repro #126 - completions polluted on second request',

test/common_server_api_protobuf_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -377,7 +377,7 @@ main() {
377377
final data = json.decode(await response.transform(utf8.decoder).join());
378378
final reply = proto.AssistsResponse()..mergeFromProto3Json(data);
379379
final assists = reply.assists;
380-
if (sdk.masterChannel) {
380+
if (sdk.masterChannel || sdk.betaChannel) {
381381
expect(assists, hasLength(3));
382382
} else {
383383
expect(assists, hasLength(2));
@@ -727,7 +727,7 @@ main() {
727727
final data = json.decode(await response.transform(utf8.decoder).join());
728728
final reply = proto.AssistsResponse()..mergeFromProto3Json(data);
729729
final assists = reply.assists;
730-
if (sdk.masterChannel) {
730+
if (sdk.masterChannel || sdk.betaChannel) {
731731
expect(assists, hasLength(3));
732732
} else {
733733
expect(assists, hasLength(2));

test/common_server_api_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -504,7 +504,7 @@ main() {
504504
final encoded = await response.transform(utf8.decoder).join();
505505
final data = json.decode(encoded) as Map<String, dynamic>;
506506
final assists = data['assists'] as List<dynamic>;
507-
if (sdk.masterChannel) {
507+
if (sdk.masterChannel || sdk.betaChannel) {
508508
expect(assists, hasLength(3));
509509
} else {
510510
expect(assists, hasLength(2));
@@ -1215,7 +1215,7 @@ main() {
12151215
final encoded = await response.transform(utf8.decoder).join();
12161216
final data = json.decode(encoded) as Map<String, dynamic>;
12171217
final assists = data['assists'] as List<dynamic>;
1218-
if (sdk.masterChannel) {
1218+
if (sdk.masterChannel || sdk.betaChannel) {
12191219
expect(assists, hasLength(3));
12201220
} else {
12211221
expect(assists, hasLength(2));

test/flutter_analysis_server_test.dart

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,8 @@ class HelloWorld extends StatelessWidget {
108108
final issue = results.issues[0];
109109
expect(issue.line, 4);
110110
expect(issue.kind, 'info');
111-
if (!sdk.oldChannel) {
112-
expect(issue.message,
113-
'An uninitialized variable should have an explicit type annotation.');
114-
} else {
115-
expect(
116-
issue.message, 'Prefer typing uninitialized variables and fields.');
117-
}
111+
expect(issue.message,
112+
'An uninitialized variable should have an explicit type annotation.');
118113
});
119114

120115
test('analyze counter app', () async {
@@ -272,13 +267,8 @@ class HelloWorld extends StatelessWidget {
272267
final issue = results.issues[0];
273268
expect(issue.line, 4);
274269
expect(issue.kind, 'info');
275-
if (!sdk.oldChannel) {
276-
expect(issue.message,
277-
'An uninitialized variable should have an explicit type annotation.');
278-
} else {
279-
expect(issue.message,
280-
'Prefer typing uninitialized variables and fields.');
281-
}
270+
expect(issue.message,
271+
'An uninitialized variable should have an explicit type annotation.');
282272
});
283273

284274
test('analyzeFiles counter app files={}', () async {

0 commit comments

Comments
 (0)