dart-lang / native / 12217257468

88%

Build Type

push

github

Committed by web-flow

Commit Message

[native_assets_builder] Report dart sources as dependencies of hooks (#1780)

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

This PR adds the Dart sources to the dependencies in `HookResult`.

It also fixes the dep-file parsing w.r.t. to escapes.

### Implementation details

We're passing around `HookOutput`s which is the deserialized `output.json`. We could add the Dart sources as dependencies to it _after_ deserializing, but then the correspondence to the json is lost. So instead I've added an extra return value to the places where we pass `HookOutput` around.

Run Details

18 of 23 new or added lines in 3 files covered. (78.26%)

19 existing lines in 3 files now uncovered.

11840 of 13316 relevant lines covered (88.92%)

34.89 hits per line