fix up tests

Author: rmorshea

docs/source/examples/material_ui_slider.py

@@ -4,7 +4,7 @@
 
 
 material_ui = idom.Module("@material-ui/core")
-Slider = material_ui.Import("Slider")
+Slider = material_ui.Import("Slider", fallback="loading...")
 
 material_ui_style = idom.html.link(
     {

idom/client/manage.py

@@ -79,7 +79,7 @@ def delete_web_modules(names: Sequence[str], skip_missing: bool = False) -> None
         if not exists and not skip_missing:
             raise ValueError(f"Module '{name}' does not exist.")
 
-        cache.delete_package(name)
+        cache.delete_package(name, skip_missing)
 
     for p in paths:
         _delete_os_paths(p)
@@ -158,12 +158,13 @@ def add_packages(self, packages: Sequence[str], exports: Sequence[str]) -> None:
         self.package_list = list(set(self.package_list + package_list))
         self.export_list = list(set(self.export_list + export_list))
 
-    def delete_package(self, export_name: str) -> None:
-        self.export_list.remove(export_name)
-        for i, pkg in enumerate(self.package_list):
-            if _export_name_from_package(pkg) == export_name:
-                del self.package_list[i]
-                break
+    def delete_package(self, export_name: str, skip_missing: bool) -> None:
+        if export_name in self.export_list:
+            self.export_list.remove(export_name)
+            for i, pkg in enumerate(self.package_list):
+                if _export_name_from_package(pkg) == export_name:
+                    del self.package_list[i]
+                    break
 
     def save(self) -> None:
         cache = {

idom/server/sanic.py

@@ -88,14 +88,15 @@ async def client_files(
             ) -> response.HTTPResponse:
                 file_extensions = [".html", ".js", ".json"]
                 abs_path = find_path(path)
-                if (
-                    abs_path is not None
-                    and abs_path.suffix in file_extensions
-                    and not abs_path.stem.startswith(".")
-                ):
-                    return await response.file_stream(str(abs_path))
-                else:
-                    response.text(f"Could not find: {path!r}", status=404)
+                return (
+                    await response.file_stream(str(abs_path))
+                    if (
+                        abs_path is not None
+                        and abs_path.suffix in file_extensions
+                        and not abs_path.stem.startswith(".")
+                    )
+                    else response.text(f"Could not find: {path!r}", status=404)
+                )
 
         if config["redirect_root_to_index"]:
 

tests/test_client/test_manage.py

@@ -6,8 +6,6 @@
 
 from idom import client
 
-from .utils import assert_file_is_touched
-
 
 @pytest.mark.slow
 def test_install():
@@ -17,8 +15,6 @@ def test_install():
     assert client.web_module_exists("jquery")
     assert client.web_module_exists("/jquery")  # works with a leading slash too
     assert "jquery" in client.installed()
-    with assert_file_is_touched(client.web_module_path("jquery")):
-        client.install("jquery", force=True)
 
     with pytest.raises(ValueError, match="already exists"):
         # can't register a module with the same name

tests/test_main.py

@@ -3,8 +3,6 @@
 from idom.__main__ import main
 from idom import client
 
-from tests.test_client.utils import assert_file_is_touched
-
 from tests.general_utils import assert_same_items
 
 
@@ -18,10 +16,6 @@ def test_simple_install(capsys):
     captured = capsys.readouterr()
     assert "- jquery" in captured.out
 
-    with assert_file_is_touched(client.web_module_path("jquery")):
-        main("install", "jquery", "--force")
-    assert client.web_module_exists("jquery")
-
     main("uninstall", "jquery")
     assert not client.web_module_exists("jquery")
 
@@ -57,11 +51,8 @@ def test_restore(capsys):
     "args, error",
     [
         (("uninstall", "x", "--exports"), ValueError("does not support exports")),
-        (("uninstall", "x", "--force"), ValueError("does not support force")),
         (("installed", "--exports"), ValueError("does not support exports")),
-        (("installed", "--force"), ValueError("does not support force")),
         (("restore", "--exports"), ValueError("does not support exports")),
-        (("restore", "--force"), ValueError("does not support force")),
     ],
 )
 def test_bad_options(capsys, args, error):