b0d4ab8ef9
This was mostly a rename/move of the folder (and the analysis server updated all references), but I also had to:
- add `integration_test/analysis_options.yaml` to import from `../test` to get the same lint ignores
- update paths of exclusions in `verify_sorted_test.dart`
By moving all of the tests that start the server out-of-process out of test, we can:
1. Use "dart test" to just run the faster tests ("dart test test") and get functionality of the pkg:test runner (for example running tests concurrently and JSON output)
2. Allow VS Code to spawn different debug sessions for the "test" and "integration_test" folder, which means we can use a `preLaunchTask` to trigger compilation of the analysis server from source whenever running integration tests (avoiding having to compile manually, or run from source in a way that compiles a new server for each test suite)
Change-Id: I37cc03dc32d08c5b51a2eab79f6338bb079b32ac
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/434801
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Samuel Rawlins <srawlins@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
130 lines
3.6 KiB
Dart
130 lines
3.6 KiB
Dart
// Copyright (c) 2014, 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:analysis_server/protocol/protocol_generated.dart';
|
|
import 'package:test/test.dart';
|
|
import 'package:test_reflective_loader/test_reflective_loader.dart';
|
|
|
|
import '../support/integration_tests.dart';
|
|
|
|
void main() {
|
|
defineReflectiveSuite(() {
|
|
defineReflectiveTests(OverridesTest);
|
|
});
|
|
}
|
|
|
|
@reflectiveTest
|
|
class OverridesTest extends AbstractAnalysisServerIntegrationTest {
|
|
Future<void> test_overrides() async {
|
|
var pathname = sourcePath('test.dart');
|
|
var text = r'''
|
|
abstract class Interface1 {
|
|
method0();
|
|
method1();
|
|
method2();
|
|
method3();
|
|
}
|
|
|
|
abstract class Interface2 {
|
|
method0();
|
|
method1();
|
|
method4();
|
|
method5();
|
|
}
|
|
|
|
abstract class Base {
|
|
method0();
|
|
method2();
|
|
method4();
|
|
method6();
|
|
}
|
|
|
|
class Target extends Base implements Interface1, Interface2 {
|
|
method0() {}
|
|
method1() {}
|
|
method2() {}
|
|
method3() {}
|
|
method4() {}
|
|
method5() {}
|
|
method6() {}
|
|
method7() {}
|
|
}
|
|
''';
|
|
writeFile(pathname, text);
|
|
await standardAnalysisSetup();
|
|
await sendAnalysisSetSubscriptions({
|
|
AnalysisService.OVERRIDES: [pathname],
|
|
});
|
|
|
|
var params = await onAnalysisOverrides.first;
|
|
expect(params.file, equals(pathname));
|
|
var overrides = params.overrides;
|
|
|
|
var targetOffset = text.indexOf('Target');
|
|
|
|
Override? findOverride(String methodName) {
|
|
var methodOffset = text.indexOf(methodName, targetOffset);
|
|
for (var override in overrides) {
|
|
if (override.offset == methodOffset) {
|
|
return override;
|
|
}
|
|
}
|
|
return null;
|
|
}
|
|
|
|
void checkOverrides(
|
|
String methodName,
|
|
bool expectedOverridesBase,
|
|
List<String> expectedOverridesInterfaces,
|
|
) {
|
|
var override = findOverride(methodName);
|
|
|
|
if (!expectedOverridesBase && expectedOverridesInterfaces.isEmpty) {
|
|
// This method overrides nothing, so it should not appear in the
|
|
// overrides list.
|
|
expect(override, isNull);
|
|
return;
|
|
} else {
|
|
override!;
|
|
}
|
|
|
|
expect(override.length, equals(methodName.length));
|
|
var superclassMember = override.superclassMember;
|
|
if (expectedOverridesBase) {
|
|
superclassMember!;
|
|
expect(superclassMember.element.name, equals(methodName));
|
|
expect(superclassMember.className, equals('Base'));
|
|
} else {
|
|
expect(superclassMember, isNull);
|
|
}
|
|
var interfaceMembers = override.interfaceMembers;
|
|
if (expectedOverridesInterfaces.isNotEmpty) {
|
|
interfaceMembers!;
|
|
var actualOverridesInterfaces = <String>{};
|
|
for (var overriddenMember in interfaceMembers) {
|
|
expect(overriddenMember.element.name, equals(methodName));
|
|
var className = overriddenMember.className;
|
|
var wasAdded = actualOverridesInterfaces.add(className);
|
|
expect(wasAdded, isTrue);
|
|
}
|
|
expect(
|
|
actualOverridesInterfaces,
|
|
equals(expectedOverridesInterfaces.toSet()),
|
|
);
|
|
} else {
|
|
expect(interfaceMembers, isNull);
|
|
}
|
|
}
|
|
|
|
checkOverrides('method0', true, ['Interface1', 'Interface2']);
|
|
checkOverrides('method1', false, ['Interface1', 'Interface2']);
|
|
checkOverrides('method2', true, ['Interface1']);
|
|
checkOverrides('method3', false, ['Interface1']);
|
|
checkOverrides('method4', true, ['Interface2']);
|
|
checkOverrides('method5', false, ['Interface2']);
|
|
checkOverrides('method6', true, []);
|
|
checkOverrides('method7', false, []);
|
|
}
|
|
}
|