Allow to pass multiple policies in the cli

tests/policy/test-test.sh

@@ -62,6 +62,22 @@ rlJournalStart
         run_test "Verify that custom value which is the same as the default is recognized" /value-source/same-as-default  "5m +10m +50m"
     rlPhaseEnd
 
+    rlPhaseStartTest "Multiple policies"
+        function run_test () {
+            rlRun "tmt -vv test export \
+                --policy-root ../../policies/test \
+                --policy-name test --policy-name contact \
+                /basic"
+            rlRun -s "tmt -vv test export \
+                --policy-root ../../policies/test \
+                --policy-name test --policy-name contact \
+                /basic 2> /dev/null"
+        }
+        run_test
+        rlRun "yq '.[0] | .test' $rlRun_LOG | grep -- 'bash -c'" 0 "Test policy was applied"
+        rlRun "yq '.[0] | .contact == \"xyzzy\"' $rlRun_LOG" 0 "Test policy was applied"
+    rlPhaseEnd
+
     rlPhaseStartTest "Test whether tmt run accepts a policy"
         function run_test () {
             local option="$1"

tmt/cli/_root.py

@@ -70,8 +70,8 @@
 
 
 def _load_policies(
-    policy_name: Optional[str],
-    policy_path: Optional[Path],
+    policy_name: list[str],
+    policy_path: list[Path],
     policy_root: Optional[Path],
 ) -> list[tmt.policy.Policy]:
     """
@@ -85,30 +85,32 @@ def _load_policies(
         directory.
     """
 
-    if policy_name is not None:
+    if policy_name:
         if policy_root is None:
             raise GeneralError(
                 "Policy can be loaded by its name only when '--policy-root' is specified."
             )
 
-        if policy_path is not None:
+        if policy_path:
             raise GeneralError(
                 "Options '--policy-name' and '--policy-file' are mutually exclusive."
             )
 
         return [
             tmt.policy.Policy.load_by_name(
-                name=policy_name,
+                name=name,
                 root=policy_root,
             )
+            for name in policy_name
         ]
 
-    if policy_path is not None:
+    if policy_path:
         return [
             tmt.policy.Policy.load_by_filepath(
-                path=policy_path,
+                path=path,
                 root=policy_root,
             )
+            for path in policy_path
         ]
 
     return []
@@ -353,8 +355,8 @@ def run(
     context: Context,
     id_: Optional[str],
     workdir_root: Optional[Path],
-    policy_file: Optional[Path],
-    policy_name: Optional[str],
+    policy_file: list[Path],
+    policy_name: list[str],
     policy_root: Optional[Path],
     recipe: Optional[Path],
     **kwargs: Any,
@@ -964,8 +966,8 @@ def tests_export(
     nitrate: bool,
     bugzilla: bool,
     template: Optional[str],
-    policy_file: Optional[Path],
-    policy_name: Optional[str],
+    policy_file: list[Path],
+    policy_name: list[str],
     policy_root: Optional[Path],
     **kwargs: Any,
 ) -> None:
@@ -1227,8 +1229,8 @@ def plans_export(
     how: str,
     format: str,
     template: Optional[str],
-    policy_file: Optional[Path],
-    policy_name: Optional[str],
+    policy_file: list[Path],
+    policy_name: list[str],
     policy_root: Optional[Path],
     **kwargs: Any,
 ) -> None:

tmt/options.py

@@ -557,6 +557,7 @@ def option(
         metavar='PATH',
         type=Path(),
         envvar='TMT_POLICY_FILE',
+        multiple=True,
         help="""
              Location of a file with policy rules for modification of
              test metadata keys. Both absolute and relative paths are
@@ -569,6 +570,7 @@ def option(
         '--policy-name',
         metavar='NAME',
         envvar='TMT_POLICY_NAME',
+        multiple=True,
         help="""
              Name of the file with policy rules for modification of test
              metadata keys. The name would be extended with ``.yaml``