fixup! [wip][feature] Add support for fsspec backends

mxmlnkn · Oct 6, 2024 · e88a60b · e88a60b
1 parent 53ebb63
commit e88a60b
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/tests/runtests.sh b/tests/runtests.sh
@@ -1810,11 +1810,11 @@ checkURLProtocolHTTP()
     #     busybox httpd -f -p 8000 &  # Does not support range requests.
     # sudo apt install ruby-webrick
     if ! command -v ruby &>/dev/null; then
-        echo "Ruby not found"
-        return 1
+        echo "Ruby not found. Please install ruby-webrick."
+        return 0
     fi
-    ruby -run -e httpd --help
-    ruby -run -e httpd . --port $port --bind-address=127.0.0.1 &
+    ruby -run -e httpd --version || returnError "$LINENO" 'Failed to start up ruby HTTP test server!'
+    ruby -run -e httpd . --port $port --bind-address=127.0.0.1 2>&1 > 'httpd-ruby-webrick.log' &
     pid=$!
     sleep 5
     wget 127.0.0.1:$port
@@ -2135,9 +2135,6 @@ EOF
 
 checkRemoteSupport()
 {
-    #checkURLProtocolSSH
-    checkURLProtocolHTTP
-    exit 0
     # Some implementations of fsspec. See e.g. this list:
     # https://filesystem-spec.readthedocs.io/en/latest/api.html#other-known-implementations