test(client): ignore lines from test coverage

Author: remarkablemark

.github/workflows/test.yml

@@ -21,12 +21,10 @@ jobs:
         run: npm ci --prefer-offline
 
       - name: Run server test
-        run: npm run test:server
+        run: npm run test:coverage
 
       - name: Generate coverage report
-        run: |
-          mkdir -p coverage
-          npx nyc report --reporter=text-lcov > coverage/lcov.info
+        run: npx nyc report --reporter=text-lcov > coverage/lcov.info
 
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v5

package.json

@@ -49,6 +49,7 @@
     "test:client": "npm run test:client:watch -- --single-run",
     "test:client:build": "NODE_ENV=test npm run build",
     "test:client:watch": "npm run test:client:build && karma start",
+    "test:coverage": "npm run test:server && nyc report --reporter=html",
     "test:esm": "npm run build && node --test test/esm",
     "test:server": "npm run build:cjs && nyc mocha"
   },

src/client/domparser.ts

@@ -46,8 +46,8 @@ if (typeof DOMParser === 'function') {
    * @returns - Document.
    */
   parseFromString = (html: string, tagName?: string): Document => {
+    /* istanbul ignore if */
     if (tagName) {
-      /* istanbul ignore next */
       html = `<${tagName}>${html}</${tagName}>`;
     }
 
@@ -74,6 +74,7 @@ if (typeof document === 'object' && document.implementation) {
    * @returns - Document
    */
   parseFromDocument = function (html: string, tagName?: string): Document {
+    /* istanbul ignore if */
     if (tagName) {
       const element = htmlDocument.documentElement.querySelector(tagName);
 
@@ -113,6 +114,7 @@ if (template && template.content) {
   };
 }
 
+/* istanbul ignore next */
 const createNodeList = () => document.createDocumentFragment().childNodes;
 
 /**
@@ -136,11 +138,13 @@ export default function domparser(html: string): NodeList {
       // so make sure to remove them if they don't actually exist
       if (!HEAD_TAG_REGEX.test(html)) {
         const element = doc.querySelector(HEAD);
+        /* istanbul ignore next */
         element?.parentNode?.removeChild(element);
       }
 
       if (!BODY_TAG_REGEX.test(html)) {
         const element = doc.querySelector(BODY);
+        /* istanbul ignore next */
         element?.parentNode?.removeChild(element);
       }
 
@@ -153,6 +157,7 @@ export default function domparser(html: string): NodeList {
 
       // if there's a sibling element, then return both elements
       if (BODY_TAG_REGEX.test(html) && HEAD_TAG_REGEX.test(html)) {
+        /* istanbul ignore next */
         return elements[0].parentNode?.childNodes ?? createNodeList();
       }
 
@@ -166,7 +171,10 @@ export default function domparser(html: string): NodeList {
         return parseFromTemplate(html);
       }
 
+      /* istanbul ignore next */
       const element = parseFromDocument(html, BODY).querySelector(BODY);
+
+      /* istanbul ignore next */
       return element?.childNodes ?? createNodeList();
     }
   }

src/client/utilities.ts

@@ -121,10 +121,12 @@ export function formatDOM(
       }
 
       case 3:
+        /* istanbul ignore next */
         current = new Text(revertEscapedCharacters(node.nodeValue ?? ''));
         break;
 
       case 8:
+        /* istanbul ignore next */
         current = new Comment(node.nodeValue ?? '');
         break;