put all modules under web_modules + more test coverage

Author: rmorshea

.pre-commit-config.yaml

@@ -1,15 +1,20 @@
 repos:
--   repo: https://github.com/ambv/black
+  - repo: https://github.com/ambv/black
     rev: stable
     hooks:
-    - id: black
-      language_version: python3.6
--   repo: https://github.com/PyCQA/flake8
+      - id: black
+        language_version: python3.6
+  - repo: https://github.com/PyCQA/flake8
     rev: 3.7.9
     hooks:
-    - id: flake8
--   repo: https://github.com/pre-commit/mirrors-mypy
+      - id: flake8
+  - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v0.761
     hooks:
-    -   id: mypy
+      - id: mypy
         additional_dependencies: [pyalect]
+  - repo: https://github.com/kynan/nbstripout
+    rev: master
+    hooks:
+      - id: nbstripout
+        files: ".ipynb"

docs/source/extras.rst

@@ -105,7 +105,7 @@ While ``your_module.py`` should contain the following:
 
     # dialect=html
     from idom import html
-    assert html("<div/>") == {"tagName": "div"}
+    assert html(f"<div/>") == {"tagName": "div"}
 
 And that's it!
 

examples/chart.js

@@ -1,11 +1,6 @@
-import React from "../web_modules/react.js";
-import {
-  VictoryBar,
-  VictoryChart,
-  VictoryTheme,
-  Bar,
-} from "../web_modules/victory.js";
-import htm from "../web_modules/htm.js";
+import React from "./react.js";
+import { VictoryBar, VictoryChart, VictoryTheme, Bar } from "./victory.js";
+import htm from "./htm.js";
 
 const html = htm.bind(React.createElement);
 

idom/__init__.py

@@ -15,11 +15,12 @@
 
 from .tools import Var, html_to_vdom
 
+# try to automatically setup the dialect's import hook
 try:
     import pyalect
     import tagged
     import htm
-except ImportError:
+except ImportError:  # pragma: no cover
     pass
 else:
     from . import dialect

idom/client/.gitignore

@@ -1,3 +1,2 @@
 node_modules
 web_modules
-user_modules

idom/client/__init__.py

@@ -12,32 +12,31 @@
 CORE_MODULES = CLIENT_DIR / "core_modules"
 NODE_MODULES = CLIENT_DIR / "node_modules"
 WEB_MODULES = CLIENT_DIR / "web_modules"
-USER_MODULES = CLIENT_DIR / "user_modules"
 
 
-def import_path(prefix: str, name: str) -> str:
-    path = f"../{prefix}/{name}.js"
-    if not module_exists(prefix, name):
+def import_path(name: str) -> str:
+    path = f"../{WEB_MODULES.name}/{name}.js"
+    if not module_exists(name):
         raise ValueError(f"Module '{path}' does not exist.")
     return path
 
 
 def define_module(name: str, source: str) -> str:
-    path = _create_module_os_path(USER_MODULES, name)
+    path = _create_web_module_os_path(name)
     with path.open("w+") as f:
         f.write(source)
-    return import_path("user_modules", name)
+    return import_path(name)
 
 
-def delete_module(prefix: str, name: str) -> None:
-    path = _find_module_os_path(prefix, name)
+def delete_module(name: str) -> None:
+    path = _find_web_module_os_path(name)
     if path is None:
-        raise ValueError(f"Module '{import_path(prefix, name)}' does not exist.")
+        raise ValueError(f"Module '{name}' does not exist.")
     _delete_os_paths(path)
 
 
-def module_exists(prefix: Union[str, Path], name: str) -> bool:
-    return _find_module_os_path(prefix, name) is not None
+def module_exists(name: str) -> bool:
+    return _find_web_module_os_path(name) is not None
 
 
 def install(dependencies: Dict[str, str]) -> None:
@@ -65,7 +64,7 @@ def install(dependencies: Dict[str, str]) -> None:
 
 
 def restore() -> None:
-    _delete_os_paths(WEB_MODULES, NODE_MODULES, USER_MODULES)
+    _delete_os_paths(WEB_MODULES, NODE_MODULES)
     _run_subprocess(["npm", "install"], CLIENT_DIR)
     _run_subprocess(["npm", "run", "snowpack"], CLIENT_DIR)
 
@@ -99,11 +98,8 @@ def _run_subprocess(args: List[str], cwd: Union[str, Path]):
         raise
 
 
-def _find_module_os_path(prefix: Union[str, Path], name: str) -> Optional[Path]:
-    if isinstance(prefix, str):
-        path = CLIENT_DIR / prefix
-    else:
-        path = prefix
+def _find_web_module_os_path(name: str) -> Optional[Path]:
+    path = WEB_MODULES
     for name_part in name.split("/"):
         if not path.is_dir():
             return None
@@ -114,11 +110,8 @@ def _find_module_os_path(prefix: Union[str, Path], name: str) -> Optional[Path]:
     return full_path
 
 
-def _create_module_os_path(prefix: Union[str, Path], name: str) -> Path:
-    if isinstance(prefix, str):
-        path = CLIENT_DIR / prefix
-    else:
-        path = prefix
+def _create_web_module_os_path(name: str) -> Path:
+    path = WEB_MODULES
     for n in name.split("/"):
         if not path.exists():
             path.mkdir()

idom/core/element.py

@@ -17,7 +17,7 @@
 )
 
 
-if TYPE_CHECKING:
+if TYPE_CHECKING:  # pragma: no cover
     from .layout import AbstractLayout
     from .vdom import VdomDict  # noqa
 

idom/server/__init__.py

@@ -13,7 +13,7 @@
 for name in ["sanic"]:
     try:
         import_module(name)
-    except ImportError:
+    except ImportError:  # pragma: no cover
         pass
     else:
         import_module(__name__ + "." + name)

idom/widgets/common.py

@@ -63,12 +63,12 @@ def __init__(
         install: Union[str, bool] = False,
     ) -> None:
         if install:
-            if not client.module_exists("web_modules", package):
+            if not client.module_exists(package):
                 if isinstance(install, str):
                     client.install({install: package})
                 else:
                     client.install({package: package})
-            new_import_path = client.import_path("web_modules", package)
+            new_import_path = client.import_path(package)
             if new_import_path is None:
                 raise ValueError(f"Unexpectedly failed to find install of {package}")
             package = new_import_path

setup.cfg

@@ -29,10 +29,11 @@ xfail_strict = True
 addopts = --cov=idom
 
 [coverage:report]
-fail_under = 86
+fail_under = 87
 show_missing = True
 skip_covered = True
 sort = Miss
 exclude_lines =
     pragma: no cover
     \.\.\.
+    raise NotImplementedError