Skip to content

Commit

Permalink
Fix test and analysis failures after SDK updates (#2624)
Browse files Browse the repository at this point in the history
  • Loading branch information
parlough authored Aug 21, 2023
1 parent fcf8566 commit 0140c1f
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/dart_services.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
strategy:
fail-fast: false
matrix:
sdk: [stable, beta, dev, old, master]
sdk: [stable, beta, dev, old, main]
steps:
- uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9
- uses: dart-lang/setup-dart@d6a63dab3335f427404425de0fbfed4686d93c4f
Expand Down
12 changes: 6 additions & 6 deletions pkgs/dart_services/test/common_server_api_protobuf_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -376,10 +376,10 @@ main() {
final data = json.decode(await response.transform(utf8.decoder).join());
final reply = proto.AssistsResponse()..mergeFromProto3Json(data);
final assists = reply.assists;
if (sdk.masterChannel || sdk.betaChannel) {
expect(assists, hasLength(3));
} else {
if (sdk.oldChannel) {
expect(assists, hasLength(2));
} else {
expect(assists, hasLength(3));
}
expect(assists.first.edits, isNotNull);
expect(assists.first.edits, hasLength(1));
Expand Down Expand Up @@ -733,10 +733,10 @@ main() {
final data = json.decode(await response.transform(utf8.decoder).join());
final reply = proto.AssistsResponse()..mergeFromProto3Json(data);
final assists = reply.assists;
if (sdk.masterChannel || sdk.betaChannel) {
expect(assists, hasLength(3));
} else {
if (sdk.oldChannel) {
expect(assists, hasLength(2));
} else {
expect(assists, hasLength(3));
}
expect(assists.first.edits, isNotNull);
expect(assists.first.edits, hasLength(1));
Expand Down
12 changes: 6 additions & 6 deletions pkgs/dart_services/test/common_server_api_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -505,10 +505,10 @@ main() {
final encoded = await response.transform(utf8.decoder).join();
final data = json.decode(encoded) as Map<String, dynamic>;
final assists = data['assists'] as List<dynamic>;
if (sdk.masterChannel || sdk.betaChannel) {
expect(assists, hasLength(3));
} else {
if (sdk.oldChannel) {
expect(assists, hasLength(2));
} else {
expect(assists, hasLength(3));
}
final firstEdit = assists.first as Map<String, dynamic>;
expect(firstEdit['edits'], isNotNull);
Expand Down Expand Up @@ -1216,10 +1216,10 @@ main() {
final encoded = await response.transform(utf8.decoder).join();
final data = json.decode(encoded) as Map<String, dynamic>;
final assists = data['assists'] as List<dynamic>;
if (sdk.masterChannel || sdk.betaChannel) {
expect(assists, hasLength(3));
} else {
if (sdk.oldChannel) {
expect(assists, hasLength(2));
} else {
expect(assists, hasLength(3));
}
final firstEdit = assists.first as Map<String, dynamic>;
expect(firstEdit['edits'], isNotNull);
Expand Down

0 comments on commit 0140c1f

Please sign in to comment.