dart-lang / native / 12396681031

push

github

[native_assets_cli] Nest `CodeConfig` OS-specific config (#1824)

Dart and Flutter have been passing the minimum OS SDK versions for a while (in non-dry-run), which means we can mark these fields non-optional if the target OS is used.

This PR changes the `CodeConfig` to nest the OS versions (and iOS target SDK: device or simulator) to be nested under the OS and required.

Other side effects of this PR:

* Many unit tests where missing these (now) mandatory fields
* `native_toolchain_c` can no longer test without the minimum OS versions, which changed the iOS test `otool` output.

Addressing:

* https://github.com/dart-lang/native/issues/1738#issuecomment-2535921222

46 of 53 new or added lines in 3 files covered. (86.79%)

12133 of 13793 relevant lines covered (87.96%)

33.83 hits per line