Refactor the code of osccore-qa-testing-tools to comply with PEP8.
Related-prod: PRODX-42195
Change-Id: Id05e7584e0d024127ce1bd5042cfe681a1b52e2d
diff --git a/testrail_bot/control/models.py b/testrail_bot/control/models.py
index a8a0910..b438dec 100644
--- a/testrail_bot/control/models.py
+++ b/testrail_bot/control/models.py
@@ -6,28 +6,28 @@
class IntegerListField(models.Field):
def __init__(self, *args, **kwargs):
- kwargs['editable'] = False
+ kwargs["editable"] = False
super(IntegerListField, self).__init__(*args, **kwargs)
def db_type(self, connection):
- return 'text'
+ return "text"
def from_db_value(self, value, expression, connection):
if not value:
return []
- return [int(x) for x in value.split(',')]
+ return [int(x) for x in value.split(",")]
def to_python(self, value):
if isinstance(value, list):
return value
if not value:
return []
- return [int(x) for x in value.split(',')]
+ return [int(x) for x in value.split(",")]
def get_prep_value(self, value):
if not value:
- return ''
- return ','.join(str(int(x)) for x in value)
+ return ""
+ return ",".join(str(int(x)) for x in value)
def default_created_after():
@@ -35,15 +35,16 @@
class TestRailTestRun(models.Model):
- project_name = models.CharField(max_length=300,
- default="Mirantis Cloud Platform")
+ project_name = models.CharField(
+ max_length=300, default="Mirantis Cloud Platform"
+ )
plan_name = models.CharField(max_length=300, default="[MCP2.0]OSCORE")
run_name = models.CharField(max_length=300, blank=True)
testrun_pattern = models.CharField(max_length=300, blank=True)
run_id = models.CharField(max_length=300)
checked_tests = IntegerListField(default=list())
caching_tests_enabled = models.BooleanField(default=False)
- created_by_id = models.IntegerField(default='109')
+ created_by_id = models.IntegerField(default="109")
filter_func = models.TextField(null=True, blank=True)
ip_filter = models.BooleanField(default=True)
uuid_filter = models.BooleanField(default=True)
@@ -91,26 +92,32 @@
class SuitePassRate(models.Model):
SUITE_CHOICES = [
- ("Tempest", (
- ("10650", "[MCP2.0_QUEENS]Tempest"),
- ("10651", "[MCP2.0_ROCKY]Tempest"),
- ("10635", "[MCP2.0_STEIN]Tempest"),
- ("10653", "[MCP2.0_TRAIN]Tempest"),
- ("10710", "[MCP2.0_USSURI]Tempest"),
- ("10888", "[MCP2.0_VICTORIA]Tempest"),
- ("11167", "[MCP2.0_WALLABY]Tempest"),
- ("11188", "[MCP2.0_XENA]Tempest"),
- ("11170", "[MCP2.0_YOGA]Tempest"),
- ("11192", "[MCP2.0_ANTELOPE]Tempest"),
- ("11195", "[MCP2.0_CARACAL]Tempest"))
- ),
- ("Stepler", (
- ("10886", "[MCP2.0_USSURI]Stepler"),
- ("10887", "[MCP2.0_VICTORIA]Stepler"),
- ("11171", "[MCP2.0_YOGA]Stepler"),
- ("11193", "[MCP2.0_ANTELOPE]Stepler"),
- ("11196", "[MCP2.0_CARACAL]Stepler"))
- ),
+ (
+ "Tempest",
+ (
+ ("10650", "[MCP2.0_QUEENS]Tempest"),
+ ("10651", "[MCP2.0_ROCKY]Tempest"),
+ ("10635", "[MCP2.0_STEIN]Tempest"),
+ ("10653", "[MCP2.0_TRAIN]Tempest"),
+ ("10710", "[MCP2.0_USSURI]Tempest"),
+ ("10888", "[MCP2.0_VICTORIA]Tempest"),
+ ("11167", "[MCP2.0_WALLABY]Tempest"),
+ ("11188", "[MCP2.0_XENA]Tempest"),
+ ("11170", "[MCP2.0_YOGA]Tempest"),
+ ("11192", "[MCP2.0_ANTELOPE]Tempest"),
+ ("11195", "[MCP2.0_CARACAL]Tempest"),
+ ),
+ ),
+ (
+ "Stepler",
+ (
+ ("10886", "[MCP2.0_USSURI]Stepler"),
+ ("10887", "[MCP2.0_VICTORIA]Stepler"),
+ ("11171", "[MCP2.0_YOGA]Stepler"),
+ ("11193", "[MCP2.0_ANTELOPE]Stepler"),
+ ("11196", "[MCP2.0_CARACAL]Stepler"),
+ ),
+ ),
]
suite_id = models.CharField(max_length=20, choices=SUITE_CHOICES)
suite_name = models.CharField(max_length=100, blank=True)
@@ -122,38 +129,42 @@
class DiffOfSuitesPassRates(models.Model):
limit = models.IntegerField(default=10, blank=True)
test_keyword = models.CharField(default="", max_length=300, blank=True)
- report1 = models.ForeignKey(to=SuitePassRate,
- related_name="report1",
- on_delete=models.CASCADE,
- blank=True)
- report2 = models.ForeignKey(to=SuitePassRate,
- related_name="report2",
- on_delete=models.CASCADE,
- blank=True)
- started_at = models.DateTimeField(auto_created=True,
- auto_now=True)
+ report1 = models.ForeignKey(
+ to=SuitePassRate,
+ related_name="report1",
+ on_delete=models.CASCADE,
+ blank=True,
+ )
+ report2 = models.ForeignKey(
+ to=SuitePassRate,
+ related_name="report2",
+ on_delete=models.CASCADE,
+ blank=True,
+ )
+ started_at = models.DateTimeField(auto_created=True, auto_now=True)
TASKS = [
- ("control.celery_tasks.tasks.check_today_testplan",
- "Check today testplan",
- []
- ),
- ("control.celery_tasks.tasks.check_specific_testplan",
- "Check specific testplan",
- ["testplan_id_arg"]
- ),
- ]
+ (
+ "control.celery_tasks.tasks.check_today_testplan",
+ "Check today testplan",
+ [],
+ ),
+ (
+ "control.celery_tasks.tasks.check_specific_testplan",
+ "Check specific testplan",
+ ["testplan_id_arg"],
+ ),
+]
-TASK_CHOICES = list(map(lambda x: x[:-1], TASKS))
+TASK_CHOICES = list(map(lambda x: x[:-1], TASKS))
class CronPeriodicTask(PeriodicTask):
- cron = models.CharField(default="",
- max_length=300,
- blank=False)
- task_name = models.CharField(max_length=300, choices=TASK_CHOICES,
- default=TASK_CHOICES[0][0])
+ cron = models.CharField(default="", max_length=300, blank=False)
+ task_name = models.CharField(
+ max_length=300, choices=TASK_CHOICES, default=TASK_CHOICES[0][0]
+ )
testplan_id_arg = models.CharField(max_length=30, blank=True, null=True)
class Meta: