Macro. Remove tests from analyzer/ and analysis_server/.

Change-Id: Iabe04d7ff8fe0cc9b605d0c18fd47591fd02d6bd
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/406681
Reviewed-by: Phil Quitslund <[email protected]>
Commit-Queue: Konstantin Shcheglov <[email protected]>

Author: scheglov

pkg/analysis_server/test/abstract_context.dart

@@ -25,14 +25,9 @@ import 'package:meta/meta.dart';
 import 'package:test/test.dart';
 
 import 'support/configuration_files.dart';
-import 'test_macros.dart';
 
 class AbstractContextTest
-    with
-        MockPackagesMixin,
-        ConfigurationFilesMixin,
-        ResourceProviderMixin,
-        TestMacros {
+    with MockPackagesMixin, ConfigurationFilesMixin, ResourceProviderMixin {
   static bool _lintRulesAreRegistered = false;
 
   static final ByteStore _byteStore = MemoryByteStore();

pkg/analysis_server/test/analysis/get_hover_test.dart

@@ -897,26 +897,6 @@ void f(A a) {
     expect(hover.propagatedType, isNull);
   }
 
-  Future<void> test_constructorInvocation_macroGenerated_named() async {
-    addMacros([declareInTypeMacro()]);
-    newFile(testFilePath, '''
-import 'macros.dart';
-
-@DeclareInType('  /// named\\n  C.named();')
-class C {}
-
-C f() => C.named(); //
-''');
-    var hover = await prepareHover('med(); //');
-    expect(hover.containingLibraryName, 'package:test/test.dart');
-    expect(hover.containingLibraryPath, testFile.path);
-    expect(hover.containingClassDescription, 'C');
-    expect(hover.dartdoc, 'named');
-    expect(hover.elementDescription, '(new) C C.named()');
-    expect(hover.elementKind, 'constructor');
-    expect(hover.staticType, isNull);
-  }
-
   Future<void>
   test_constructorInvocation_referenceFromAugmentation_default() async {
     var file = newFile('$testPackageLibPath/a.dart', '''
@@ -1449,32 +1429,6 @@ void f() {
     expect(hover.propagatedType, null);
   }
 
-  Future<void>
-  test_methodInvocation_macroGenerated_referenceToGenerated() async {
-    addMacros([declareInTypeMacro()]);
-    newFile(testFilePath, '''
-import 'macros.dart';
-
-@DeclareInType(\'''
-  /// method m
-  void m() {}
-\''')
-class C {}
-
-void f(C c) {
-  c.m();
-}
-''');
-    var hover = await prepareHover('m();');
-    expect(hover.containingLibraryName, 'package:test/test.dart');
-    expect(hover.containingLibraryPath, testFile.path);
-    expect(hover.containingClassDescription, 'C');
-    expect(hover.dartdoc, 'method m');
-    expect(hover.elementDescription, 'void m()');
-    expect(hover.elementKind, 'method');
-    expect(hover.staticType, 'void Function()');
-  }
-
   Future<void> test_methodInvocation_recordType() async {
     newFile(testFilePath, '''
 class C {

pkg/analysis_server/test/analysis/notification_highlights2_test.dart

@@ -708,19 +708,6 @@ class A {
 ''');
   }
 
-  Future<void> test_class_macroKeyword() async {
-    var testCode = TestCode.parse(r'''
-macro class A {}
-''');
-    addTestFile(testCode.code);
-    await prepareHighlights();
-    assertHighlightText(testCode, -1, r'''
-0 + 5 |macro| BUILT_IN
-6 + 5 |class| KEYWORD
-12 + 1 |A| CLASS
-''');
-  }
-
   Future<void> test_class_method_functionTypedFormalParameter() async {
     var testCode = TestCode.parse(r'''
 class A {

pkg/analysis_server/test/analysis_server_base.dart

@@ -27,7 +27,6 @@ import 'package:unified_analytics/unified_analytics.dart';
 import 'constants.dart';
 import 'mocks.dart';
 import 'support/configuration_files.dart';
-import 'test_macros.dart';
 
 // TODO(scheglov): This is duplicate with pkg/linter/test/rule_test_support.dart.
 // Keep them as consistent with each other as they are today. Ultimately combine
@@ -236,7 +235,7 @@ abstract class ContextResolutionTest with ResourceProviderMixin {
 }
 
 class PubPackageAnalysisServerTest extends ContextResolutionTest
-    with MockPackagesMixin, ConfigurationFilesMixin, TestMacros {
+    with MockPackagesMixin, ConfigurationFilesMixin {
   // TODO(scheglov): Consider turning it back into a getter.
   late String testFilePath = resourceProvider.convertPath(
     '$testPackageLibPath/test.dart',

pkg/analysis_server/test/integration/coverage.md

@@ -105,4 +105,3 @@ server calls. This file is validated by `coverage_test.dart`.
 
 ## lsp domain
 - [x] lsp.handle
-- [x] lsp.notification

pkg/analysis_server/test/integration/coverage_test.dart

@@ -109,42 +109,6 @@ void main() {
           });
         }
       });
-
-      // notifications
-      group('notification', () {
-        for (var notification in domain.notifications) {
-          var fullName = '${domain.name}.${notification.event}';
-          test(fullName, () {
-            if (!allMembers.contains(fullName)) {
-              fail('$fullName not found in ${coverageFile.path}');
-            }
-
-            var fileName = getCamelWords(
-              notification.event,
-            ).map((s) => s.toLowerCase()).join('_');
-            var testName = path.join(domain.name, '${fileName}_test.dart');
-            var testPath = path.join(
-              pathPrefix,
-              'test',
-              'integration',
-              testName,
-            );
-
-            // Test that if checked, a test file exists; if not checked, no such
-            // file exists.
-            var fileExists = FileSystemEntity.isFileSync(testPath);
-            var isMarkedAsCovered = coveredMembers.contains(fullName);
-            expect(
-              fileExists,
-              isMarkedAsCovered,
-              reason:
-                  isMarkedAsCovered
-                      ? '$testName marked as covered but has no test at $testPath'
-                      : '$testName marked as not covered has test at $testPath',
-            );
-          });
-        }
-      });
     });
   }
 

pkg/analysis_server/test/integration/dtd/dtd_test.dart

@@ -24,7 +24,7 @@ class DtdTest
         with SharedDtdTests {
   @override
   void createFile(String path, String content) {
-    newFile(path, content);
+    writeFile(path, content);
   }
 
   @override

pkg/analysis_server/test/integration/edit/get_refactoring_test.dart

@@ -75,47 +75,4 @@ void bar() {
     await analysisFinished;
     expect(currentAnalysisErrors[pathname], isEmpty);
   }
-
-  Future<void> test_rename_macro() async {
-    addMacros([declareInTypeMacro()]);
-    var pathname = sourcePath('lib/test.dart');
-    var text = r'''
-import 'macros.dart';
-
-@DeclareInType('  /// named\\n  C.named();')
-class C {}
-''';
-    writeFile(pathname, text);
-    await standardAnalysisSetup();
-    await analysisFinished;
-    expect(currentAnalysisErrors[pathname], isEmpty);
-
-    // expect a valid rename refactoring
-    var result = await sendEditGetRefactoring(
-      RefactoringKind.RENAME,
-      pathname,
-      text.indexOf('C {'),
-      0,
-      false,
-      options: RenameOptions('Coo'),
-    );
-    expect(result.initialProblems, isEmpty);
-    expect(result.optionsProblems, isEmpty);
-    expect(result.finalProblems, isEmpty);
-    expect(result.potentialEdits, isNull);
-
-    var change = result.change!;
-    expect(change.edits.length, 1);
-    var fileEdit = change.edits.first;
-
-    // apply the refactoring, expect that the new code has no errors
-    expect(fileEdit.edits.length, 1);
-    for (var edit in fileEdit.edits) {
-      text = text.replaceRange(edit.offset, edit.end, edit.replacement);
-    }
-    await sendAnalysisUpdateContent({pathname: AddContentOverlay(text)});
-
-    await analysisFinished;
-    expect(currentAnalysisErrors[pathname], isEmpty);
-  }
 }

pkg/analysis_server/test/integration/lsp/notification_test.dart

@@ -5,11 +5,9 @@
 import 'package:test_reflective_loader/test_reflective_loader.dart';
 
 import 'handle_test.dart' as handle;
-import 'notification_test.dart' as notification;
 
 void main() {
   defineReflectiveSuite(() {
     handle.main();
-    notification.main();
   }, name: 'lsp');
 }