From 66f9312cb8383619d20d537c25da33cfe003f989 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@thegeeklab.de>
Date: Mon, 7 Jun 2021 22:28:25 +0200
Subject: [PATCH] ci: add python file checks to ci

---
 .drone.jsonnet                         | 22 ++++++++++++++++++++++
 .drone.yml                             | 18 +++++++++++++++++-
 molecule/centos7/tests/test_default.py |  8 ++------
 3 files changed, 41 insertions(+), 7 deletions(-)

diff --git a/.drone.jsonnet b/.drone.jsonnet
index 7eb7c0b..ecf6cd6 100644
--- a/.drone.jsonnet
+++ b/.drone.jsonnet
@@ -13,6 +13,28 @@ local PipelineLinting = {
         'ansible-later',
       ],
     },
+    {
+      name: 'python-format',
+      image: 'python:3.9',
+      environment: {
+        PY_COLORS: 1,
+      },
+      commands: [
+        'pip install -qq yapf',
+        'yapf -dr ./',
+      ],
+    },
+    {
+      name: 'python-flake8',
+      image: 'python:3.9',
+      environment: {
+        PY_COLORS: 1,
+      },
+      commands: [
+        'pip install -qq flake8',
+        'flake8',
+      ],
+    },
   ],
   trigger: {
     ref: ['refs/heads/master', 'refs/tags/**', 'refs/pull/**'],
diff --git a/.drone.yml b/.drone.yml
index 74c2f19..613c6f9 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -12,6 +12,22 @@ steps:
   commands:
   - ansible-later
 
+- name: python-format
+  image: python:3.9
+  commands:
+  - pip install -qq yapf
+  - yapf -dr ./
+  environment:
+    PY_COLORS: 1
+
+- name: python-flake8
+  image: python:3.9
+  commands:
+  - pip install -qq flake8
+  - flake8
+  environment:
+    PY_COLORS: 1
+
 trigger:
   ref:
   - refs/heads/master
@@ -131,6 +147,6 @@ depends_on:
 
 ---
 kind: signature
-hmac: 7d959e1562b2b79f92f8df4954722e5b0a1a2f495cf9d90f77619a23e5d359e3
+hmac: fb9c08dd68c4110376254afb5f926f9c88028319d308295944e5592b80093ba9
 
 ...
diff --git a/molecule/centos7/tests/test_default.py b/molecule/centos7/tests/test_default.py
index a2c619a..4cd5a87 100644
--- a/molecule/centos7/tests/test_default.py
+++ b/molecule/centos7/tests/test_default.py
@@ -2,13 +2,9 @@ import os
 
 import testinfra.utils.ansible_runner
 
-import warnings
-
-warnings.filterwarnings("ignore", category=DeprecationWarning)
-
 testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
-    os.environ['MOLECULE_INVENTORY_FILE']
-).get_hosts('all')
+    os.environ["MOLECULE_INVENTORY_FILE"]
+).get_hosts("all")
 
 
 def test_pam_config(host):