fix: [AAP-38471] - add default values to the user inputs of credentials

[hsong-rh]

https://issues.redhat.com/browse/AAP-38471

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Alex-Izquierdo]

I don't want to be a pain, but I really think all the static functions should have unit tests, especially since they are pure functions and therefore easy to test. In this case, we have involved three of them without any tests: add_default_values_to_user_inputs, substitute_envs, and get_env_vars_from_credentials.

[hsong-rh]

I don't want to be a pain, but I really think all the static functions should have unit tests, especially since they are pure functions and therefore easy to test. In this case, we have involved three of them without any tests: add_default_values_to_user_inputs, substitute_envs, and get_env_vars_from_credentials.

@Alex-Izquierdo I agree with you about test coverage. In this PR the test for add_default_values_to_user_inputs has already been included. The consumer test is a little bit different, the newly added test case test_handle_workers_with_env_vars has covered both substitute_envs and get_env_vars_from_credentials functions.