Add reporter flag

Author: NamelessOne91

jenkins_pipelines/scripts/bsc_list_generator/bsc_finder.py

@@ -38,12 +38,13 @@ def parse_cli_args() -> argparse.Namespace:
     parser.add_argument("-p", "--product-version", dest="product_version", help="Product version of SUMA you want to run this script for, the options are 4.3 and 5.0. The default is 4.3 for now",
         choices=_PRODUCT_VERSIONS, default="4.3", action='store'
     )
-    parser.add_argument("-n", "--release-notes", dest="use_release", default=False, help="Obrain the bugs list from the release notes for the specified SUMA version", action="store_true")
+    parser.add_argument("-n", "--release-notes", dest="use_release", default=False, help="Obtain the bugs list from the release notes for the specified SUMA version", action="store_true")
     parser.add_argument("-c", "--cloud", dest="cloud", default=False, help="Return BSCs for SUMA in Public Clouds", action="store_true")
     parser.add_argument("-s", "--status", dest="status", help="Status to filter BSCs by", action="store",
         choices=["NEW", "CONFIRMED", "IN_PROGRESS", "RESOLVED"]
     )
     parser.add_argument("-r", "--resolution", dest="resolution", help="Resolution to filter issues for (default empty string means open bugs)", action="store")
+    parser.add_argument("-t", "--reporter", dest="reporter", help="Email address of the person who opened the bug to filter issues for", action="store")
     # output related flags
     parser.add_argument("-o", "--output", dest="output_file", help="File in which the results will be saved", action="store")
     parser.add_argument("-f", "--format", dest="output_format", default="txt", help="Output file format (txt default)", action="store",
@@ -104,10 +105,10 @@ def main():
         product_versions: list[str] = _PRODUCT_VERSIONS if args.all else [args.product_version]
         for version in product_versions:
             release_notes_paths: tuple[tuple[str, str, str]] =_IBS_RELEASE_NOTES_FOR_SUMA_VERSION[version]
-            product_bugs[f"SUSE Manager {version}"] = bugzilla_client.bscs_from_release_notes(release_notes_paths, status = args.status, resolution = args.resolution)
+            product_bugs[f"SUSE Manager {version}"] = bugzilla_client.bscs_from_release_notes(release_notes_paths, reporter = args.reporter, status = args.status, resolution = args.resolution)
     else:
         bugzilla_products: list[str] = get_suma_bugzilla_products(args.all, args.product_version, args.cloud)
-        product_bugs = bugzilla_client.find_suma_bscs(bugzilla_products, status = args.status, resolution = args.resolution)
+        product_bugs = bugzilla_client.find_suma_bscs(bugzilla_products, reporter = args.reporter, status = args.status, resolution = args.resolution)
 
     output_format: str = args.output_format
     output_file: str = args.output_file if args.output_file else _FORMATS_DEFAULT_FILE_NAMES[output_format]

jenkins_pipelines/scripts/tests/test_bsc_finder.py

@@ -98,12 +98,16 @@ def test_parse_cli_args_default_values(self):
         self.assertFalse(args.cloud)
         self.assertIsNone(args.status)
         self.assertIsNone(args.resolution)
+        self.assertIsNone(args.reporter)
         self.assertIsNone(args.output_file)
         self.assertEqual(args.output_format, "txt")
 
     def test_parse_cli_args_success(self):
         # shorthand flags
-        sys.argv = ['bsc_finder.py', "-k", "test_key", "-a", "-p", "5.0", "-n", "-c", "-s", "CONFIRMED", "-r", "", "-o", "test.json", "-f", "json"]
+        sys.argv = [
+            'bsc_finder.py', "-k", "test_key", "-a", "-p", "5.0", "-n", "-c", "-s", "CONFIRMED",
+            "-r", "", "-t", "tester@suse.com", "-o", "test.json", "-f", "json"
+        ]
         args = parse_cli_args()
 
         self.assertEqual(args.api_key, "test_key")
@@ -113,13 +117,14 @@ def test_parse_cli_args_success(self):
         self.assertTrue(args.cloud)
         self.assertEqual(args.status, "CONFIRMED")
         self.assertEqual(args.resolution, '')
+        self.assertEqual(args.reporter, "tester@suse.com")
         self.assertEqual(args.output_file, "test.json")
         self.assertEqual(args.output_format, "json")
 
         # long flags
         sys.argv = [
             "bsc_finder.py", "--api-key", "test_key", "--all", "--product-version", "5.0", "--release-notes", "--cloud", "-s", "CONFIRMED",
-            "--resolution", "", "--output", "test.json", "--format", "json"
+            "--resolution", "", "--reporter", "tester@suse.com", "--output", "test.json", "--format", "json"
         ]
         args = parse_cli_args()
 
@@ -130,6 +135,7 @@ def test_parse_cli_args_success(self):
         self.assertTrue(args.cloud)
         self.assertEqual(args.status, "CONFIRMED")
         self.assertEqual(args.resolution, '')
+        self.assertEqual(args.reporter, "tester@suse.com")
         self.assertEqual(args.output_file, "test.json")
         self.assertEqual(args.output_format, "json")
 

jenkins_pipelines/scripts/tests/test_bugzilla_client.py

@@ -16,7 +16,7 @@ def setUp(self):
     def test_get_bugs_success(self, mock_api_call):
         mock_api_call.side_effect = mock_requests_get_success
 
-        bugs: list[dict[str, Any]] = self.bugzilla_client._get_bugs(product = "Test Product", status = None, release = None)
+        bugs: list[dict[str, Any]] = self.bugzilla_client._get_bugs(product = "Test Product", reporter = None, status = None, release = None)
         mock_api_call.assert_called_once()
         # check None keys are dropped
         mock_api_call.assert_called_with(self.bugzilla_client._bugs_endpoint, params = {'Bugzilla_api_key': 'test_key', "product": "Test Product"})
@@ -28,7 +28,7 @@ def test_get_bugs_success(self, mock_api_call):
             self.assertEqual(bug['id'], i+1)
 
         # just check the arguments are correctly passed when there's a value
-        self.bugzilla_client._get_bugs(product = "Test Product", status = "CONFIRMED", release = None)
+        self.bugzilla_client._get_bugs(product = "Test Product", reporter = None, status = "CONFIRMED", release = None)
         mock_api_call.assert_called_with(self.bugzilla_client._bugs_endpoint, params = {'Bugzilla_api_key': 'test_key', "product": "Test Product", "status": "CONFIRMED"})
 
     @patch('requests.get')