Address code review comment - Add '"' for keywords in error message

PR #4709

Author: VineeshJain

st2common/st2common/util/pack.py

@@ -107,9 +107,9 @@ def validate_config_against_schema(config_schema, config_object, config_path,
     for key in config_object:
         if (jinja_utils.is_jinja_expression(value=config_object.get(key)) and
                 "decrypt_kv" in config_object.get(key) and config_schema.get(key).get('secret')):
-            raise ValueValidationException('Values specified as secret: True in config schema '
+            raise ValueValidationException('Values specified as "secret: True" in config schema '
                                            'are automatically decrypted by default. Use of '
-                                           'decrypt_kv jinja filter is not allowed for such '
+                                           '"decrypt_kv" jinja filter is not allowed for such '
                                            'values.')
 
     schema = util_schema.get_schema_for_resource_parameters(parameters_schema=config_schema,

st2common/tests/unit/test_configs_registrar.py

@@ -168,10 +168,9 @@ def test_register_all_configs_with_config_schema_validation_validation_failure_3
         registrar._register_pack(pack_name='dummy_pack_11', pack_dir=PACK_11_PATH)
         packs_base_paths = content_utils.get_packs_base_paths()
 
-        expected_msg = ('Values specified as secret: True in config schema '
-                        'are automatically decrypted by default. Use of '
-                        'decrypt_kv jinja filter is not allowed for such '
-                        'values.')
+        expected_msg = ('Values specified as "secret: True" in config schema are automatically '
+                        'decrypted by default. Use of "decrypt_kv" jinja filter is not allowed '
+                        'for such values.')
 
         self.assertRaisesRegexp(ValueError, expected_msg,
                                 registrar.register_from_packs,