[dartdev] Run hooks for dev dependencies in dart run

The user might want to access the assets from dev dependencies.
The Dart code might want to use dev dependencies. `dart run test`.

Closes: dart-lang/tools#2237
Change-Id: Ib8c9b4a7f3f2dedb33777e1f29cb82fc678edd4f
Cq-Include-Trybots: luci.dart.try:pkg-linux-debug-try,pkg-linux-release-arm64-try,pkg-linux-release-try,pkg-mac-release-arm64-try,pkg-mac-release-try,pkg-win-release-arm64-try,pkg-win-release-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/462980
Commit-Queue: Daco Harkes <dacoharkes@google.com>
Reviewed-by: Michael Goderbauer <goderbauer@google.com>

Author: dcharkes

pkg/dartdev/lib/src/commands/run.dart

@@ -501,7 +501,10 @@ Usage: dart [vm-options] run [arguments] [<dart-file>|<local-package>|<remote-ex
           packageConfigUri: packageConfigUri,
           packageConfig: packageConfig,
           runPackageName: runPackageName,
-          includeDevDependencies: false,
+          // Enable accessing assets of dev dependencies in the debugger and
+          // enabling commands such as `dart run test` and `dart run
+          // coverage_with_test` that rely on having dev dependencies.
+          includeDevDependencies: true,
           verbose: verbose,
           dataAssetsExperimentEnabled: dataAssetsExperimentEnabled,
         );

pkg/dartdev/test/native_assets/run_test.dart

@@ -82,6 +82,31 @@ void main([List<String> args = const []]) async {
     });
   });
 
+  for (final subcommand in ['test', 'test/my_test.dart']) {
+    test('dart run $subcommand (dev_dependency_with_hook)',
+        timeout: longTimeout, () async {
+      await nativeAssetsTest('dev_dependency_with_hook', (packageUri) async {
+        final result = await runDart(
+          arguments: [
+            'run',
+            subcommand,
+          ],
+          workingDirectory: packageUri,
+          logger: logger,
+        );
+        expect(
+          result.stdout,
+          stringContainsInOrder(
+            [
+              'native add test',
+              'All tests passed!',
+            ],
+          ),
+        );
+      });
+    });
+  }
+
   test('dart run some_dev_dep', timeout: longTimeout, () async {
     await nativeAssetsTest('native_add', (packageUri) async {
       final result = await runDart(
@@ -94,7 +119,7 @@ void main([List<String> args = const []]) async {
         logger: logger,
       );
       // It should not build native_add for running ffigen.
-      expect(result.stdout, isNot(contains('build.dart')));
+      expect(result.stdout, isNot(contains('Running build hooks')));
     });
   });