Merge ~sylvain-pineau/checkbox-ng:fix-1804172 into checkbox-ng:master

[Sylvain Pineau]

Fixes the linked bug. -o should indeed be a mandatory arg to all merge commands.

[Sylvain Pineau (sylvain-pineau)]

self-approved

review: Approve

diff --git a/checkbox_ng/launcher/merge_reports.py b/checkbox_ng/launcher/merge_reports.py
index c9efa74..bafd977 100644
--- a/checkbox_ng/launcher/merge_reports.py
+++ b/checkbox_ng/launcher/merge_reports.py
@@ -45,7 +45,7 @@ class MergeReports(Command):
             'submission', nargs='*', metavar='SUBMISSION',
             help='submission tarball')
         parser.add_argument(
-            '-o', '--output-file', metavar='FILE',
+            '-o', '--output-file', metavar='FILE', required=True,
             help='save combined test results to the specified FILE')
 
     def _parse_submission(self, submission, tmpdir, mode="list"):
diff --git a/checkbox_ng/launcher/merge_submissions.py b/checkbox_ng/launcher/merge_submissions.py
index b73852b..adab550 100644
--- a/checkbox_ng/launcher/merge_submissions.py
+++ b/checkbox_ng/launcher/merge_submissions.py
@@ -35,7 +35,7 @@ class MergeSubmissions(MergeReports):
             'submission', nargs='*', metavar='SUBMISSION',
             help='submission tarball')
         parser.add_argument(
-            '-o', '--output-file', metavar='FILE',
+            '-o', '--output-file', metavar='FILE',  required=True,
             help='save combined test results to the specified FILE')
         parser.add_argument(
             '--title', action='store', metavar='SESSION_NAME',