14 – native_toolchain_c_macos
91%
main: 91%

Ran 26 Mar 2025 07:45AM UTC

Files 26

Run time 0s

Badge

Embed ▾

Committed 26 Mar 2025 07:36AM UTC coverage: 68.855%. Remained the same

Subproject # native_toolchain_c_macos – 14077879620.14

Build Type

push

github

Committed by

web-flow

Commit Message

[native_assets_cli] Prevent `EncodedAsset` and `Metadata` hashcode changing (#2131)

Closes: https://github.com/dart-lang/native/issues/2045

We have two types of classes in the semantic API:

1. Pure views on top of a JSON.
   These classes are mirrored by a builder class.
   Objects from these classes cannot be passed from input to output in hooks.
   These classes do not override `hashCode` and `operator ==`.
   Examples: `CodeConfig`, `BuildInput`.
2. Data classes that eagerly copy all fields out of the JSON.
   These classes have a public constructor with it's constituent fields.
   Objects from these classes may be passed from input to output in hooks.
   These classes do override `hashCode` and `operator ==`.
   Examples: `CodeAsset`, `Architecture`, `OS`, `DataAsset`.

This changes `Metadata` and `EncodedAsset` to follow pattern 2 fully:

* All data is copied on construction.
* The map accessors do not allow modification.

This means the `hashCode` and `operator ==` are now correct.

We could consider changing `Metadata` and `EncodedAsset` to pattern 1, but I didn't want to refactor that code in this PR. For more details:

* https://github.com/dart-lang/native/issues/2045

Run Details

776 of 1127 relevant lines covered (68.86%)

10.95 hits per line

dart-lang / native / 14077879620 / 14 – native_toolchain_c_macos
91%
main: 91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_toolchain_c_macos - 14077879620.14

dart-lang / native / 14077879620 / 14 – native_toolchain_c_macos 91% main: 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job native_toolchain_c_macos - 14077879620.14

dart-lang / native / 14077879620 / 14 – native_toolchain_c_macos
91%
main: 91%

README BADGES
x