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

Side by Side Diff: pkg/analysis_server/test/integration/server/status_test.dart

Issue 2394683006: Switch analysis_server to use 'package:test'. (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library test.integration.server.status; 5 library test.integration.server.status;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 8
9 import 'package:analysis_server/plugin/protocol/protocol.dart'; 9 import 'package:analysis_server/plugin/protocol/protocol.dart';
10 import 'package:test/test.dart';
10 import 'package:test_reflective_loader/test_reflective_loader.dart'; 11 import 'package:test_reflective_loader/test_reflective_loader.dart';
11 import 'package:unittest/unittest.dart';
12 12
13 import '../../utils.dart'; 13 import '../../utils.dart';
14 import '../integration_tests.dart'; 14 import '../integration_tests.dart';
15 15
16 main() { 16 main() {
17 initializeTestEnvironment(); 17 initializeTestEnvironment();
18 defineReflectiveTests(Test); 18 defineReflectiveSuite(() {
19 defineReflectiveTests(Test);
20 });
19 } 21 }
20 22
21 @reflectiveTest 23 @reflectiveTest
22 class Test extends AbstractAnalysisServerIntegrationTest { 24 class Test extends AbstractAnalysisServerIntegrationTest {
23 test_status() { 25 test_status() {
24 // After we kick off analysis, we should get one server.status message with 26 // After we kick off analysis, we should get one server.status message with
25 // analyzing=true, and another server.status message after that with 27 // analyzing=true, and another server.status message after that with
26 // analyzing=false. 28 // analyzing=false.
27 Completer analysisBegun = new Completer(); 29 Completer analysisBegun = new Completer();
28 Completer analysisFinished = new Completer(); 30 Completer analysisFinished = new Completer();
(...skipping 16 matching lines...) Expand all
45 }'''); 47 }''');
46 standardAnalysisSetup(); 48 standardAnalysisSetup();
47 expect(analysisBegun.isCompleted, isFalse); 49 expect(analysisBegun.isCompleted, isFalse);
48 expect(analysisFinished.isCompleted, isFalse); 50 expect(analysisFinished.isCompleted, isFalse);
49 return analysisBegun.future.then((_) { 51 return analysisBegun.future.then((_) {
50 expect(analysisFinished.isCompleted, isFalse); 52 expect(analysisFinished.isCompleted, isFalse);
51 return analysisFinished.future; 53 return analysisFinished.future;
52 }); 54 });
53 } 55 }
54 } 56 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698