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/views.py b/testrail_bot/control/views.py
index e6655ad..8a362ef 100644
--- a/testrail_bot/control/views.py
+++ b/testrail_bot/control/views.py
@@ -1,16 +1,18 @@
import datetime
import json
import os
-from .celery_tasks import test_rail_api
-from django.shortcuts import render, redirect, HttpResponse
+from django.shortcuts import HttpResponse, redirect, render
from django_celery_beat.models import CrontabSchedule, PeriodicTasks
-from . import models
-from . import forms
-from .celery_tasks.tasks import process_run, update_plot_data, \
- get_test_passability_in_suite
-from .utils import short_names_for_dict, get_dict_diff
+from . import forms, models
+from .celery_tasks import test_rail_api
+from .celery_tasks.tasks import (
+ get_test_passability_in_suite,
+ process_run,
+ update_plot_data,
+)
+from .utils import get_dict_diff, short_names_for_dict
def index(request):
@@ -32,9 +34,11 @@
else:
form = forms.TestRunForm(instance=run)
- return render(request, "control/update_run.html",
- {"form": form, "run_id": run_id, "checked_tests":
- run.checked_tests})
+ return render(
+ request,
+ "control/update_run.html",
+ {"form": form, "run_id": run_id, "checked_tests": run.checked_tests},
+ )
def create_run(request):
@@ -46,8 +50,9 @@
else:
form = forms.TestRunForm()
- form.fields["created_after"].initial = datetime.date.today() + \
- datetime.timedelta(days=-3 * 30)
+ form.fields["created_after"].initial = (
+ datetime.date.today() + datetime.timedelta(days=-3 * 30)
+ )
form.fields["created_before"].initial = datetime.date.today()
return render(request, "control/update_run.html", {"form": form})
@@ -60,20 +65,26 @@
def single_report(request, report_id):
report = models.TestRailReport.objects.get(pk=report_id)
data = report.path.read().decode("utf-8")
- if request.method == "POST" \
- and request.META.get('HTTP_X_REQUESTED_WITH') == 'XMLHttpRequest':
+ if (
+ request.method == "POST"
+ and request.META.get("HTTP_X_REQUESTED_WITH") == "XMLHttpRequest"
+ ):
return HttpResponse(
json.dumps({"data": data, "finished": report.finished}),
- content_type="application/json")
+ content_type="application/json",
+ )
- return render(request, "control/report.html",
- {"report_id": report.id,
- "report": data,
- "finished": report.finished})
+ return render(
+ request,
+ "control/report.html",
+ {"report_id": report.id, "report": data, "finished": report.finished},
+ )
def delete_report(request, report_id):
- report:models.TestRailReport = models.TestRailReport.objects.get(pk=report_id)
+ report: models.TestRailReport = models.TestRailReport.objects.get(
+ pk=report_id
+ )
try:
os.remove(report.path.path)
except FileNotFoundError:
@@ -94,8 +105,9 @@
_name = f"Plan {testrail_run['name']}"
else:
parent_plan_id = testrail_run["plan_id"]
- parent_plan_name = \
- test_rail_api.get_plan_by_id(parent_plan_id)["name"]
+ parent_plan_name = test_rail_api.get_plan_by_id(parent_plan_id)[
+ "name"
+ ]
_name = f"Run {testrail_run['name']} from {parent_plan_name}"
run.run_name = _name
run.save()
@@ -103,14 +115,13 @@
report_name = "{}-run_id-{}-date-{}".format(
run.run_name,
run.run_id,
- datetime.datetime.isoformat(datetime.datetime.now()))
+ datetime.datetime.isoformat(datetime.datetime.now()),
+ )
path = os.path.join(models.fs.location, report_name)
with open(path, "w"):
pass
- report = models.TestRailReport(
- report_name=report_name,
- path=path)
+ report = models.TestRailReport(report_name=report_name, path=path)
report.save()
process_run.delay(run_id, report.id, path, is_testplan)
return redirect("single_report", report.id)
@@ -133,7 +144,8 @@
except models.ActionLog.DoesNotExist:
pass
update = models.ActionLog(
- name="update_plot_started", date=datetime.datetime.now())
+ name="update_plot_started", date=datetime.datetime.now()
+ )
update.save()
update_plot_data.delay()
return HttpResponse("Started Update", status=200)
@@ -142,7 +154,8 @@
def jenkins_plot(request):
try:
update_date = models.ActionLog.objects.get(
- name="update_jenkins_plot").date
+ name="update_jenkins_plot"
+ ).date
except models.ActionLog.DoesNotExist:
update_date = None
try:
@@ -157,13 +170,18 @@
try:
with open(job_names_path, "r") as f:
job_names = json.load(f)
- except:
+ except Exception:
pass
return render(
- request, "control/jenkins_plot.html",
- {"update_date": update_date, "update_started": update_started,
- "job_names": enumerate(job_names, 1)})
+ request,
+ "control/jenkins_plot.html",
+ {
+ "update_date": update_date,
+ "update_started": update_started,
+ "job_names": enumerate(job_names, 1),
+ },
+ )
def submit_suites(request):
@@ -180,7 +198,7 @@
report1=report1,
report2=report2,
limit=form.cleaned_data["limit"],
- test_keyword=form.cleaned_data["test_keyword"]
+ test_keyword=form.cleaned_data["test_keyword"],
)
diff_model.save()
get_test_passability_in_suite.delay(diff_model.id, report1.id)
@@ -194,60 +212,72 @@
return submit_suites(request)
diff_form = forms.DiffPassRatesForm()
- report1_form = forms.SuitePassRateForm(prefix='first')
- report2_form = forms.SuitePassRateForm(prefix='second')
+ report1_form = forms.SuitePassRateForm(prefix="first")
+ report2_form = forms.SuitePassRateForm(prefix="second")
return render(
- request, "control/compare_suites.html",
+ request,
+ "control/compare_suites.html",
{
- "diff_form": diff_form,
- "report1_form": report1_form,
- "report2_form": report2_form,
- "finished": None
- })
+ "diff_form": diff_form,
+ "report1_form": report1_form,
+ "report2_form": report2_form,
+ "finished": None,
+ },
+ )
def list_of_comparing_reports(request):
list_of_reports = models.DiffOfSuitesPassRates.objects.all()
return render(
- request, "control/list_comparing_suites.html",
- {
- "reports": list_of_reports
- })
+ request,
+ "control/list_comparing_suites.html",
+ {"reports": list_of_reports},
+ )
def report_comparing_suites(request, report_id):
report = models.DiffOfSuitesPassRates.objects.get(pk=report_id)
- passrate1 = short_names_for_dict(json.loads(
- report.report1.passrate_by_tests))
- passrate2 = short_names_for_dict(json.loads(
- report.report2.passrate_by_tests))
+ passrate1 = short_names_for_dict(
+ json.loads(report.report1.passrate_by_tests)
+ )
+ passrate2 = short_names_for_dict(
+ json.loads(report.report2.passrate_by_tests)
+ )
- diff_table = get_dict_diff(dict1=passrate1,
- dict2=passrate2,
- compare_by_key="rate")
+ diff_table = get_dict_diff(
+ dict1=passrate1, dict2=passrate2, compare_by_key="rate"
+ )
diff_form = forms.DiffPassRatesForm(instance=report)
- report1_form = forms.SuitePassRateForm(instance=report.report1,
- prefix="first")
- report2_form = forms.SuitePassRateForm(instance=report.report2,
- prefix="second")
+ report1_form = forms.SuitePassRateForm(
+ instance=report.report1, prefix="first"
+ )
+ report2_form = forms.SuitePassRateForm(
+ instance=report.report2, prefix="second"
+ )
return render(
- request, "control/compare_suites.html",
- {"diff_form": diff_form,
- "report1_form": report1_form,
- "report2_form": report2_form,
- "report1": report.report1,
- "report2": report.report2,
- "is_finished": report.report1.finished and report.report2.finished,
- "diff_table": diff_table})
+ request,
+ "control/compare_suites.html",
+ {
+ "diff_form": diff_form,
+ "report1_form": report1_form,
+ "report2_form": report2_form,
+ "report1": report.report1,
+ "report2": report.report2,
+ "is_finished": report.report1.finished and report.report2.finished,
+ "diff_table": diff_table,
+ },
+ )
def schedulers(request):
return render(
- request, "control/schedulers.html",
- {"schedulers": models.CronPeriodicTask.objects.all()})
+ request,
+ "control/schedulers.html",
+ {"schedulers": models.CronPeriodicTask.objects.all()},
+ )
def scheduler(request, pk=None):
@@ -261,33 +291,31 @@
form = forms.PeriodicTaskForm()
return render(
- request, "control/scheduler.html",
- {
- "form": form,
- "pk": pk,
- "TASKS": models.TASKS
- }
+ request,
+ "control/scheduler.html",
+ {"form": form, "pk": pk, "TASKS": models.TASKS},
)
def save_scheduler(request, pk=None):
print(f"{request.POST=}")
- minute, hour, day_of_month, month_of_year, day_of_week = \
- request.POST.get("cron", "* * * * *").split(" ")
+ minute, hour, day_of_month, month_of_year, day_of_week = request.POST.get(
+ "cron", "* * * * *"
+ ).split(" ")
if pk is None:
sch = CrontabSchedule.objects.create(
minute=minute,
hour=hour,
day_of_month=day_of_month,
month_of_year=month_of_year,
- day_of_week=day_of_week
+ day_of_week=day_of_week,
)
task = models.CronPeriodicTask.objects.create(
crontab=sch,
cron=request.POST.get("cron"),
name=request.POST.get("name"),
task_name=request.POST.get("task_name"),
- enabled=request.POST.get("enabled") == 'on',
+ enabled=request.POST.get("enabled") == "on",
testplan_id_arg=request.POST.get("testplan_id_arg"),
)
else:
@@ -303,7 +331,7 @@
hour=hour,
day_of_month=day_of_month,
month_of_year=month_of_year,
- day_of_week=day_of_week
+ day_of_week=day_of_week,
)
if not form.is_valid():
print(f"{form.errors=}")
@@ -311,12 +339,9 @@
form.save()
PeriodicTasks.update_changed()
return render(
- request, "control/scheduler.html",
- {
- "form": form,
- "pk": task.id,
- "TASKS": models.TASKS
- }
+ request,
+ "control/scheduler.html",
+ {"form": form, "pk": task.id, "TASKS": models.TASKS},
)