pytest-molecule is going black

We are going black and unlikely to look back. No more debates about
code-style.

Author: ssbarnea

.flake8

@@ -0,0 +1,8 @@
+[flake8]
+# do not add excludes for files in repo
+exclude = .venv/,.tox/,dist/,build/,.eggs/
+format = pylint
+# E203: https://github.com/python/black/issues/315
+ignore = E741,W503,W504,H,E501,E203
+# 88 is official black default:
+max-line-length = 88

.pre-commit-config.yaml

@@ -1,5 +1,10 @@
 ---
 repos:
+  - repo: https://github.com/python/black
+    rev: 19.3b0
+    hooks:
+      - id: black
+        language_version: python3
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v2.1.0
     hooks:
@@ -14,6 +19,8 @@ repos:
     rev: 3.7.7
     hooks:
       - id: flake8
+        additional_dependencies:
+          - flake8-black
   - repo: https://github.com/adrienverge/yamllint.git
     rev: v1.15.0
     hooks:

README.rst

@@ -14,6 +14,10 @@ pytest-molecule
     :target: https://travis-ci.org/pycontribs/pytest-molecule
     :alt: See Build Status on Travis CI
 
+.. image:: https://img.shields.io/badge/code%20style-black-000000.svg
+    :target: https://github.com/python/black
+    :alt: Python Black Code Style
+
 PyTest Molecule Plugin :: auto detects and runs molecule tests
 
 ----

pytest_molecule/__init__.py

@@ -19,7 +19,7 @@ def pytest_configure(config):
         raise Exception("Failed to ping docker server.")
 
     # validate selinux availability
-    if sys.platform == 'linux' and os.path.isfile("/etc/selinux/config"):
+    if sys.platform == "linux" and os.path.isfile("/etc/selinux/config"):
         try:
             import selinux  # noqa
         except Exception:
@@ -29,7 +29,8 @@ def pytest_configure(config):
                 "libselinux python bindings installed. These can only be "
                 "installed using your distro package manager and are specific "
                 "to each python version. Common package names: "
-                "libselinux-python python2-libselinux python3-libselinux")
+                "libselinux-python python2-libselinux python3-libselinux"
+            )
             # we do not re-raise this exception because missing or broken
             # selinux bindings are not guaranteed to fail molecule execution.
 
@@ -41,7 +42,7 @@ def pytest_collect_file(parent, path):
 
 class MoleculeFile(pytest.File):
     def collect(self):
-        yield MoleculeItem('test', self)
+        yield MoleculeItem("test", self)
 
     def __str__(self):
         return str(self.fspath.relto(os.getcwd()))
@@ -53,11 +54,11 @@ def __init__(self, name, parent):
 
     def runtest(self):
         folders = self.fspath.dirname.split(os.sep)
-        cwd = os.path.abspath(os.path.join(self.fspath.dirname, '../..'))
+        cwd = os.path.abspath(os.path.join(self.fspath.dirname, "../.."))
         scenario = folders[-1]
         role = folders[-3]  # noqa
-        cmd = [sys.executable, '-m', 'molecule', self.name, '-s', scenario]
-        print("running: %s (from %s)" % (" " .join(cmd), cwd))
+        cmd = [sys.executable, "-m", "molecule", self.name, "-s", scenario]
+        print("running: %s (from %s)" % (" ".join(cmd), cwd))
 
         try:
             # Workaround for STDOUT/STDERR line ordering issue:
@@ -67,19 +68,20 @@ def runtest(self):
                 cwd=cwd,
                 stdout=subprocess.PIPE,
                 stderr=subprocess.STDOUT,
-                universal_newlines=True)
+                universal_newlines=True,
+            )
             for line in p.stdout:
                 print(line, end="")
             p.wait()
             if p.returncode != 0:
                 pytest.fail(
-                    "Error code %s returned by: %s" % (
-                        p.returncode, " ".join(cmd)),
-                    pytrace=False)
+                    "Error code %s returned by: %s" % (p.returncode, " ".join(cmd)),
+                    pytrace=False,
+                )
         except Exception as e:
             pytest.fail(
-                "Exception %s returned by: %s" % (e, " ".join(cmd)),
-                pytrace=False)
+                "Exception %s returned by: %s" % (e, " ".join(cmd)), pytrace=False
+            )
 
     def reportinfo(self):
         return self.fspath, 0, "usecase: %s" % self.name