Upstream 16289 - Do not add optional survey fields with empty strings

[cigamit]

When survey fields are optional and not filled out and not backed by a variable in extra_vars, do not inlcude them at all for the job launch.

[Copilot]

Pull request overview

Updates survey default handling so optional survey questions with an empty-string default don’t automatically inject that key into extra_vars during job launch unless the user explicitly supplies it at runtime.

Changes:

• Skip adding optional, non-password survey defaults when the default is an empty string and the variable is not present in runtime extra_vars.
• Update/extend unit tests to reflect the new behavior and add coverage for explicitly-provided runtime empty strings.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
awx/main/tests/unit/models/test_survey_models.py	Updates expectations for optional defaults and adds a new test ensuring runtime-provided empty strings are retained.
awx/main/models/mixins.py	Adjusts _update_unified_job_kwargs to omit empty-string defaults for optional survey fields unless explicitly provided at runtime.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.