Merge pull request #75 from timdaman/workflow

Adding initial workflow

Author: timdaman

.coveragerc

@@ -2,5 +2,5 @@
 include =
   check_docker/check_*.py
 omit =
-  check_docker/tests/*
+  tests/*
   */__init__.py

.github/workflows/tests.yml

@@ -0,0 +1,41 @@
+name: Run tests
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python_version: [3.6, 3.7, 3.8]
+    
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Python
+      uses: actions/setup-python@v2
+      with:
+        python-version: ${{ matrix.python_version }}
+    - name: Install dependencies
+      run: |
+        python -V
+        printenv
+        python -m pip install --upgrade pip
+        pip install flake8 pytest coverage pyfakefs pytest-cov
+    - name: Lint with flake8
+      run: |
+        # stop the build if there are Python syntax errors or undefined names
+        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+    - name: Test with pytest
+      run: |
+        pytest --cov=check_docker --cov-fail-under 90 --cov-report term --cov-report html
+    - uses: actions/upload-artifact@v2
+      with:
+        name: coverage_report
+        path: htmlcov

check_docker/check_docker.py

@@ -101,8 +101,9 @@ def __getattr__(self, item):
 # urllib and http.client's  capabilities the class below tweaks HttpConnection and passes it
 # to urllib registering for socket:// connections
 
+# This is all side effect so excluding coverage
 class SocketFileHandler(AbstractHTTPHandler):
-    class SocketFileToHttpConnectionAdaptor(HTTPConnection):
+    class SocketFileToHttpConnectionAdaptor(HTTPConnection): # pragma: no cover
         def __init__(self, socket_file, timeout=DEFAULT_TIMEOUT):
             super().__init__(host='', port=0, timeout=timeout)
             self.socket_file = socket_file
@@ -765,7 +766,7 @@ def process_args(args):
                         action='store',
                         type=str,
                         metavar='WARN:CRIT',
-                        help='Check cpu usage percentage taking into account any limits. Valid values are 0 - 100.')
+                        help='Check cpu usage percentage taking into account any limits.')
 
     # Memory
     parser.add_argument('--memory',

check_docker/check_swarm.py

@@ -52,8 +52,8 @@
 # urllib and http.client's  capabilities the class below tweaks HttpConnection and passes it
 # to urllib registering for socket:// connections
 
-
-class SocketFileHandler(AbstractHTTPHandler):
+# This is all side effect so excluding coverage
+class SocketFileHandler(AbstractHTTPHandler):  # pragma: no cover
     class SocketFileToHttpConnectionAdaptor(HTTPConnection):
         def __init__(self, socket_file, timeout=DEFAULT_TIMEOUT):
             super().__init__(host='', port=0, timeout=timeout)

tests/test_version.py

@@ -37,6 +37,8 @@ def test_package_present():
 
 @pytest.mark.xfail('TRAVIS_BRANCH' in os.environ and os.environ['TRAVIS_BRANCH'].lower != 'master',
                    reason="Ignore version check outside of master")
+@pytest.mark.xfail('GITHUB_HEAD_REF' in os.environ and os.environ['GITHUB_HEAD_REF'].lower != 'master',
+                   reason="Ignore version check outside of master")
 @pytest.mark.skipif('isolated' in os.environ and os.environ['isolated'].lower != 'false',
                     reason="Can not reach Python package index when isolated")
 @pytest.mark.skipif(sys.version_info[0:2] != (3, 8), reason="Only check on python 3.8")