Fix under and over continuation indentation

Fix Flake8 E127 and E128 errors (under and over continuation
indentation)

Reviewed By: cmatthews

Differential Revision: https://reviews.llvm.org/D94723

Author: Thomas Preud'homme

lnt/lnttool/viewcomparison.py

@@ -89,7 +89,7 @@ def action_view_comparison(report_a, report_b, hostname, port, dry_run,
         with contextlib.closing(config.get_database('default')) as db:
             session = db.make_session()
             import_and_report(config, 'default', db, session, report_a,
-                                  '<auto>', testsuite, select_machine='match')
+                              '<auto>', testsuite, select_machine='match')
             import_and_report(config, 'default', db, session, report_b,
                               '<auto>', testsuite, select_machine='match')
 

lnt/server/db/testsuitedb.py

@@ -59,7 +59,7 @@ def __init__(self, v4db, name, test_suite):
         self.order_fields = list(self.test_suite.order_fields)
         self.run_fields = list(self.test_suite.run_fields)
         self.sample_fields = list(sorted(self.test_suite.sample_fields,
-            key = lambda s: s.schema_index))
+                                         key = lambda s: s.schema_index))
         self.machine_to_latest_order_cache = {}
         sample_field_indexes = dict()
 

lnt/server/reporting/latestrunsreport.py

@@ -24,7 +24,8 @@ def build(self, session):
             field_results = []
             for machine in machines:
                 machine_results = []
-                machine_runs = list(reversed(session.query(ts.Run)
+                machine_runs = list(reversed(
+                    session.query(ts.Run)
                     .filter(ts.Run.machine_id == machine.id)
                     .order_by(ts.Run.start_time.desc())
                     .limit(self.run_count)
@@ -38,12 +39,12 @@ def build(self, session):
                 # take all tests from latest run and do a comparison
                 oldest_run = machine_runs[0]
 
-                run_tests = session.query(ts.Test) \
-                        .join(ts.Sample) \
-                        .join(ts.Run) \
-                        .filter(ts.Sample.run_id == oldest_run.id) \
-                        .filter(ts.Sample.test_id == ts.Test.id) \
-                        .all()
+                run_tests = (session.query(ts.Test)
+                             .join(ts.Sample)
+                             .join(ts.Run)
+                             .filter(ts.Sample.run_id == oldest_run.id)
+                             .filter(ts.Sample.test_id == ts.Test.id)
+                             .all())
 
                 # Create a run info object.
                 sri = lnt.server.reporting.analysis.RunInfo(session, ts, machine_runs_ids)

lnt/server/reporting/report.py

@@ -126,9 +126,9 @@ def complete(self):
 
 report_css_styles = {
         "body": ("color:#000000; background-color:#ffffff; "
-            "font-family: Helvetica, sans-serif; font-size:9pt"),
+                 "font-family: Helvetica, sans-serif; font-size:9pt"),
         "table": ("font-size:9pt; border-spacing: 0px; "
-            "border: 1px solid black"),
+                  "border: 1px solid black"),
         "th": (
             "background-color:#eee; color:#666666; font-weight: bold; "
             "cursor: default; text-align:center; font-weight: bold; "

lnt/server/ui/regression_views.py

@@ -60,7 +60,7 @@ def v4_new_regressions():
         regression, _ = new_regression(session, ts, form.field_changes.data)
         flash("Created " + regression.title, FLASH_SUCCESS)
         return v4_redirect(v4_url_for(".v4_regression_list",
-                        highlight=regression.id))
+                                      highlight=regression.id))
     if request.method == 'POST' and request.form['btn'] == "Ignore Changes":
         msg = "Ignoring changes: "
         ignored = []
@@ -158,7 +158,7 @@ def v4_regression_list():
                 links.append(r.bug)
 
         new_regress, _ = new_regression(session, ts,
-                                     [x.field_change_id for x in reg_inds])
+                                        [x.field_change_id for x in reg_inds])
         new_regress.state = regressions[target].state
         new_regress.title = regressions[target].title
         new_regress.bug = ' '.join(links)
@@ -298,8 +298,8 @@ def v4_regression_detail(id):
         session.commit()
         flash("Updated " + regression_info.title, FLASH_SUCCESS)
         return v4_redirect(v4_url_for(".v4_regression_list",
-                        highlight=regression_info.id,
-                        state=int(form.edit_state.data)))
+                                      highlight=regression_info.id,
+                                      state=int(form.edit_state.data)))
     if request.method == 'POST' and \
             request.form['save_btn'] == "Split Regression":
         # For each of the regression indicators, grab their field ids.
@@ -318,8 +318,8 @@ def v4_regression_detail(id):
         session.commit()
         flash("Split " + second_regression.title, FLASH_SUCCESS)
         return v4_redirect(v4_url_for(".v4_regression_list",
-                        highlight=second_regression.id,
-                        state=int(form.edit_state.data)))
+                                      highlight=second_regression.id,
+                                      state=int(form.edit_state.data)))
     if request.method == 'POST' and request.form['save_btn'] == "Delete":
         # For each of the regression indicators, grab their field ids.
         title = regression_info.title
@@ -334,7 +334,7 @@ def v4_regression_detail(id):
         session.commit()
         flash("Deleted " + title, FLASH_SUCCESS)
         return v4_redirect(v4_url_for(".v4_regression_list",
-                        state=int(form.edit_state.data)))
+                                      state=int(form.edit_state.data)))
     form.field_changes.choices = list()
     form.state.default = regression_info.state
     form.process()

lnt/server/ui/views.py

@@ -288,7 +288,7 @@ def v4_machine_compare(machine_id):
         .first()
 
     return v4_redirect(v4_url_for('.v4_run', id=machine_1_run.id,
-                               compare_to=machine_2_run.id))
+                                  compare_to=machine_2_run.id))
 
 
 @v4_route("/machine/<int:id>")
@@ -468,7 +468,7 @@ def simple_run(tag, id):
     # If we found one, redirect to it's report.
     if matched_run is not None:
         return v4_redirect(db_url_for(".v4_run", testsuite_name=tag,
-                                   id=matched_run.id))
+                                      id=matched_run.id))
 
     # Otherwise, report an error.
     return render_template("error.html", message="""\
@@ -1421,8 +1421,8 @@ def v4_daily_report_overview():
     extra_args.pop("day", None)
 
     return v4_redirect(v4_url_for(".v4_daily_report",
-                               year=date.year, month=date.month, day=date.day,
-                               **extra_args))
+                                  year=date.year, month=date.month,
+                                  day=date.day, **extra_args))
 
 
 @v4_route("/daily_report/<int:year>/<int:month>/<int:day>")

lnt/tests/nt.py

@@ -1716,7 +1716,7 @@ def run_test(self, opts):
 
             for i in range(opts.multisample):
                 print("%s: (multisample) running iteration %d" %\
-                        (timestamp(), i), file=sys.stderr)
+                      (timestamp(), i), file=sys.stderr)
                 report = run_test(opts.label, i, config)
                 reports.append(report)
 

tests/server/db/ImportV4TestSuiteInstance.py

@@ -134,9 +134,9 @@
 assert sorted(run_b.parameters.items()) == [('inferred_run_order', '2')]
 
 # Validate the samples.
-samples = list(session.query(ts.Sample)\
-    .join(ts.Run) \
-    .order_by(ts.Run.order_id, ts.Sample.id))
+samples = list(session.query(ts.Sample)
+               .join(ts.Run)
+               .order_by(ts.Run.order_id, ts.Sample.id))
 assert len(samples) == 3
 sample_a_0, sample_a_1, sample_b = samples
 assert sample_a_0.run is run_a