[dds/dap] Remove macro tests

This removes the macro tests but leaves the URI support until it's clearer we won't use this (or that it won't become a fully documented feature of DAP, in which case it would be better to use URIs instead of Paths anyway).

Fixes #60034

Change-Id: I4df448c98eb4cc352558234c8365a5e849deed7c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/408280
Commit-Queue: Ben Konyi <[email protected]>
Reviewed-by: Ben Konyi <[email protected]>
Reviewed-by: Derek Xu <[email protected]>

Author: DanTup

pkg/dds/test/dap/integration/debug_test.dart

@@ -525,125 +525,6 @@ main() {
       expect(vmServiceUri.path, matches(vmServiceAuthCodePathPattern));
     });
 
-    for (final folderName in ['bin', 'lib']) {
-      /// Gets the expected name and URI for a macro-generated source.
-      ({String name, Uri fileLikeUri}) getExpectedMacroSource(File testFile) {
-        // Drive letters are always normalized to uppercase so expect
-        // uppercase in the path part of the macro URI.
-        final fileLikeUri = Uri.file(uppercaseDriveLetter(testFile.path))
-            .replace(scheme: 'dart-macro+file');
-        // The expected source name will differ for inside/outside the lib
-        // folder.
-        final name = folderName == 'lib'
-            ? 'dart-macro+package:my_test_project/main.dart'
-            : fileLikeUri.toString();
-
-        return (name: name, fileLikeUri: fileLikeUri);
-      }
-
-      test(
-          'can download source code from the VM for macro-generated files '
-          'in "$folderName" when the client does not support Dart URIs',
-          () async {
-        final client = dap.client;
-
-        // Create the macro impl, the script that uses it and set up macro
-        // support.
-        dap.createTestFile(
-          filename: '$folderName/with_hello.dart',
-          withHelloMacroImplementation,
-        );
-        final testFile = dap.createTestFile(
-          filename: '$folderName/main.dart',
-          withHelloMacroProgram,
-        );
-        dap.createPubspec(dap.testAppDir, 'my_test_project');
-        await dap.enableMacroSupport();
-        final macroSource = getExpectedMacroSource(testFile);
-
-        // Hit the initial breakpoint.
-        final breakpointLine = lineWith(testFile, breakpointMarker);
-        final stop = await dap.client.hitBreakpoint(
-          testFile,
-          breakpointLine,
-          toolArgs: ['--enable-experiment=macros'],
-        );
-
-        // Step in to the hello() method provided by the macro.
-        final responses = await Future.wait([
-          client.expectStop('step', sourceName: macroSource.name),
-          client.stepIn(stop.threadId!),
-        ], eagerError: true);
-        final stopResponse = responses.first as StoppedEventBody;
-
-        // Fetch the top stack frame (which should be inside print).
-        final stack = await client.getValidStack(
-          stopResponse.threadId!,
-          startFrame: 0,
-          numFrames: 1,
-        );
-        final topFrame = stack.stackFrames.first;
-
-        // Downloaded macro sources should have a sourceReference and no path.
-        expect(topFrame.source!.path, isNull);
-        expect(topFrame.source!.sourceReference, isPositive);
-
-        // Source code should contain the augmentation for class A.
-        final source = await client.getValidSource(topFrame.source!);
-        expect(source.content, contains('augment class A'));
-      });
-
-      test(
-          'can use local source code for macro-generated files '
-          'in "$folderName" when the client supports Dart URIs', () async {
-        final client = dap.client;
-
-        // Create the macro impl, the script that uses it and set up macro
-        // support.
-        dap.createTestFile(
-          filename: '$folderName/with_hello.dart',
-          withHelloMacroImplementation,
-        );
-        final testFile = dap.createTestFile(
-          filename: '$folderName/main.dart',
-          withHelloMacroProgram,
-        );
-        dap.createPubspec(dap.testAppDir, 'my_test_project');
-        await dap.enableMacroSupport();
-        final macroSource = getExpectedMacroSource(testFile);
-        // Tell the DA we can handle the special URIs.
-        client.supportUris = true;
-
-        // Hit the initial breakpoint.
-        final breakpointLine = lineWith(testFile, breakpointMarker);
-        final stop = await dap.client.hitBreakpoint(
-          testFile,
-          breakpointLine,
-          toolArgs: ['--enable-experiment=macros'],
-        );
-
-        // Step in to the hello() method provided by the macro.
-        final responses = await Future.wait([
-          client.expectStop('step', sourceName: macroSource.name),
-          client.stepIn(stop.threadId!),
-        ], eagerError: true);
-        final stopResponse = responses.first as StoppedEventBody;
-
-        // Fetch the top stack frame (which should be inside print).
-        final stack = await client.getValidStack(
-          stopResponse.threadId!,
-          startFrame: 0,
-          numFrames: 1,
-        );
-        final topFrame = stack.stackFrames.first;
-
-        // When we use local editor-provided sources, there should be a URI in
-        // pathand no sourceReference.
-        expect(topFrame.source!.sourceReference, isNull);
-        expect(topFrame.source!.path, macroSource.fileLikeUri.toString());
-      });
-    }
-
     test('can map SDK source code to a local path', () async {
       final client = dap.client;
       final testFile = dap.createTestFile(simpleBreakpointProgram);

pkg/dds/test/dap/integration/test_scripts.dart

@@ -163,42 +163,6 @@ const simpleBreakpointProgram = '''
   }
 ''';
 
-/// A simple script that provides a @withHello macro that adds a
-/// `hello()` method to a class that prints "Hello".
-const withHelloMacroImplementation = '''
-import 'package:macros/macros.dart';
-
-macro class WithHello implements ClassDeclarationsMacro {
-  const WithHello();
-
-  @override
-  Future<void> buildDeclarationsForClass(
-    ClassDeclaration clazz,
-    MemberDeclarationBuilder builder,
-  ) async {
-    builder.declareInType(DeclarationCode.fromString(\'''
-  void hello() {
-    print('Hello');
-  }
-\'''));
-  }
-}
-''';
-
-/// A simple script that uses [withHelloMacroImplementation] and calls the
-/// `hello()` method.
-const withHelloMacroProgram = '''
-import 'with_hello.dart';
-
-void main() {
-  final a = A();
-  a.hello(); $breakpointMarker
-}
-
-@WithHello()
-class A {}
-''';
-
 /// A simple Dart script that prints "Hello" and then "World" with a breakpoint
 /// on the line that prints "World". By restarting from the parent frame after
 /// hitting the breakpoint, the output would be "Hello", "Hello", "World".

pkg/dds/test/dap/integration/test_support.dart

@@ -199,27 +199,6 @@ foo() {
     );
   }
 
-  /// Sets up packages for macro support.
-  Future<void> enableMacroSupport() async {
-    // Compute a path to the local package that we can use.
-    final dapIntegrationTestFolder = path.dirname(Platform.script.toFilePath());
-    assert(path.split(dapIntegrationTestFolder).last == 'integration');
-
-    final sdkRoot =
-        path.normalize(path.join(dapIntegrationTestFolder, '../../../../..'));
-    final macrosPath = path.join(sdkRoot, 'pkg', 'macros');
-    await addPackageDependency(testAppDir, 'macros', Uri.file(macrosPath));
-
-    createTestFile(
-      filename: 'analysis_options.yaml',
-      '''
-analyzer:
-  enable-experiment:
-    - macros
-''',
-    );
-  }
-
   void createPubspec(Directory dir, String projectName) {
     final pubspecFile = File(path.join(dir.path, 'pubspec.yaml'));
     pubspecFile