Fix empty value handling in get_variable_values

If an XCCDF Value doesn't have any value but has a selector,
we should use the actual selector name. It seems the variable
name holding the selector name was accidentally quoted so instead
of the actual selector name we got "selector" literally.

Author: jan-cerny

build-scripts/generate_profile_remediations.py

@@ -129,7 +129,7 @@ def get_variable_values(variable):
         if selector is None:
             selector = DEFAULT_SELECTOR
         if value.text is None:
-            values["selector"] = ""
+            values[selector] = ""
         else:
             values[selector] = value.text
     return values

tests/unit/build-scripts/test_generate_profile_remediations.py

@@ -256,8 +256,9 @@ def test_get_variable_values_empty_text():
     )
     variable = ET.fromstring(xml)
     result = gpr.get_variable_values(variable)
-    # When value.text is None, the code sets values["selector"] = ""
-    assert "selector" in result
+    # When value.text is None, the code sets values[selector] = ""
+    assert "empty" in result
+    assert result["empty"] == ""
 
 
 # ---------------------------------------------------------------------------