From 46e4699d5a7718630ba8815c83887a7d683eaaa9 Mon Sep 17 00:00:00 2001
From: Robert Kaussow <mail@thegeeklab.de>
Date: Mon, 7 Jun 2021 22:28:28 +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 17b90f8..298c287 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: 66e540c49b6cf042055a69df47b4907af185a1a822a3a2d59132e794738e4894
+hmac: 5909b86e6bc5dae7f092a0f598e28b74449e270dd2e7485e03d0e93c6f9f4bef
 
 ...
diff --git a/molecule/centos7/tests/test_default.py b/molecule/centos7/tests/test_default.py
index 2720165..8bd06e7 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_incron_is_installed(host):