Brian Wilkerson
7ba63bad5c
I got tired of trying to compute diagnostic locations, so I also added a utility method that will use our standard markdown format. I tried just modifying the existing `assertDiagnostics` method, but there are a few tests that include text that `TestCode` parses as markdown. Closes https://github.com/dart-lang/sdk/issues/62023 Change-Id: Iccc410cc4036c299d0cb000f1d1642ddef63e98a Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/491703 Commit-Queue: Brian Wilkerson <brianwilkerson@google.com> Reviewed-by: Samuel Rawlins <srawlins@google.com>
60 lines
2.0 KiB
Dart
60 lines
2.0 KiB
Dart
// Copyright (c) 2021, the Dart project authors. Please see the AUTHORS file
|
|
// for details. All rights reserved. Use of this source code is governed by a
|
|
// BSD-style license that can be found in the LICENSE file.
|
|
|
|
import 'package:analyzer/src/lint/registry.dart';
|
|
import 'package:analyzer/src/test_utilities/test_code_format.dart';
|
|
import 'package:analyzer_testing/analysis_rule/analysis_rule.dart';
|
|
import 'package:analyzer_testing/src/analysis_rule/pub_package_resolution.dart';
|
|
import 'package:linter/src/rules.dart';
|
|
import 'package:meta/meta.dart';
|
|
import 'package:test/test.dart';
|
|
|
|
export 'package:analyzer/src/error/codes.dart';
|
|
export 'package:analyzer_testing/analysis_rule/analysis_rule.dart' show error;
|
|
export 'package:linter/src/lint_names.dart';
|
|
|
|
mixin LanguageVersion219Mixin on PubPackageResolutionTest {
|
|
@override
|
|
String? get testPackageLanguageVersion => '2.19';
|
|
}
|
|
|
|
/// A base test class for all of the "built-in" lint rules.
|
|
abstract class LintRuleTest extends AnalysisRuleTest {
|
|
static bool _lintRulesAreRegistered = false;
|
|
|
|
@override
|
|
String get analysisRule => lintRule;
|
|
|
|
/// The lint rule being tested.
|
|
String get lintRule;
|
|
|
|
/// Assert that the given [content] has diagnostics at the marked ranges.
|
|
///
|
|
/// See the [TestCode] class for more information about the markdown format.
|
|
Future<void> assertDiagnosticsFromMarkdown(String content) {
|
|
var testCode = TestCode.parse(content);
|
|
if (testCode.ranges.isEmpty) {
|
|
fail('Either ranges or expected diagnostics must be provided.');
|
|
}
|
|
var expectedDiagnostics = [
|
|
for (var range in testCode.ranges)
|
|
lint(range.sourceRange.offset, range.sourceRange.length),
|
|
];
|
|
return super.assertDiagnostics(testCode.code, expectedDiagnostics);
|
|
}
|
|
|
|
@mustCallSuper
|
|
@override
|
|
void setUp() {
|
|
if (!_lintRulesAreRegistered) {
|
|
registerLintRules();
|
|
_lintRulesAreRegistered = true;
|
|
}
|
|
rule = Registry.ruleRegistry.rules.firstWhere(
|
|
(r) => r.name == analysisRule,
|
|
);
|
|
super.setUp();
|
|
}
|
|
}
|