Add additional test cases in response to PR feedback

Author: abiola-aws

enableDetective.py

@@ -303,7 +303,7 @@ def enable_detective(d_client: botocore.client.BaseClient, region: str, tags: di
         confirm = input('Should Amazon Detective be enabled in {}? Enter [Y/N]: '.format(region))
 
         if confirm == 'Y' or confirm == 'y':
-            logging.info(f'Enabling Amazon Detective in {region}' + (f'with tags {tags}' if tags else ''))
+            logging.info(f'Enabling Amazon Detective in {region}' + (f' with tags {tags}' if tags else ''))
             graphs = [d_client.create_graph(Tags=tags)['GraphArn']]
         else:
             logging.info(f'Skipping {region}')

tests/test_scripts.py

@@ -26,11 +26,13 @@ def test_setup_command_line_enableDetective():
     assert args.master_account == '000000000001'
     assert args.tags == None
 
-    args = enableDetective.setup_command_line("--master_account 123456789012 --assume_role detectiveAdmin --input_file accounts.csv --tags TagKey1=TagValue1,TagKey2=TagValue2,TagKey3=TagValue3".split(" "))
+    args = enableDetective.setup_command_line("--master_account 123456789012 --assume_role detectiveAdmin --input_file accounts.csv --tags TagKey1=TagValue1,TagKey2=TagValue2,TagKey3=TagValue3,TagKey4=,TagKey5=TagValue5".split(" "))
     assert args.tags == {
         "TagKey1": "TagValue1",
         "TagKey2": "TagValue2",
         "TagKey3": "TagValue3",
+        "TagKey4": "",
+        "TagKey5": "TagValue5",
     }
 
     # Wrong master account