Merge pull request #37 from swansonk14/windows-tests

Fixed windows GitHub Actions tests tempfile issue

Author: swansonk14

.github/workflows/tap-code-coverage.yml renamed to .github/workflows/code-coverage.yml

@@ -24,6 +24,12 @@ jobs:
       uses: actions/setup-python@v2
       with:
         python-version: ${{ matrix.python-version }}
+    - name: Set temp directories on Windows
+      if: matrix.os == 'windows-latest'
+      run: |
+        echo "TMPDIR=$env:USERPROFILE\AppData\Local\Temp" >> $env:GITHUB_ENV
+        echo "TEMP=$env:USERPROFILE\AppData\Local\Temp" >> $env:GITHUB_ENV
+        echo "TMP=$env:USERPROFILE\AppData\Local\Temp" >> $env:GITHUB_ENV
     - name: Install dependencies
       run: |
         git config --global user.email "[email protected]"

.github/workflows/tap-tests.yml renamed to .github/workflows/tests.yml

@@ -1,4 +1,5 @@
 .idea
+.vscode
 __pycache__
 .DS_Store
 *.json

.gitignore

@@ -57,21 +57,17 @@ def test_has_git_false(self) -> None:
             self.assertFalse(has_git())
             os.chdir(self.temp_dir.name)
 
-    # TODO: fix this test on Windows
-    @unittest.skipIf(platform.system() == 'Windows', 'Inconsistent user path on Windows in GitHub Actions')
     def test_get_git_root(self) -> None:
         # Ideally should be self.temp_dir.name == get_git_root() but the OS may add a prefix like /private
-        self.assertTrue(get_git_root().endswith(self.temp_dir.name))
+        self.assertTrue(get_git_root().endswith(self.temp_dir.name.replace('\\', '/')))
 
-    # TODO: fix this test on Windows
-    @unittest.skipIf(platform.system() == 'Windows', 'Inconsistent user path on Windows in GitHub Actions')
     def test_get_git_root_subdir(self) -> None:
         subdir = os.path.join(self.temp_dir.name, 'subdir')
         os.makedirs(subdir)
         os.chdir(subdir)
 
         # Ideally should be self.temp_dir.name == get_git_root() but the OS may add a prefix like /private
-        self.assertTrue(get_git_root().endswith(self.temp_dir.name))
+        self.assertTrue(get_git_root().endswith(self.temp_dir.name.replace('\\', '/')))
 
         os.chdir(self.temp_dir.name)