Merge pull request #359 from lincc-frameworks/delucchi/windows

Confirm behavior under windows.

Author: delucchi-cmu

.github/workflows/testing-windows.yml

@@ -0,0 +1,33 @@
+# This workflow will install Python dependencies and run tests in a Windows environment.
+# This is intended to catch any file-system specific issues, and so runs less
+# frequently than other test suites.
+
+name: Windows unit test
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  build:
+
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        python-version: ['3.10']
+
+    steps:
+    - uses: actions/checkout@v5
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v5
+      with:
+        python-version: ${{ matrix.python-version }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -e .[dev]
+    - name: Run unit tests with pytest
+      run: |
+        python -m pytest tests

tests/nested_pandas/nestedframe/test_io.py

@@ -302,7 +302,8 @@ def test_read_empty_parquet():
     """Test that we can read empty parquet files"""
     orig_nf = generate_data(1, 2).iloc[:0]
 
-    with tempfile.NamedTemporaryFile("wb", suffix="parquet") as tmpfile:
+    with tempfile.NamedTemporaryFile("wb", suffix=".parquet") as tmpfile:
+        tmpfile.close()
         orig_nf.to_parquet(tmpfile.name)
         # All columns
         # Do not check dtype because of:
@@ -333,7 +334,8 @@ def test_read_parquet_list_autocast():
             "d": [[10, 20, 30], [40, 50, 60], [70, 80, 90]],
         }
     )
-    with tempfile.NamedTemporaryFile("wb", suffix="parquet") as tmpfile:
+    with tempfile.NamedTemporaryFile("wb", suffix=".parquet") as tmpfile:
+        tmpfile.close()
         list_nf.to_parquet(tmpfile.name)
 
         nf = read_parquet(tmpfile.name, autocast_list=True)