Refactor integration tests.

Author: davidmorgan

_test/build.throws.yaml

@@ -7,7 +7,6 @@ library;
 
 import 'dart:io';
 
-import 'package:build_runner/src/internal.dart';
 import 'package:path/path.dart' as p;
 import 'package:test/test.dart';
 
@@ -127,25 +126,6 @@ void main() {
       expect(dartSource.existsSync(), true);
     });
 
-    test('Re-snapshots if there is no asset graph', () async {
-      final assetGraph = assetGraphPathFor(scriptKernelLocation);
-      await File(assetGraph).delete();
-
-      final nextBuild = await runBuild();
-      expect(
-        (nextBuild.stdout as String).split('\n'),
-        containsAllInOrder([
-          contains('Generating the build script'),
-          contains('Compiling the build script.'),
-          contains('Creating the asset graph.'),
-          contains(
-            'Building, full build because there is no valid asset graph.',
-          ),
-          contains(BuildLog.successPattern),
-        ]),
-      );
-    });
-
     test('incremental build after resolve missing import', () async {
       final dartSource = File(p.join('lib', 'app.dart'));
       dartSource.writeAsStringSync(

_test/test/build_integration_test.dart

@@ -79,29 +79,5 @@ builders:
         expect(result.stdout, contains('could not be parsed'));
       });
     });
-
-    test('checks builder keys in global_options', () async {
-      await d.dir('a', [
-        d.file('build.yaml', '''
-global_options:
-  a:a:
-    runs_before:
-      - b:b
-'''),
-      ]).create();
-
-      final result = await runPub('a', 'run', args: ['build_runner', 'build']);
-      expect(result.stderr, isEmpty);
-      expect(
-        result.stdout,
-        allOf(
-          contains('Ignoring `global_options` for unknown builder `a:a`.'),
-          contains(
-            'Ignoring `runs_before` in `global_options` '
-            'referencing unknown builder `b:b`.',
-          ),
-        ),
-      );
-    });
   });
 }

_test/test/exception_handling_test.dart

@@ -27,6 +27,13 @@ Future<ProcessResult> pubGet(String package, {bool offline = true}) async {
   return pubGetresult;
 }
 
+/// Runs `dart run build_runner` [command] on [package] with [args].
+Future<ProcessResult> runBuildRunner(
+  String package,
+  String command, {
+  Iterable<String>? args,
+}) => runPub(package, 'run', args: ['build_runner', command, ...?args]);
+
 /// Runs the `pub` [command] on [package] with [args].
 Future<ProcessResult> runPub(
   String package,
@@ -38,6 +45,13 @@ Future<ProcessResult> runPub(
   ...?args,
 ], workingDirectory: p.join(d.sandbox, package));
 
+/// Starts `dart run build_runner` [command] on [package] with [args].
+Future<Process> startBuildRunner(
+  String package,
+  String command, {
+  Iterable<String>? args,
+}) => startPub(package, 'run', args: ['build_runner', command, ...?args]);
+
 /// Starts the `pub` [command] on [package] with [args].
 Future<Process> startPub(
   String package,

build_runner/test/bootstrap/build_script_generate_test.dart

@@ -0,0 +1,49 @@
+// Copyright (c) 2025, 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.
+
+@Tags(['integration'])
+library;
+
+import 'package:test/test.dart';
+
+import 'build_runner_tester.dart';
+import 'fixture_packages.dart';
+
+void main() async {
+  test('build command --build-filter', () async {
+    final pubspecs = await Pubspecs.load();
+    final tester = BuildRunnerTester(pubspecs);
+
+    tester.writeFixturePackage(
+      copyBuilderPackage(buildToCache: true, applyToAllPackages: true),
+    );
+    tester.writePackage(
+      name: 'root_pkg',
+      dependencies: ['build_runner'],
+      pathDependencies: ['builder_pkg', 'other_pkg'],
+      files: {
+        'lib/a.txt': 'a',
+        'lib/b.txt': 'b',
+        'web/a.txt': 'a',
+        'web/b.txt': 'b',
+      },
+    );
+    tester.writePackage(
+      name: 'other_pkg',
+      files: {'lib/a.txt': 'a', 'lib/b.txt': 'b'},
+    );
+
+    await tester.run(
+      'root_pkg',
+      'dart run build_runner build '
+          '--build-filter package:*/a.txt.copy '
+          '--build-filter web/a.txt.copy ',
+    );
+    expect(tester.readFileTree('root_pkg/.dart_tool/build/generated'), {
+      'root_pkg/web/a.txt.copy': 'a',
+      'root_pkg/lib/a.txt.copy': 'a',
+      'other_pkg/lib/a.txt.copy': 'a',
+    });
+  });
+}

build_runner/test/common/sdk.dart

@@ -0,0 +1,80 @@
+// Copyright (c) 2025, 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.
+
+@Tags(['integration'])
+library;
+
+import 'package:io/io.dart';
+import 'package:test/test.dart';
+
+import 'build_runner_tester.dart';
+
+void main() async {
+  test('build command config validation', () async {
+    final pubspecs = await Pubspecs.load();
+    final tester = BuildRunnerTester(pubspecs);
+
+    tester.writePackage(
+      name: 'root_pkg',
+      dependencies: ['build_runner'],
+      files: {
+        'build.yaml': r'''
+targets:
+  $default:
+    builders:
+      bad:builder:
+        enabled: true
+''',
+      },
+    );
+
+    var output = await tester.run('root_pkg', 'dart run build_runner build');
+    expect(
+      output,
+      contains(
+        'Ignoring options for unknown builder `bad:builder` '
+        'in target `root_pkg:root_pkg`.',
+      ),
+    );
+
+    tester.write('root_pkg/build.yaml', r'''
+global_options:
+  bad:builder:
+    options: {}
+''');
+    output = await tester.run('root_pkg', 'dart run build_runner build');
+    expect(
+      output,
+      contains('Ignoring `global_options` for unknown builder `bad:builder`.'),
+    );
+
+    tester.delete('root_pkg/build.yaml');
+    output = await tester.run(
+      'root_pkg',
+      'dart run build_runner build --define=bad:key=foo=bar',
+    );
+    expect(
+      output,
+      contains('Ignoring options overrides for unknown builder `bad:key`.'),
+    );
+
+    // Compile errors caused by config errors.
+    tester.write('root_pkg/build.yaml', r'''
+builders:
+  test_builder:
+    import: 'missing_builder.dart'
+    builder_factories: ['missingFactory']
+    build_extensions: {'.txt': ['.txt.copy']}
+    auto_apply: root_package
+    build_to: source
+''');
+    output = await tester.run(
+      'root_pkg',
+      'dart run build_runner build',
+      expectOutputCode: ExitCode.config.code,
+    );
+    expect(output, contains("Error when reading 'missing_builder.dart'"));
+    expect(output, contains("Undefined name 'missingFactory'"));
+  });
+}

build_runner/test/integration_tests/build_command_build_filter_test.dart

@@ -0,0 +1,84 @@
+// Copyright (c) 2025, 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.
+
+@Tags(['integration'])
+library;
+
+import 'package:test/test.dart';
+
+import 'build_runner_tester.dart';
+
+void main() async {
+  test('build command --define', () async {
+    final pubspecs = await Pubspecs.load();
+    final tester = BuildRunnerTester(pubspecs);
+
+    tester.writePackage(
+      name: 'builder_pkg',
+      dependencies: ['build', 'build_runner'],
+      files: {
+        'build.yaml': r'''
+builders:
+  test_builder:
+    import: 'package:builder_pkg/builder.dart'
+    builder_factories: ['testBuilderFactory']
+    build_extensions: {'.txt': ['.txt.copy']}
+    auto_apply: root_package
+    build_to: source
+''',
+        'lib/builder.dart': '''
+import 'package:build/build.dart';
+
+Builder testBuilderFactory(BuilderOptions options) =>
+    TestBuilder(AssetId.parse(options.config['copy_from'] as String));
+
+class TestBuilder implements Builder {
+  final AssetId copyFrom;
+
+  TestBuilder(this.copyFrom);
+
+  @override
+  Map<String, List<String>> get buildExtensions => {'.txt': ['.txt.copy']};
+
+  @override
+  Future<void> build(BuildStep buildStep) async {
+    buildStep.writeAsString(
+        buildStep.inputId.addExtension('.copy'),
+        await buildStep.readAsString(copyFrom),
+    );
+  }
+}''',
+      },
+    );
+    tester.writePackage(
+      name: 'root_pkg',
+      dependencies: ['build_runner'],
+      pathDependencies: ['builder_pkg'],
+      files: {
+        'build.yaml': r'''
+targets:
+  $default:
+    builders:
+      builder_pkg:test_builder:
+        options:
+          copy_from: root_pkg|web/b.txt
+''',
+        'web/a.txt': 'a',
+        'web/b.txt': 'b',
+      },
+    );
+
+    // Config in `build.yaml` is the default.
+    await tester.run('root_pkg', 'dart run build_runner build');
+    expect(tester.read('root_pkg/web/a.txt.copy'), 'b');
+
+    // Override it with `--define`.
+    await tester.run(
+      'root_pkg',
+      'dart run build_runner build '
+          '--define=builder_pkg:test_builder=copy_from=root_pkg|web/a.txt',
+    );
+    expect(tester.read('root_pkg/web/a.txt.copy'), 'a');
+  });
+}