kwargs widget should be initialized with parameter (#1360)

fsspec · Sep 13, 2023 · d0b6f70 · d0b6f70
1 parent 765809a
commit d0b6f70
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/fsspec/gui.py b/fsspec/gui.py
@@ -262,7 +262,9 @@ def _setup(self):
             name="protocol",
             align="center",
         )
-        self.kwargs = pn.widgets.TextInput(name="kwargs", value="{}", align="center")
+        self.kwargs = pn.widgets.TextInput(
+            name="kwargs", value=self.init_kwargs, align="center"
+        )
         self.go = pn.widgets.Button(name="⇨", align="end", width=45)
         self.main = SingleSelect(size=10)
         self.home = pn.widgets.Button(name="🏠", width=40, height=30, align="end")

diff --git a/fsspec/tests/test_gui.py b/fsspec/tests/test_gui.py
@@ -8,3 +8,12 @@ def test_basic():
 
     gui = fsspec.gui.FileSelector()
     assert "url" in str(gui.panel)
+
+
+def test_kwargs(tmpdir):
+    """confirm kwargs are passed to the filesystem instance"""
+    import fsspec.gui
+
+    gui = fsspec.gui.FileSelector(f"file://{tmpdir}", kwargs="{'auto_mkdir': True}")
+
+    assert gui.fs.auto_mkdir