Merge branch 'master' into patch-1

mtth · Aug 2, 2023 · 600494a · 600494a
2 parents ef9d3b2 + 30fda2f
commit 600494a
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 11 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -14,18 +14,28 @@ jobs:
   test:
     name: Test
     runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version:
+          # - '3.6' (see https://github.com/actions/setup-python/issues/544)
+          - '3.7'
+          - '3.8'
+          - '3.9'
+          # - '3.10' 3.10+ require moving off of nose (https://github.com/nose-devs/nose/issues/1099)
+          # - '3.11'
     steps:
       - name: Check out
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
       - name: Setup Java
-        uses: actions/setup-java@v2
+        uses: actions/setup-java@v3
         with:
           distribution: 'adopt'
           java-version: '8'
       - name: Setup Python
-        uses: actions/setup-python@v2
+        uses: actions/setup-python@v4
         with:
-          python-version: '3.6'
+          python-version: ${{ matrix.python-version }}
       - name: Download Hadoop
         run: |
           echo "HADOOP_HOME=$(./scripts/hadoop.sh download)" >>"$GITHUB_ENV"

diff --git a/setup.py b/setup.py
@@ -44,9 +44,9 @@ def _get_long_description():
     'Intended Audience :: Developers',
     'License :: OSI Approved :: MIT License',
     'Programming Language :: Python',
-    'Programming Language :: Python :: 2.7',
-    'Programming Language :: Python :: 3.5',
-    'Programming Language :: Python :: 3.6',
+    'Programming Language :: Python :: 3.7',
+    'Programming Language :: Python :: 3.8',
+    'Programming Language :: Python :: 3.9',
   ],
   install_requires=[
     'docopt',

diff --git a/test/test_client.py b/test/test_client.py
@@ -56,14 +56,14 @@ class TestOptions(_IntegrationTest):
 
   """Test client options."""
 
+  @nottest # TODO: Investigate why this fails in Python 3.7 and 3.9
+  @raises(ConnectTimeout, ReadTimeout)
   def test_timeout(self):
-    self.client._timeout = 1e-5 # Small enough for it to always timeout.
+    self.client._timeout = 1e-6 # Small enough for it to always timeout.
     try:
       self.client.status('.')
-    except (ConnectTimeout, ReadTimeout):
+    finally:
       self.client._timeout = None
-    else:
-      raise HdfsError('No timeout.')
 
 
 class TestApi(_IntegrationTest):