add test cov for refactor

Author: rmorshea

src/idom/client/module.py

@@ -125,7 +125,7 @@ def __init__(
 
         if self.source_type == URL_SOURCE:
             if check_exports is True:
-                raise ValueError("Cannot check exports for source type {source_type!r}")
+                raise ValueError(f"Cannot check exports of source type {source_type!r}")
             elif source_file is not None:
                 raise ValueError(f"File given, but source type is {source_type!r}")
             else:

tests/test_client/test_manage.py

@@ -30,6 +30,14 @@ def test_add_web_module_source_must_exist(tmp_path):
         add_web_module("test", tmp_path / "file-does-not-exist.js")
 
 
+def test_cannot_add_web_module_if_already_exists(tmp_path):
+    temp_file = tmp_path / "temp.js"
+    temp_file.write_text("console.log('hello!')")  # this won't get run
+    add_web_module("test", temp_file)
+    with pytest.raises(ValueError, match="already exists"):
+        add_web_module("test", temp_file)
+
+
 def test_web_module_path_must_exist():
     with pytest.raises(ValueError, match="does not exist at path"):
         web_module_path("this-does-not-exist", must_exist=True)

tests/test_client/test_module.py

@@ -116,3 +116,26 @@ def test_module_must_export_mount_if_exports_mount_is_set():
             source_file=JS_FIXTURES / "component-without-mount.js",
             exports_mount=True,
         )
+
+
+def test_cannot_have_source_file_for_url_source_type():
+    with pytest.raises(ValueError, match="File given, but source type is 'URL'"):
+        idom.Module("test", source_file="something.js", source_type=URL_SOURCE)
+
+
+def test_cannot_check_exports_for_url_source_type():
+    with pytest.raises(ValueError, match="Can't check exports for source type 'URL'"):
+        idom.Module("test", check_exports=True, source_type=URL_SOURCE)
+
+
+def test_invalid_source_type():
+    with pytest.raises(ValueError, match="Invalid source type"):
+        idom.Module("test", source_type="TYPE_DOES_NOT_EXIST")
+
+
+def test_attribute_error_if_lowercase_name_doesn_not_exist():
+    mod = idom.Module("test", source_type=URL_SOURCE)
+    with pytest.raises(AttributeError, match="this_attribute_does_not_exist"):
+        # This attribute would otherwise be considered to
+        # be the name of a component the module exports.
+        mod.this_attribute_does_not_exist