CiscoDevNet / terraform-provider-cml2 / 25677095211

84%

main: 84%

Pull #190

github

fix(node): keep explicit named configs, avoid collapsing

- internal/cmlschema: NewNamedConfigs returns null only when the API has
  no named configs; stop collapsing a single "default" entry so explicit
  named configs are preserved
- provider/resource/node (create): read req.Config into configData and
  use it to detect explicit `configurations`; validate provider
  `named_configs` option and error if both `configuration` and
  `configurations` are provided (treat explicit null as not-provided)
- provider/resource/node (modify_plan): when explicit named configs are
  configured, force the plan's single `configuration` to null to avoid
  UseStateForUnknown pinning prior values and causing a post-apply shape
  mismatch
- provider/resource/node (update): guard normalization/collapse logic so
  single-configuration paths only run when `configurations` is null;
  preserve external_connector back-compat only for configuration-only
  cases
- add null checks throughout to avoid treating explicit nulls as
  unknowns and to make intent (config vs configurations) deterministic

19 of 21 new or added lines in 4 files covered. (90.48%)

4842 of 5747 relevant lines covered (84.25%)

1316.97 hits per line