Merge pull request #19 from okurz/feature/black

Apply current black style suggestions

mtui/commands/downgrade.py

@@ -23,7 +23,6 @@ def _add_arguments(cls, parser) -> None:
 
     @requires_update
     def __call__(self):
-
         targets = self.parse_hosts()
         if not targets:
             raise NoRefhostsDefinedError

mtui/commands/hostsunlock.py

@@ -22,7 +22,6 @@ def _add_arguments(cls, parser):
         return parser
 
     def __call__(self):
-
         hosts = self.parse_hosts()
         hosts.unlock(force=self.args.force)
 

mtui/commands/prepare.py

@@ -44,7 +44,6 @@ def _add_arguments(cls, parser) -> None:
 
     @requires_update
     def __call__(self):
-
         targets = self.parse_hosts()
         if not targets:
             raise NoRefhostsDefinedError

mtui/commands/quit.py

@@ -27,7 +27,6 @@ def _close_target(self, target, args):
         self.targets.pop(target)
 
     def __call__(self):
-
         args_ = [self.args.bootarg] if self.args.bootarg else []
 
         with concurrent.futures.ThreadPoolExecutor() as executor:

mtui/commands/run.py

@@ -30,7 +30,6 @@ def _add_arguments(cls, parser) -> None:
         cls._add_hosts_arg(parser)
 
     def __call__(self):
-
         targets = self.parse_hosts()
         if not targets:
             raise NoRefhostsDefinedError

mtui/commands/setrepo.py

@@ -12,7 +12,6 @@ class SetRepo(Command):
 
     @classmethod
     def _add_arguments(cls, parser) -> None:
-
         group = parser.add_mutually_exclusive_group(required=True)
         group.add_argument(
             "-A",
@@ -36,7 +35,6 @@ def _add_arguments(cls, parser) -> None:
 
     @requires_update
     def __call__(self):
-
         operation = self.args.operation
         hosts = self.parse_hosts()
 

mtui/commands/showdiff.py

@@ -17,7 +17,6 @@ class ShowDiff(Command):
 
     @requires_update
     def __call__(self):
-
         diff = self.metadata.report_wd() / "source.diff"
         text = diff.read_text().split("\n")
         page(text, self.prompt.interactive)
@@ -32,7 +31,6 @@ class AnalyzeDiff(Command):
 
     @requires_update
     def __call__(self):
-
         patchdef = re.compile(r"[+-]Patch(\d*):\s+(.*\.patch)$", flags=re.M)
         patchapply = re.compile(r"[+-]%patch(\d+|)\s+(?:-p\d*|)", flags=re.M)
 

mtui/commands/simplelists.py

@@ -22,7 +22,6 @@ class ListLocks(Command):
     command = "list_locks"
 
     def __call__(self):
-
         self.hosts.select(enabled=True).report_locks(self.display.list_locks)
 
 
@@ -37,7 +36,6 @@ class ListHosts(Command):
     command = "list_hosts"
 
     def __call__(self):
-
         self.targets.report_self(self.display.list_host)
 
 
@@ -49,7 +47,6 @@ class ListTimeout(Command):
     command = "list_timeout"
 
     def __call__(self):
-
         self.targets.report_timeout(self.display.list_timeout)
 
 

mtui/commands/simpleset.py

@@ -20,7 +20,6 @@ class SessionName(Command):
 
     @classmethod
     def _add_arguments(cls, parser) -> None:
-
         parser.add_argument(
             "name",
             action="store",
@@ -45,13 +44,11 @@ class SetLocation(Command):
 
     @classmethod
     def _add_arguments(cls, parser) -> None:
-
         parser.add_argument(
             "site", action="store", type=str, nargs=1, help="location name"
         )
 
     def __call__(self):
-
         old = self.config.location
         new = str(self.args.site[0])
         self.config.location = new
@@ -80,7 +77,6 @@ class SetLogLevel(Command):
 
     @classmethod
     def _add_arguments(cls, parser) -> None:
-
         parser.add_argument(
             "level",
             action="store",
@@ -135,7 +131,6 @@ def _add_arguments(cls, parser) -> None:
         cls._add_hosts_arg(parser)
 
     def __call__(self):
-
         value = self.args.timeout[0]
         targets = self.parse_hosts()
 

mtui/commands/update.py

@@ -23,7 +23,6 @@ class Update(Command):
 
     @classmethod
     def _add_arguments(cls, parser) -> None:
-
         parser.add_argument(
             "--newpackage",
             action="store_const",
@@ -47,7 +46,6 @@ def _add_arguments(cls, parser) -> None:
 
     @requires_update
     def __call__(self):
-
         logger.info("Updating")
 
         targets = self.parse_hosts()

mtui/config.py

@@ -58,7 +58,7 @@ def location(self):
     def location(self, x):
         try:
             self.refhosts(self).check_location_sanity(x)
-        except (InvalidLocationError) as e:
+        except InvalidLocationError as e:
             logger.error(e)
             return
         except RefhostsResolveFailed:

mtui/connection.py

@@ -518,7 +518,6 @@ def get(self, remote, local):
 
     # Similar to 'get' but handles folders.
     def get_folder(self, remote_folder, local_folder):
-
         remote_folder = str(remote_folder)
         local_folder = str(local_folder)
         sftp = self.__sftp_reconnect()

mtui/connector/smelt.py

@@ -26,7 +26,6 @@ def __init__(self, rrid, apiurl="https://smelt.suse.de/graphql/"):
         self.data = self._get_data()
 
     def _get_data(self):
-
         query_incident = f"""{{
   incidents(incidentId: {self.rrid.maintenance_id} ) {{
     edges {{
@@ -127,7 +126,6 @@ def openqa_links_verbose(self):
 
     @staticmethod
     def _comments(data):
-
         if not data:
             return None
         if "comments" not in data:

mtui/display.py

@@ -12,13 +12,12 @@ def println(self, msg="", eol="\n"):
         return self.output.write(msg + eol)
 
     def list_bugs(self, bugs, jira, url):
-
         ids = sorted(bugs.keys())
         if ids == [""]:
             self.println("No bugs associated with Release Request.")
         else:
             self.println(f'Buglist: {url}/buglist.cgi?bug_id={",".join(ids)}')
-            for (bug, summary) in [(bug, bugs[bug]) for bug in ids]:
+            for bug, summary in [(bug, bugs[bug]) for bug in ids]:
                 self.println()
                 self.println("Bug #{0:5}: {1}".format(bug, summary))
                 self.println(f"{url}/show_bug.cgi?id={bug}")
@@ -28,7 +27,7 @@ def list_bugs(self, bugs, jira, url):
             self.println()
             self.println("No Jira issues associated with Release Request.")
         else:
-            for (issue, summary) in [(issue, jira[issue]) for issue in ids]:
+            for issue, summary in [(issue, jira[issue]) for issue in ids]:
                 self.println()
                 self.println("Jira #{0:5}: {1}".format(issue, summary))
                 self.println(f"https://jira.suse.com/browse/{issue}")

mtui/export/manual.py

@@ -257,7 +257,6 @@ def _fillup_hosts_to_template(self):
                     self.template[index + 1] = "=> PASSED\n"
 
     def _host_installog_to_template(self, target):
-
         t = []
         try:
             host_log = [host for host in self.results if host.hostname == target][0]

mtui/main.py

@@ -33,7 +33,6 @@ def main():
 
 
 def run_mtui(sys, config, logger, Prompt, Display, args):
-
     if args.debug:
         logger.setLevel(level=logging.DEBUG)
 

mtui/target/__init__.py

@@ -122,13 +122,11 @@ def query_versions(self, packages=None) -> None:
                 else:
                     self.packages[p].current = None
         elif self.state == "dryrun":
-
             logger.info(
                 'dryrun: {} running "rpm -q {}"'.format(self.hostname, packages)
             )
             self.out.append(["rpm -q {}".format(packages), "dryrun\n", "", 0, 0])
         elif self.state == "disabled":
-
             self.out.append(["", "", "", 0, 0])
 
     def query_package_versions(self, packages) -> Optional[Dict[str, RPMVersion]]:
@@ -256,11 +254,9 @@ def run(self, command, lock=None) -> None:
                 ]
             )
         elif self.state == "dryrun":
-
             logger.info('dryrun: {} running "{}"'.format(self.hostname, command))
             self.out.append([command, "dryrun\n", "", 0, 0])
         elif self.state == "disabled":
-
             self.out.append(["", "", "", 0, 0])
 
     def shell(self):
@@ -287,7 +283,6 @@ def put(self, local, remote):
             logger.info("dryrun: put {} {}:{}".format(local, self.hostname, remote))
 
     def get(self, remote, local):
-
         if remote.endswith("/"):
             f = self.connection.get_folder
             s = "folder"

mtui/template/testreport.py

@@ -160,7 +160,6 @@ def _parse(self, tpl: str) -> None:
         self._warn_missing_fields()
 
     def _parse_json(self, data, tpl: str) -> None:
-
         if self.path:
             raise TestReportAlreadyLoaded(self.path)
 

tests/test_utils.py

@@ -18,7 +18,6 @@ def create_temp(tmpdir_factory):
 
 
 class TestEnsureDirExists(object):
-
     _callback_paths = []
 
     def test_create(self, create_temp):