Allow modification of a BuildOutput's raw dependencies (#169)

Author: parlough

pkgs/native_assets_cli/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.3.2
+
+- Fixed an issue where `Depenendencies.dependencies` could not be
+  modified when expected to.
+
 ## 0.3.1
 
 - Added `Target.androidRiscv64`.

pkgs/native_assets_cli/lib/src/model/build_output.dart

@@ -33,8 +33,10 @@ class BuildOutput {
     Metadata? metadata,
   })  : timestamp = (timestamp ?? DateTime.now()).roundDownToSeconds(),
         assets = assets ?? [],
-        dependencies = dependencies ?? const Dependencies([]),
-        metadata = metadata ?? const Metadata({});
+        // ignore: prefer_const_constructors
+        dependencies = dependencies ?? Dependencies([]),
+        // ignore: prefer_const_constructors
+        metadata = metadata ?? Metadata({});
 
   static const _assetsKey = 'assets';
   static const _dependenciesKey = 'dependencies';

pkgs/native_assets_cli/lib/src/model/dependencies.dart

@@ -10,6 +10,7 @@ import '../utils/uri.dart';
 import '../utils/yaml.dart';
 
 class Dependencies {
+  /// The dependencies a build relied on.
   final List<Uri> dependencies;
 
   const Dependencies(this.dependencies);
@@ -19,7 +20,8 @@ class Dependencies {
     if (yaml is YamlList) {
       return Dependencies.fromYaml(yaml);
     }
-    return const Dependencies([]);
+    // ignore: prefer_const_constructors
+    return Dependencies([]);
   }
 
   factory Dependencies.fromYaml(YamlList? yamlList) => Dependencies([

pkgs/native_assets_cli/pubspec.yaml

@@ -2,7 +2,7 @@ name: native_assets_cli
 description: >-
   A library that contains the argument and file formats for implementing a
   native assets CLI.
-version: 0.3.1
+version: 0.3.2
 repository: https://github.com/dart-lang/native/tree/main/pkgs/native_assets_cli
 
 topics:

pkgs/native_assets_cli/test/model/build_output_test.dart

@@ -159,4 +159,15 @@ version: ${BuildOutput.version}'''),
       throwsFormatException,
     );
   });
+
+  test('BuildOutput dependencies can be modified', () {
+    // TODO(https://github.com/dart-lang/native/issues/25):
+    // Remove once dependencies are made immutable.
+    final buildOutput = BuildOutput();
+    expect(
+      () => buildOutput.dependencies.dependencies
+          .add(Uri.file('path/to/file.ext')),
+      returnsNormally,
+    );
+  });
 }