Refactor the code of osccore-qa-testing-tools to comply with PEP8.
Related-prod: PRODX-42195
Change-Id: Id05e7584e0d024127ce1bd5042cfe681a1b52e2d
diff --git a/update_testrail_xml/cmd.py b/update_testrail_xml/cmd.py
index bec9097..ea7a5f3 100644
--- a/update_testrail_xml/cmd.py
+++ b/update_testrail_xml/cmd.py
@@ -9,13 +9,13 @@
from reporter import Reporter
-warnings.simplefilter('always', DeprecationWarning)
+warnings.simplefilter("always", DeprecationWarning)
logger = logging.getLogger(__name__)
if sys.version_info[0] == 3:
str_cls = str
else:
- str_cls = eval('unicode')
+ str_cls = eval("unicode")
def filename(string):
@@ -30,140 +30,164 @@
def parse_args(args):
defaults = {
- 'TESTRAIL_URL': 'https://mirantis.testrail.com',
- 'TESTRAIL_USER': 'user@example.com',
- 'TESTRAIL_PASSWORD': 'password',
- 'TESTRAIL_PROJECT': 'Mirantis OpenStack',
- 'TESTRAIL_MILESTONE': '9.0',
- 'TESTRAIL_TEST_SUITE': '[{0.testrail_milestone}] MOSQA',
- 'XUNIT_REPORT': 'report.xml',
- 'OUTPUT_XUNIT_REPORT': 'output_report.xml',
- 'XUNIT_NAME_TEMPLATE': '{id}',
- 'TESTRAIL_NAME_TEMPLATE': '{custom_report_label}',
- 'TRACE_LEN': 100,
- 'ISO_ID': None,
- 'TESTRAIL_PLAN_NAME': None,
- 'ENV_DESCRIPTION': '',
- 'TEST_RESULTS_LINK': '',
- 'PASTE_BASE_URL': None
+ "TESTRAIL_URL": "https://mirantis.testrail.com",
+ "TESTRAIL_USER": "user@example.com",
+ "TESTRAIL_PASSWORD": "password",
+ "TESTRAIL_PROJECT": "Mirantis OpenStack",
+ "TESTRAIL_MILESTONE": "9.0",
+ "TESTRAIL_TEST_SUITE": "[{0.testrail_milestone}] MOSQA",
+ "XUNIT_REPORT": "report.xml",
+ "OUTPUT_XUNIT_REPORT": "output_report.xml",
+ "XUNIT_NAME_TEMPLATE": "{id}",
+ "TESTRAIL_NAME_TEMPLATE": "{custom_report_label}",
+ "TRACE_LEN": 100,
+ "ISO_ID": None,
+ "TESTRAIL_PLAN_NAME": None,
+ "ENV_DESCRIPTION": "",
+ "TEST_RESULTS_LINK": "",
+ "PASTE_BASE_URL": None,
}
defaults = {k: os.environ.get(k, v) for k, v in defaults.items()}
- parser = argparse.ArgumentParser(description='xUnit to testrail reporter')
+ parser = argparse.ArgumentParser(description="xUnit to testrail reporter")
parser.add_argument(
- 'xunit_report',
+ "xunit_report",
type=filename,
- default=defaults['XUNIT_REPORT'],
- help='xUnit report XML file')
+ default=defaults["XUNIT_REPORT"],
+ help="xUnit report XML file",
+ )
parser.add_argument(
- '--output-xunit-report',
+ "--output-xunit-report",
type=str_cls,
- default=defaults['OUTPUT_XUNIT_REPORT'],
- help='Output xUnit report XML file after update')
+ default=defaults["OUTPUT_XUNIT_REPORT"],
+ help="Output xUnit report XML file after update",
+ )
parser.add_argument(
- '--xunit-name-template',
+ "--xunit-name-template",
type=str_cls,
- default=defaults['XUNIT_NAME_TEMPLATE'],
- help='template for xUnit cases to make id string')
+ default=defaults["XUNIT_NAME_TEMPLATE"],
+ help="template for xUnit cases to make id string",
+ )
parser.add_argument(
- '--testrail-name-template',
+ "--testrail-name-template",
type=str_cls,
- default=defaults['TESTRAIL_NAME_TEMPLATE'],
- help='template for TestRail cases to make id string')
+ default=defaults["TESTRAIL_NAME_TEMPLATE"],
+ help="template for TestRail cases to make id string",
+ )
parser.add_argument(
- '--env-description',
+ "--env-description",
type=str_cls,
- default=defaults['ENV_DESCRIPTION'],
- help='env deploy type description (for TestRun name)')
+ default=defaults["ENV_DESCRIPTION"],
+ help="env deploy type description (for TestRun name)",
+ )
group = parser.add_mutually_exclusive_group()
group.add_argument(
- '--iso-id',
+ "--iso-id",
type=str_cls,
- default=defaults['ISO_ID'],
- help='id of build Fuel iso (DEPRECATED)')
+ default=defaults["ISO_ID"],
+ help="id of build Fuel iso (DEPRECATED)",
+ )
group.add_argument(
- '--testrail-plan-name',
+ "--testrail-plan-name",
type=str_cls,
- default=defaults['TESTRAIL_PLAN_NAME'],
- help='name of test plan to be displayed in testrail')
+ default=defaults["TESTRAIL_PLAN_NAME"],
+ help="name of test plan to be displayed in testrail",
+ )
parser.add_argument(
- '--test-results-link',
+ "--test-results-link",
type=str_cls,
- default=defaults['TEST_RESULTS_LINK'],
- help='link to test job results')
+ default=defaults["TEST_RESULTS_LINK"],
+ help="link to test job results",
+ )
parser.add_argument(
- '--testrail-url',
+ "--testrail-url",
type=str_cls,
- default=defaults['TESTRAIL_URL'],
- help='base url of testrail')
+ default=defaults["TESTRAIL_URL"],
+ help="base url of testrail",
+ )
parser.add_argument(
- '--testrail-user',
+ "--testrail-user",
type=str_cls,
- default=defaults['TESTRAIL_USER'],
- help='testrail user')
+ default=defaults["TESTRAIL_USER"],
+ help="testrail user",
+ )
parser.add_argument(
- '--testrail-password',
+ "--testrail-password",
type=str_cls,
- default=defaults['TESTRAIL_PASSWORD'],
- help='testrail password')
+ default=defaults["TESTRAIL_PASSWORD"],
+ help="testrail password",
+ )
parser.add_argument(
- '--testrail-project',
+ "--testrail-project",
type=str_cls,
- default=defaults['TESTRAIL_PROJECT'],
- help='testrail project name')
+ default=defaults["TESTRAIL_PROJECT"],
+ help="testrail project name",
+ )
parser.add_argument(
- '--testrail-milestone',
+ "--testrail-milestone",
type=str_cls,
- default=defaults['TESTRAIL_MILESTONE'],
- help='testrail project milestone')
+ default=defaults["TESTRAIL_MILESTONE"],
+ help="testrail project milestone",
+ )
parser.add_argument(
- '--testrail-suite',
+ "--testrail-suite",
type=str_cls,
- default=defaults['TESTRAIL_TEST_SUITE'],
- help='testrail project suite name')
+ default=defaults["TESTRAIL_TEST_SUITE"],
+ help="testrail project suite name",
+ )
parser.add_argument(
- '--trace-len',
+ "--trace-len",
type=int,
- default=defaults['TRACE_LEN'],
- help='testrail project name')
+ default=defaults["TRACE_LEN"],
+ help="testrail project name",
+ )
parser.add_argument(
- '--send-skipped',
- action='store_true',
+ "--send-skipped",
+ action="store_true",
default=False,
- help='send skipped cases to testrail')
+ help="send skipped cases to testrail",
+ )
parser.add_argument(
- '--send-duplicates',
- action='store_true',
+ "--send-duplicates",
+ action="store_true",
default=False,
- help='send duplicated cases to testrail')
+ help="send duplicated cases to testrail",
+ )
parser.add_argument(
- '--paste-url',
+ "--paste-url",
type=str_cls,
- default=defaults['PASTE_BASE_URL'],
- help=('pastebin service JSON API URL to send test case logs and trace,'
- ' example: http://localhost:5000/'))
+ default=defaults["PASTE_BASE_URL"],
+ help=(
+ "pastebin service JSON API URL to send test case logs and trace,"
+ " example: http://localhost:5000/"
+ ),
+ )
parser.add_argument(
- '--testrail-run-update',
- dest='use_test_run_if_exists',
- action='store_true',
+ "--testrail-run-update",
+ dest="use_test_run_if_exists",
+ action="store_true",
default=False,
- help='don\'t create new test run if such already exists')
+ help="don't create new test run if such already exists",
+ )
parser.add_argument(
- '--dry-run', '-n',
- action='store_true',
+ "--dry-run",
+ "-n",
+ action="store_true",
default=False,
- help='Just print mapping table')
+ help="Just print mapping table",
+ )
parser.add_argument(
- '--verbose',
- '-v',
- action='store_true',
+ "--verbose",
+ "-v",
+ action="store_true",
default=False,
- help='Verbose mode')
+ help="Verbose mode",
+ )
return parser.parse_args(args)
@@ -175,16 +199,19 @@
args = parse_args(args)
if not args.testrail_plan_name:
- args.testrail_plan_name = ('{0.testrail_milestone} iso '
- '#{0.iso_id}').format(args)
+ args.testrail_plan_name = (
+ "{0.testrail_milestone} iso " "#{0.iso_id}"
+ ).format(args)
- msg = ("--iso-id parameter is DEPRECATED. "
- "It is recommended to use --testrail-plan-name parameter.")
+ msg = (
+ "--iso-id parameter is DEPRECATED. "
+ "It is recommended to use --testrail-plan-name parameter."
+ )
warnings.warn(msg, DeprecationWarning)
logger_dict = dict(stream=sys.stderr)
if args.verbose:
- logger_dict['level'] = logging.DEBUG
+ logger_dict["level"] = logging.DEBUG
logging.basicConfig(**logger_dict)
trace_len = args.trace_len
@@ -194,7 +221,8 @@
output_xunit_report=args.output_xunit_report,
env_description=args.env_description,
test_results_link=args.test_results_link,
- paste_url=args.paste_url)
+ paste_url=args.paste_url,
+ )
suite = args.testrail_suite.format(args)
reporter.config_testrail(
base_url=args.testrail_url,
@@ -204,7 +232,8 @@
tests_suite=suite,
send_skipped=args.send_skipped,
send_duplicates=args.send_duplicates,
- use_test_run_if_exists=args.use_test_run_if_exists)
+ use_test_run_if_exists=args.use_test_run_if_exists,
+ )
empty_classnames = reporter.get_empty_classnames()
all_empty_cases = reporter.get_testcases(empty_classnames)
@@ -215,7 +244,7 @@
reporter.delete_temporary_file()
-if __name__ == '__main__':
+if __name__ == "__main__":
try:
main()
except Exception: