wip

.github/workflows/tests.yml

@@ -98,6 +98,9 @@ jobs:
 
     steps:
     - uses: actions/checkout@v4
+      with:
+        # We need one tag for testing the git mount.
+        fetch-tags: true
 
     - name: Set up Python ${{ matrix.python-version }}
       uses: actions/setup-python@v5
@@ -216,7 +219,12 @@ jobs:
         # Segfaults (139) are not allowed but other exit codes are valid!
         python3 ratarmount.py tests/simple.bz2 || [ $? != 139 ]
 
+    - name: Install pip Test Dependencies
+      run: |
+        python3 -m pip install -r tests/requirements-tests.txt
+
     - name: Unit Tests
+      if: false
       run: |
         python3 -m pip install pytest pytest-xdist
         for file in core/tests/test_*.py tests/test_*.py; do
@@ -246,6 +254,15 @@ jobs:
         done
         python3 tests/tests.py
 
+    - name: Git Debug
+      if: ${{ !startsWith( matrix.os, 'macos' ) }}
+      run: |
+        git status
+        pwd
+        ls -la
+        ls -la .git
+        git show --stat v0.15.2
+
     - name: Regression Tests (FUSE 3)
       if: ${{ !startsWith( matrix.os, 'macos' ) }}
       run: |

core/ratarmountcore/FSSpecMountSource.py

@@ -13,7 +13,6 @@
 try:
     import fsspec
     import fsspec.core
-    import fsspec.implementations.http
 except ImportError:
     fsspec = None  # type: ignore
 
@@ -36,7 +35,7 @@ def __init__(self, urlOrOpenFile, **options) -> None:
         # The fsspec filesystems are not uniform! http:// expects the arguments to isdir with prefixed
         # protocol while other filesystem implementations are fine with only the path.
         # https://github.com/ray-project/ray/issues/26423#issuecomment-1179561181
-        self._isHTTP = isinstance(self.fileSystem, fsspec.implementations.http.HTTPFileSystem)
+        self._isHTTP = self.fileSystem.protocol.startswith('http')
         prefix = self.openFile.path
         self.prefix = prefix.rstrip("/") if prefix.strip("/") and self.fileSystem.isdir(prefix) else ""
 

tests/runtests.sh

@@ -2039,9 +2039,7 @@ checkRemoteSupport()
     # https://filesystem-spec.readthedocs.io/en/latest/api.html#other-known-implementations
     checkURLProtocolFile
     checkURLProtocolSamba
-    #checkURLProtocolGit  # TODO does not work in CI for some reason...
-    # Warning:  Trying to open with fsspec raised an exception: 'v0.15.2'
-    # Error:  Mount source does not exist: git://v0.15.2@tests/single-file.tar
+    checkURLProtocolGit  # TODO does not work in CI for some reason...
 
     #checkURLProtocolHTTP  # TODO
     checkURLProtocolFTP