build: refactor building steps

.github/workflows/unit.yaml

@@ -1,8 +1,10 @@
 name: Build
 
 on:
-  - push
-  - pull_request
+  push:
+    branches:
+      - main
+  pull_request:
 
 jobs:
   lint:

noxfile.py

@@ -13,7 +13,7 @@ def docs(session):
     session.install(".[doc]")
     b = session.posargs[0] if session.posargs else "html"
     dst = Path(__file__).parent / "docs" / "_build" / b
-    session.run("sphinx-build", f"-b={b}", "-a", "-E", "docs", str(dst))
+    session.run("sphinx-build", f"-b={b}", "-a", "-E", "docs", dst.resolve())
 
 
 @nox.session(reuse_venv=True)
@@ -34,6 +34,6 @@ def mypy(session):
 @nox.session(reuse_venv=True)
 def test(session):
     """Run all the test using the environment varialbe of the running machine."""
-    session.install("--verbose", ".[test]")
-    # session.run("pytest", "--color=yes", "--cov", "--cov-report=html", *test_files)
-    session.run("sphinx-build", "--bug-report")
+    session.install(".[test]")
+    test_files = session.posargs or ["tests"]
+    session.run("pytest", "--color=yes", "--cov", "--cov-report=xml", *test_files)

pyproject.toml

@@ -48,7 +48,8 @@ test = [
     "pytest",
     "beautifulsoup4",
     "pytest-regressions",
-    "pytest-cov"
+    "pytest-cov",
+    "defusedxml",
 ]
 doc = [
     "pydata-sphinx-theme",
@@ -88,6 +89,8 @@ convention = "google"
 
 [tool.coverage.run]
 source = ["sphinxcontrib.icon"]
+branch = true
+relative_files = true
 
 [tool.mypy]
 scripts_are_modules = true

setup.py

@@ -3,9 +3,7 @@
 
 from pynpm import NPMPackage
 from setuptools import Command, setup
-from setuptools.command.build_py import build_py
 from setuptools.command.egg_info import egg_info
-from setuptools.command.sdist import sdist
 
 ROOT = Path(__file__).parent
 
@@ -16,8 +14,7 @@
 
 def update_package_data(distribution) -> None:
     """Update package_data to catch changes during setup."""
-    build_py = distribution.get_command_obj("build_py")
-    build_py.finalize_options()
+    distribution.get_command_obj("build_py").finalize_options()
 
 
 def js_prerelease(command: Command, strict: bool = False) -> Command:
@@ -28,36 +25,12 @@ class DecoratedCommand(command):
 
         def run(self) -> None:
             """Run the command."""
-            self.distribution.run_command("jsdeps")
-            command.run(self)
+            package = ROOT / "sphinxcontrib" / "icon" / "package.json"
+            NPMPackage(package.resolve(), shell=is_windows).install()
             update_package_data(self.distribution)
+            command.run(self)
 
     return DecoratedCommand
 
 
-class NPM(Command):
-    """install package.json dependencies using npm."""
-
-    def initialize_options(self):
-        """Ignore initialize_options."""
-        pass
-
-    def finalize_options(self):
-        """Ignore finalize_options."""
-        pass
-
-    def run(self):
-        """Run the command."""
-        package = ROOT / "sphinxcontrib" / "icon" / "package.json"
-        NPMPackage(str(package), shell=is_windows).install()
-        update_package_data(self.distribution)
-
-
-setup(
-    cmdclass={
-        "build_py": js_prerelease(build_py),
-        "egg_info": js_prerelease(egg_info),
-        "sdist": js_prerelease(sdist, strict=True),
-        "jsdeps": NPM,
-    }
-)
+setup(cmdclass={"egg_info": js_prerelease(egg_info)})

sphinxcontrib/icon/__init__.py

@@ -22,7 +22,6 @@ def setup(app: Sphinx) -> Dict[str, Any]:
     font_handler = Fontawesome()
 
     # install html related files
-    raise Exception(str(font_handler.css_file.resolve()))
     app.add_css_file(str(font_handler.css_file.resolve()))
     app.add_js_file(str(font_handler.js_file.resolve()))
 

sphinxcontrib/icon/font_handler.py

@@ -43,7 +43,7 @@ def add_latex_font_files(self, app: Sphinx) -> None:
             dst = Path(app.builder.outdir) / f.name
             logger.info(f"Writing: {f.name}")
             ensuredir(app.builder.outdir)
-            copyfile(str(f), str(dst))
+            copyfile(f.resolve(), dst.resolve())
 
     def add_latex_font(self, app: Sphinx, config: Config) -> None:
         """Add the fontawesome fontfamily in the preamble of the .tex file."""