Moved from Travis to github actions; Workaround for pygls throwing RuntimeErrors during tests

Author: suoto

.github/workflows/unit_tests.yml

@@ -0,0 +1,56 @@
+---
+name: Unit tests
+
+on: [push, pull_request]
+
+jobs:
+  Python36:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Setup Python 3.6
+        uses: actions/setup-python@v1
+        with:
+          python-version: 3.6
+      - name: Get Docker image
+        run: docker pull suoto/hdl_checker_test:latest
+      - name: Run tests
+        run: |
+          ./run_tests.sh -e py36-linux -- -v
+  Python37:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Setup Python 3.7
+        uses: actions/setup-python@v1
+        with:
+          python-version: 3.7
+      - name: Get Docker image
+        run: docker pull suoto/hdl_checker_test:latest
+      - name: Run tests
+        run: |
+          ./run_tests.sh -e py37-linux -- -v
+  Python38:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Setup Python 3.8
+        uses: actions/setup-python@v1
+        with:
+          python-version: 3.8
+      - name: Get Docker image
+        run: docker pull suoto/hdl_checker_test:latest
+      - name: Run tests
+        run: |
+          ./run_tests.sh -e py38-linux -- -v
+
+  Pypi:
+    runs-on: ubuntu-latest
+    needs: [Python36, Python37, Python38]
+    steps:
+      - name: Publish package
+        if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags')
+        uses: pypa/gh-action-pypi-publish@master
+        with:
+          user: __token__
+          password: ${{ secrets.pypi_password }}

hdl_checker/tests/test_lsp.py

@@ -22,7 +22,6 @@
 import logging
 import os
 import os.path as p
-import time
 from threading import Thread
 from typing import Any, List, Optional, Union
 
@@ -191,23 +190,12 @@ def tearDown(self):  # pylint:disable=inconsistent-return-statements
             LSP_REQUEST_TIMEOUT
         )
         self.client.lsp.notify(features.EXIT)
-
-        for i in range(100):
-            if not self.server_thread.is_alive():
-                return
-            _logger.debug(
-                "[%d] server alive: %s, client alive: %s",
-                i,
-                self.server_thread.is_alive(),
-                self.client_thread.is_alive(),
-            )
-            time.sleep(0.1)
-
-        self.fail("Timeout waiting for server thread to complete")
-        self.client.shutdown()
-
         self.assertIsNone(shutdown_response)
-
+        # pygls server has cleanup routines run when the interpreter is
+        # exiting, at which point the client will be long gone. This results in
+        # RuntimeError: cannot schedule new futures after shutdown. To work
+        # around this, we're accessing its itnernal event to force it to exit
+        self.client._stop_event.set()
         self.server_thread.join()
         self.client_thread.join()