Skip to content

Commit dc7c6ca

Browse files
jensjohaCommit Queue
authored andcommitted
[analyzer] Fix analysis_context_collection_test on Windows (2nd try)
This fixes some randomeness caused by HashSet which seemingly triggered different sorting on Windows than in Linux. Change-Id: I54fffdbd2e056ec87958533aeafc327164808046 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/435861 Reviewed-by: Johnni Winther <[email protected]> Commit-Queue: Jens Johansen <[email protected]>
1 parent a76f343 commit dc7c6ca

File tree

2 files changed

+19
-21
lines changed

2 files changed

+19
-21
lines changed

pkg/analyzer/lib/src/dart/analysis/context_locator.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5-
import 'dart:collection';
6-
75
import 'package:analyzer/dart/analysis/context_root.dart';
86
import 'package:analyzer/file_system/file_system.dart';
97
import 'package:analyzer/file_system/physical_file_system.dart'
@@ -840,7 +838,7 @@ class ContextLocatorImpl {
840838
/// Return a list of paths that contains all of the unique elements from the
841839
/// given list of [paths], sorted such that shorter paths are first.
842840
List<String> _uniqueSortedPaths(List<String> paths) {
843-
Set<String> uniquePaths = HashSet<String>.from(paths);
841+
Set<String> uniquePaths = Set<String>.from(paths);
844842
List<String> sortedPaths = uniquePaths.toList();
845843
sortedPaths.sort((a, b) => a.length - b.length);
846844
return sortedPaths;

pkg/analyzer/test/src/dart/analysis/analysis_context_collection_test.dart

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1148,21 +1148,21 @@ contexts
11481148
uri: package:package2/package2.dart
11491149
analysisOptions_1
11501150
workspacePackage_0_1
1151-
/home/packages/package2
1151+
/home/packages/package1
11521152
packagesFile: /home/.dart_tool/package_config.json
11531153
workspace: workspace_1
11541154
analyzedFiles
1155-
/home/packages/package2/lib/package2.dart
1156-
uri: package:package2/package2.dart
1157-
analysisOptions_1
1155+
/home/packages/package1/lib/package1.dart
1156+
uri: package:package1/package1.dart
1157+
analysisOptions_0
11581158
workspacePackage_1_0
1159-
/home/packages/package1
1159+
/home/packages/package2
11601160
packagesFile: /home/.dart_tool/package_config.json
11611161
workspace: workspace_2
11621162
analyzedFiles
1163-
/home/packages/package1/lib/package1.dart
1164-
uri: package:package1/package1.dart
1165-
analysisOptions_0
1163+
/home/packages/package2/lib/package2.dart
1164+
uri: package:package2/package2.dart
1165+
analysisOptions_1
11661166
workspacePackage_2_0
11671167
analysisOptions
11681168
analysisOptions_0: /home/packages/package1/analysis_options.yaml
@@ -1181,13 +1181,13 @@ workspaces
11811181
root: /home
11821182
pubPackages
11831183
workspacePackage_1_0: PubPackage
1184-
root: /home/packages/package2
1184+
root: /home/packages/package1
11851185
sdkVersionConstraint: ^3.6.0
11861186
workspace_2: PackageConfigWorkspace
11871187
root: /home
11881188
pubPackages
11891189
workspacePackage_2_0: PubPackage
1190-
root: /home/packages/package1
1190+
root: /home/packages/package2
11911191
sdkVersionConstraint: ^3.6.0
11921192
''');
11931193
}
@@ -1213,26 +1213,26 @@ contexts
12131213
packagesFile: /home/.dart_tool/package_config.json
12141214
workspace: workspace_0
12151215
analyzedFiles
1216-
/home/packages/package2/lib/package2.dart
1217-
uri: package:package2/package2.dart
1218-
analysisOptions_0
1219-
workspacePackage_0_0
12201216
/home/packages/package1/lib/package1.dart
12211217
uri: package:package1/package1.dart
1218+
analysisOptions_0
1219+
workspacePackage_0_0
1220+
/home/packages/package2/lib/package2.dart
1221+
uri: package:package2/package2.dart
12221222
analysisOptions_1
12231223
workspacePackage_0_1
12241224
analysisOptions
1225-
analysisOptions_0: /home/packages/package2/analysis_options.yaml
1226-
analysisOptions_1: /home/packages/package1/analysis_options.yaml
1225+
analysisOptions_0: /home/packages/package1/analysis_options.yaml
1226+
analysisOptions_1: /home/packages/package2/analysis_options.yaml
12271227
workspaces
12281228
workspace_0: PackageConfigWorkspace
12291229
root: /home
12301230
pubPackages
12311231
workspacePackage_0_0: PubPackage
1232-
root: /home/packages/package2
1232+
root: /home/packages/package1
12331233
sdkVersionConstraint: ^3.6.0
12341234
workspacePackage_0_1: PubPackage
1235-
root: /home/packages/package1
1235+
root: /home/packages/package2
12361236
sdkVersionConstraint: ^3.6.0
12371237
''');
12381238
}

0 commit comments

Comments
 (0)