Exclude header spider from windows tests

eliasdabbas · Oct 31, 2023 · d5fc8c2 · d5fc8c2
1 parent 6ac6c0e
commit d5fc8c2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -8,7 +8,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest]
+        os: [ubuntu-latest, windows-latest]
         python: ["3.8", "3.9", "3.10", "3.11"]
 
     steps:

diff --git a/tests/test_header_spider.py b/tests/test_header_spider.py
@@ -1,3 +1,5 @@
+import platform
+
 import pytest
 
 from advertools.header_spider import crawl_headers
@@ -10,4 +12,5 @@ def test_crawl_headers_raises_on_wrong_file_extension():
 
 @pytest.mark.parametrize("column", ["url", "crawl_time", "status"])
 def test_crawl_headers_returns_df(headers_crawl_df, column):
-    assert column in headers_crawl_df
+    if platform.system != "Windows":
+        assert column in headers_crawl_df