github workflow: give pre-commit its own job (python-telegram-bot#1612)

tsnoam · web-flow · commit 2fdf48023bbd · 2019-11-15T23:35:44.000+02:00
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -16,7 +16,6 @@ jobs:
           - os: ubuntu-latest
             python-version: 3.7
             test-build: True
-            test-pre-commit: True
           - os: windows-latest
             python-version: 3.7
             test-build: True
@@ -89,4 +88,27 @@ jobs:
         env:
           TEST_OFFICIAL: "true"
         shell: bash --noprofile --norc {0}
-
+  test_pre_commit:
+    name: test-pre-commit
+    runs-on: ${{matrix.os}}
+    strategy:
+      matrix:
+        python-version: [3.7]
+        os: [ubuntu-latest]
+      fail-fast: False
+    steps:
+      - uses: actions/checkout@v1
+      - name: Initialize vendored libs
+        run:
+          git submodule update --init --recursive
+      - name: Set up Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v1
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Install dependencies
+        run: |
+          python -W ignore -m pip install --upgrade pip
+          python -W ignore -m pip install -r requirements.txt
+          python -W ignore -m pip install -r requirements-dev.txt
+      - name: Run pre-commit tests
+        run: pre-commit run --all-files
diff --git a/tests/test_helpers.py b/tests/test_helpers.py
@@ -58,8 +58,8 @@ def test_to_float_timestamp_absolute_aware(self, timezone):
         # we're parametrizing this with two different UTC offsets to exclude the possibility
         # of an xpass when the test is run in a timezone with the same UTC offset
         datetime = dtm.datetime(2019, 11, 11, 0, 26, 16, 10**5, tzinfo=timezone)
-        assert (helpers.to_float_timestamp(datetime) ==
-                1573431976.1 - timezone.utcoffset(None).total_seconds())
+        assert (helpers.to_float_timestamp(datetime)
+                == 1573431976.1 - timezone.utcoffset(None).total_seconds())
 
     def test_to_float_timestamp_absolute_no_reference(self):
         """A reference timestamp is only relevant for relative time specifications"""
@@ -94,8 +94,8 @@ def test_to_float_timestamp_time_of_day_timezone(self, timezone):
         # first test that naive time is assumed to be utc:
         assert helpers.to_float_timestamp(time_of_day, ref_t) == pytest.approx(ref_t)
         # test that by setting the timezone the timestamp changes accordingly:
-        assert (helpers.to_float_timestamp(time_of_day.replace(tzinfo=timezone), ref_t) ==
-                pytest.approx(ref_t + (-utc_offset.total_seconds() % (24 * 60 * 60))))
+        assert (helpers.to_float_timestamp(time_of_day.replace(tzinfo=timezone), ref_t)
+                == pytest.approx(ref_t + (-utc_offset.total_seconds() % (24 * 60 * 60))))
 
     @pytest.mark.parametrize('time_spec', RELATIVE_TIME_SPECS, ids=str)
     def test_to_float_timestamp_default_reference(self, time_spec):
