Add automatic tests with Github Actions

.github/workflows/tox.yml

@@ -0,0 +1,41 @@
+name: "tox.yml"
+
+on:
+    push:
+        branches: [ master ]
+    pull_request:
+        branches: [ master ]
+
+jobs:
+    build:
+        runs-on: ${{ matrix.os }}
+        env: 
+            TOXENV: ${{ matrix.test }}
+            RUN_INTEGRATION_TESTS: ${{ matrix.test_number}}
+        steps:
+            - uses: actions/checkout@v3
+            - run: pip install "tox"
+            - run: pip install bandit
+            - run: pip install codecov
+            - run: pip install slugs
+            - run: sudo chmod -R 777 /usr/local/lib/
+            - run: python3 setup.py install
+            - run: ./.travis/run.sh
+            - run: codecov
+        strategy:
+            matrix:
+                test: ["pep8", "bandit", "docs"]
+                os: [ubuntu-22.04,ubuntu-20.04]
+                include:
+                    - test: "py38"
+                      test_number: "0"
+                      os: ubuntu-20.04
+                    - test: "py38"
+                      test_number: "1"
+                      os: ubuntu-20.04
+                    - test: "py310"
+                      test_number: "0"
+                      os: ubuntu-22.04
+                    - test: "py310"
+                      test_number: "1"
+                      os: ubuntu-22.04

.travis/run.sh

@@ -3,6 +3,9 @@
 set -e
 set -x
 
+pkill -f run_server.py || true
+sleep 1
+
 if [[ "${RUN_INTEGRATION_TESTS}" == "1" ]]; then
     sudo mkdir -p /etc/pykmip/certs
     sudo mkdir -p /etc/pykmip/policies
@@ -14,6 +17,7 @@ if [[ "${RUN_INTEGRATION_TESTS}" == "1" ]]; then
     sudo cp ./.travis/policy.json /etc/pykmip/policies/policy.json
     sudo mkdir -p /var/log/pykmip
     sudo chmod 777 /var/log/pykmip
+    sudo chmod -R 777 /etc/pykmip/
     python3 ./bin/run_server.py &
     tox -e integration -- --config client
 elif [[ "${RUN_INTEGRATION_TESTS}" == "2" ]]; then

docs/source/conf.py

@@ -64,7 +64,7 @@
 #
 # This is also used if you do content translation via gettext catalogs.
 # Usually you set "language" from the command line for these cases.
-language = None
+language = 'en'
 
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.

kmip/services/server/auth/slugs.py

@@ -88,7 +88,7 @@ def authenticate(self,
         )
 
         try:
-            response = requests.get(self.users_url.format(user_id))
+            response = requests.get(self.users_url.format(user_id), timeout=10)
         except Exception:
             raise exceptions.ConfigurationError(
                 "A connection could not be established using the SLUGS URL."
@@ -98,7 +98,7 @@ def authenticate(self,
                 "Unrecognized user ID: {}".format(user_id)
             )
 
-        response = requests.get(self.groups_url.format(user_id))
+        response = requests.get(self.groups_url.format(user_id), timeout=10)
         if response.status_code == 404:
             raise exceptions.PermissionDenied(
                 "Group information could not be retrieved for user ID: "

kmip/services/server/crypto/engine.py

@@ -269,7 +269,7 @@ def mac(self, algorithm, key, data):
             )
             cipher_algorithm = self._symmetric_key_algorithms.get(algorithm)
             try:
-                # ARC4 and other non-block cipher algorithms will raise TypeError exceptions
+                # ARC4 and other non-block cipher algorithm will raise TypeError
                 c = cmac.CMAC(cipher_algorithm(key), backend=default_backend())
                 c.update(data)
                 mac_data = c.finalize()

kmip/services/server/session.py

@@ -127,7 +127,8 @@ def _handle_message_loop(self):
         )
 
         try:
-            if hasattr(self._connection, 'shared_ciphers') and self._connection.shared_ciphers() is not None:
+            if (hasattr(self._connection, 'shared_ciphers')
+                    and self._connection.shared_ciphers() is not None):
                 shared_ciphers = self._connection.shared_ciphers()
                 self._logger.debug(
                     "Possible session ciphers: {0}".format(len(shared_ciphers))

kmip/tests/unit/services/server/auth/test_slugs.py

@@ -137,10 +137,10 @@ def test_authenticate(self, mock_get_client_identity, mock_request_get):
 
         mock_get_client_identity.assert_called_once_with("test")
         mock_request_get.assert_any_call(
-            "http://127.0.0.1:8080/test/slugs/users/John Doe"
+            "http://127.0.0.1:8080/test/slugs/users/John Doe", timeout=10
         )
         mock_request_get.assert_any_call(
-            "http://127.0.0.1:8080/test/slugs/users/John Doe/groups"
+            "http://127.0.0.1:8080/test/slugs/users/John Doe/groups", timeout=10
         )
         self.assertEqual(('John Doe', ['Group A', 'Group B']), result)
 

kmip/tests/unit/services/test_kmip_client.py

@@ -81,7 +81,7 @@ def setUp(self):
         self.cred_factory = CredentialFactory()
         self.secret_factory = SecretFactory()
 
-        self.client = KMIPProxy()
+        self.client = KMIPProxy(config_file="/dev/null")
 
         KMIP_PORT = 9090
         CA_CERTS_PATH = os.path.normpath(os.path.join(os.path.dirname(

test-requirements.txt

@@ -6,5 +6,5 @@ fixtures
 mock
 slugs
 sphinx
-PyYAML<=5.2
-bandit==1.5.1
+PyYAML
+bandit

tox.ini

@@ -1,8 +1,8 @@
 [tox]
-envlist = pep8,py27,py34,py35,py36,py37,bandit,docs
+envlist = pep8,py38,py310,bandit,docs
 
 [testenv]
-passenv = TRAVIS TRAVIS_JOB_ID TRAVIS_BRANCH
+passenv = *
 
 deps = -r{toxinidir}/requirements.txt
        -r{toxinidir}/test-requirements.txt
@@ -48,3 +48,4 @@ commands =
 
 [flake8]
 exclude = .git,.tox,dist,rpmbuild,*.egg-info
+max-line-length = 88