Start writing integration tests for the analysis server.

pkg/analysis_server/test/integration/analysis_error_inttest.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.
+
+library test.integration.analysis.error;
+
+import 'package:analysis_testing/reflective_tests.dart';
+import 'package:unittest/unittest.dart';
+
+import 'integration_tests.dart';
+
+@ReflectiveTestCase()
+class AnalysisErrorIntegrationTest extends AbstractAnalysisServerIntegrationTest
+    {
+  test_detect_simple_error() {
+    createFile('test.dart',
+        '''
+main() {
+  var x // parse error: missing ';'
+}''');
+    setAnalysisRoots(['']);
+    return analysisFinished.then((_) {
+      String filePath = normalizePath('test.dart');
+      expect(currentAnalysisErrors[filePath], isList);
+      List errors = currentAnalysisErrors[filePath];
+      expect(errors, hasLength(1));
+      expect(errors[0], isAnalysisError);
+      expect(errors[0]['location']['file'], equals(filePath));

[Brian Wilkerson, 2014/07/16 14:08:51]

Consider adding specialized matcher builders, such as

  expect(errors[0], isAnalysisErrorWith(location: isLocationWith(file:
equals(filePath))));

that would safely check the values of specific fields.

Otherwise, this might want to be re-written to check that 'location' is not null
and produce a specialized failure message if it is.

[Paul Berry, 2014/07/16 16:00:00]

Good idea.  I will do this in a future CL.

+    });
+  }
+}
+
+main() {
+  runReflectiveTests(AnalysisErrorIntegrationTest);
+}