Skip to content

Commit a597dd0

Browse files
authored
Fix the analyzer-from-sdk test to work on stable (#2361)
* Fix the analyzer-from-sdk test to work on stable. * dartfmt
1 parent 9b932a2 commit a597dd0

File tree

2 files changed

+10
-7
lines changed

2 files changed

+10
-7
lines changed

.travis.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,6 @@ matrix:
1818
# it is unlikely to work and produces uninteresting
1919
# results.
2020
dart: stable
21-
allow_failures:
22-
# Some packages at HEAD require 2.10.0-dev.
23-
- env: DARTDOC_BOT=sdk-analyzer
24-
dart: stable
2521

2622
env:
2723
jobs:

tool/grind.dart

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -435,8 +435,9 @@ Future<String> createComparisonDartdoc() async {
435435
/// to be a git repository), configured to use packages from the Dart SDK.
436436
///
437437
/// This copy of dartdoc depends on the HEAD versions of various packages
438-
/// developed within the SDK, such as 'analyzer' and 'meta'.
439-
Future<String> createSdkDartdoc() async {
438+
/// developed within the SDK, such as 'analyzer' and '_fe_analyzer_shared'.
439+
/// 'meta' is overridden if [overrideMeta] is true.
440+
Future<String> createSdkDartdoc(bool overrideMeta) async {
440441
var launcher = SubprocessLauncher('create-sdk-dartdoc');
441442
var dartdocSdk = Directory.systemTemp.createTempSync('dartdoc-sdk');
442443
await launcher
@@ -473,9 +474,13 @@ dependency_overrides:
473474
path: '${sdkClone.path}/pkg/analyzer'
474475
_fe_analyzer_shared:
475476
path: '${sdkClone.path}/pkg/_fe_analyzer_shared'
477+
''', mode: FileMode.append);
478+
if (overrideMeta) {
479+
dartdocPubspec.writeAsStringSync('''
476480
meta:
477481
path: '${sdkClone.path}/pkg/meta'
478482
''', mode: FileMode.append);
483+
}
479484
await launcher.runStreamed(sdkBin('pub'), ['get'],
480485
workingDirectory: dartdocSdk.path);
481486
return dartdocSdk.path;
@@ -484,7 +489,9 @@ dependency_overrides:
484489
@Task('Run grind tasks with the analyzer SDK.')
485490
Future<void> testWithAnalyzerSdk() async {
486491
var launcher = SubprocessLauncher('test-with-analyzer-sdk');
487-
var sdkDartdoc = await createSdkDartdoc();
492+
// Do not override meta on branches outside of stable.
493+
var sdkDartdoc =
494+
await createSdkDartdoc(RegExp('[.]\w+').hasMatch(Platform.version));
488495
var defaultGrindParameter =
489496
Platform.environment['DARTDOC_GRIND_STEP'] ?? 'test';
490497
await launcher.runStreamed(

0 commit comments

Comments
 (0)