Adds testing on Windows to the GitHub action

Author: erikhuck

.github/workflows/main.yml

@@ -12,10 +12,11 @@ on:
 jobs:
   build:
 
-    runs-on: ubuntu-latest
     strategy:
       matrix:
         python-version: ["3.8", "3.9", "3.10"]
+        os: [ ubuntu-latest, windows-latest ]
+    runs-on: ${{matrix.os}}
 
     steps:
     - uses: actions/checkout@v3
@@ -25,5 +26,7 @@ jobs:
         python-version: ${{ matrix.python-version }}
     - name: Install testing environment and kegg_pull package
       run: bash dev/install.sh
+      shell: bash
     - name: Test with pytest
       run: bash dev/test.sh
+      shell: bash

dev/install.sh

@@ -2,7 +2,7 @@ echo "Removing previous .env/ directory if it exists..."
 rm -rf .env/
 echo "Creating new .env/ directory..."
 python3 -m venv .env/
-source .env/bin/activate
+source .env/bin/activate || source .env/Scripts/activate # Windows has Scripts instead of bin
 python3 -m pip install --upgrade pip
 python3 -m pip install pytest pytest-mock pytest-cov sphinx sphinx-rtd-theme notebook
 python3 -m pip install -e .

dev/test.sh

@@ -1,3 +1,3 @@
-source .env/bin/activate
+source .env/bin/activate || source .env/Scripts/activate # Windows has Scripts instead of bin
 
-python -m pytest dev --cov --cov-branch --cov-report=term-missing
+python3 -m pytest dev --cov --cov-branch --cov-report=term-missing

dev/test_main.py

@@ -160,6 +160,7 @@ def test_main_pull(mocker, args: list, output: str):
 
     # If running on Windows, change the actual files names to have underscores instead of colons.
     if os.name == 'nt':  # pragma: no cover
+        expected_output_files: list = expected_output_files[:-1]  # The last brite gives different output on Windows
         successful_entry_ids: list = expected_output_files  # pragma: no cover
 
     for successful_entry_id, expected_output_file in zip(successful_entry_ids, expected_output_files):

src/kegg_pull/_utils.py

@@ -90,8 +90,9 @@ def save_file(file_location: str, file_content: t.Union[str, bytes], file_name:
 
         file_path: str = os.path.join(file_location, file_name)
         save_type: str = 'wb' if type(file_content) is bytes else 'w'
+        encoding: t.Union[str, None] = None if type(file_content) is bytes else 'utf-8'
 
-        with open(file_path, save_type) as file:
+        with open(file_path, save_type, encoding=encoding) as file:
             file.write(file_content)