Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(707)

Unified Diff: pkg/analyzer/test/src/task/dart_work_manager_test.dart

Issue 1399873002: Issue 24538. Route SDK results into SDK context DartWorkManager. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/analyzer/test/src/context/mock_sdk.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/test/src/task/dart_work_manager_test.dart
diff --git a/pkg/analyzer/test/src/task/dart_work_manager_test.dart b/pkg/analyzer/test/src/task/dart_work_manager_test.dart
index e15aa7b4ef85d8cf1f81e208e43589903bbf3685..469d5f35022417c761f31de4488a270344b1e7bb 100644
--- a/pkg/analyzer/test/src/task/dart_work_manager_test.dart
+++ b/pkg/analyzer/test/src/task/dart_work_manager_test.dart
@@ -16,6 +16,7 @@ import 'package:analyzer/src/generated/engine.dart'
import 'package:analyzer/src/generated/error.dart' show AnalysisError;
import 'package:analyzer/src/generated/java_engine.dart' show CaughtException;
import 'package:analyzer/src/generated/scanner.dart' show ScannerErrorCode;
+import 'package:analyzer/src/generated/sdk.dart';
import 'package:analyzer/src/generated/source.dart';
import 'package:analyzer/src/generated/testing/ast_factory.dart';
import 'package:analyzer/src/task/dart.dart';
@@ -604,6 +605,28 @@ class DartWorkManagerTest {
expect(cache.getState(part1, CONTAINING_LIBRARIES), CacheState.INVALID);
}
+ void test_resultsComputed_inSDK() {
+ DartWorkManager sdkDartWorkManagerMock = new _DartWorkManagerMock();
+ // SDK context mock
+ InternalAnalysisContext sdkContextMock = new _InternalAnalysisContextMock();
+ when(sdkContextMock.workManagers).thenReturn([sdkDartWorkManagerMock]);
+ // SDK mock
+ DartSdk sdkMock = new _DartSdkMock();
+ when(sdkMock.context).thenReturn(sdkContextMock);
+ // SourceFactory mock
+ SourceFactory sourceFactory = new _SourceFactoryMock();
+ when(sourceFactory.dartSdk).thenReturn(sdkMock);
+ when(context.sourceFactory).thenReturn(sourceFactory);
+ // SDK source mock
+ Source source = new _SourceMock('test.dart');
+ when(source.source).thenReturn(source);
+ when(source.isInSystemLibrary).thenReturn(true);
+ // notify and validate
+ Map<ResultDescriptor, dynamic> outputs = <ResultDescriptor, dynamic>{};
+ manager.resultsComputed(source, outputs);
+ verify(sdkDartWorkManagerMock.resultsComputed(source, outputs)).once();
+ }
+
void test_resultsComputed_noSourceKind() {
manager.unknownSourceQueue.addAll([source1, source2]);
manager.resultsComputed(source1, {});
@@ -706,6 +729,14 @@ class DartWorkManagerTest {
}
}
+class _DartSdkMock extends TypedMock implements DartSdk {
+ noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
+}
+
+class _DartWorkManagerMock extends TypedMock implements DartWorkManager {
+ noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
+}
+
class _InternalAnalysisContextMock extends TypedMock
implements InternalAnalysisContext {
@override
@@ -739,3 +770,17 @@ class _InternalAnalysisContextMock extends TypedMock
noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
}
+
+class _SourceFactoryMock extends TypedMock implements SourceFactory {
+ noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
+}
+
+class _SourceMock extends TypedMock implements Source {
+ final String shortName;
+ _SourceMock(this.shortName);
+ @override
+ String get fullName => '/' + shortName;
+ noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
+ @override
+ String toString() => fullName;
+}
« no previous file with comments | « pkg/analyzer/test/src/context/mock_sdk.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698