-
Story
-
Resolution: Done
-
Minor
-
DO378 - RHBoQ1.11-en-1-20210930
-
None
-
ILT, ROLE, VT
-
en-US (English)
URL:
Reporter RHNID:
Section: -
Language: en-US (English)||
Workaround:
Description: In lab-apps-review script, test_with_flag_disabled searches for a specific state in the features.session-integration toggle, which is broken. The user may have left the application.properties file in an undefined state or set the property to whatever value, so it is incorrect to assume it is currently set to true, or indeed set at all.
Furthermore, the test assumes there are no spaces around the assignment character, which is broken as well. Valid syntax can include as much whitespace as desired on either side of the equals sign.
function test_with_flag_disabled {
...
}}{{ sed -i 's/features.session-integration=true/features.session-integration=false/g' src/main/resources/application.properties
...
}
The above should be rewritten to not assume anything.
Same for the test_with_flag_enabled function.