TBD

Author: BeyondEvil

pyproject.toml

@@ -42,7 +42,7 @@ classifiers = [
 ]
 dependencies = [
   "pytest>=7.0.0",
-  "pytest-metadata>=2.0.2",
+  "pytest-metadata>=3.0.0",
   "Jinja2>=3.0.0",
 ]
 dynamic = [

src/pytest_html/basereport.py

@@ -3,6 +3,7 @@
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 import datetime
 import json
+import math
 import os
 import re
 import warnings
@@ -12,8 +13,6 @@
 
 from pytest_html import __version__
 from pytest_html import extras
-from pytest_html.table import Header
-from pytest_html.table import Row
 from pytest_html.util import cleanup_unserializable
 
 
@@ -60,8 +59,8 @@ def _generate_report(self, self_contained=False):
 
         self._write_report(rendered_report)
 
-    def _generate_environment(self):
-        metadata = self._config._metadata
+    def _generate_environment(self, metadata_key):
+        metadata = self._config.stash[metadata_key]  # self._config._metadata
         for key in metadata.keys():
             value = metadata[key]
             if self._is_redactable_environment_variable(key):
@@ -145,16 +144,16 @@ def _write_report(self, rendered_report):
 
     @pytest.hookimpl(trylast=True)
     def pytest_sessionstart(self, session):
-        config = session.config
-        if hasattr(config, "_metadata") and config._metadata:
-            self._report.set_data("environment", self._generate_environment())
+        # config = session.config
+        from pytest_metadata.plugin import metadata_key
+
+        # if hasattr(config, "_metadata") and config._metadata:
+        self._report.set_data("environment", self._generate_environment(metadata_key))
 
         session.config.hook.pytest_html_report_title(report=self._report)
 
-        header_cells = Header()
-        session.config.hook.pytest_html_results_table_header(cells=header_cells)
-        self._report.set_data("resultsTableHeader", header_cells.html)
-        self._report.set_data("headerPops", header_cells.get_pops())
+        headers = self._report.data["resultsTableHeader"]
+        session.config.hook.pytest_html_results_table_header(cells=headers)
 
         self._report.set_data("runningState", "Started")
         self._generate_report()
@@ -173,7 +172,8 @@ def pytest_sessionfinish(self, session):
     @pytest.hookimpl(trylast=True)
     def pytest_terminal_summary(self, terminalreporter):
         terminalreporter.write_sep(
-            "-", f"Generated html report: file://{self._report_path.resolve()}"
+            "-",
+            f"Generated html report: file://{self._report_path.resolve().as_posix()}",
         )
 
     @pytest.hookimpl(trylast=True)
@@ -188,35 +188,58 @@ def pytest_runtest_logreport(self, report):
                 DeprecationWarning,
             )
 
+        data = dict()
+        cells = list()
+
+        result = _process_outcome(report)
         data = {
-            "duration": report.duration,
+            "result": result,
         }
+        cells.append(f'<td class="col-result">{result}</td>')
 
         test_id = report.nodeid
         if report.when != "call":
             test_id += f"::{report.when}"
         data["testId"] = test_id
+        cells.append(f'<td class="col-name">{test_id}</td>')
+
+        total_duration = self._report.data["totalDuration"]
+        total_duration["total"] += report.duration
+        total_duration["formatted"] = _format_duration(total_duration["total"])
 
-        row_cells = Row()
-        self._config.hook.pytest_html_results_table_row(report=report, cells=row_cells)
-        if row_cells.html is None:
+        duration = _format_duration(report.duration)
+        cells.append(f'<td class="col-duration">{duration}</td>')
+        cells.append('<td class="col-links"></td>')
+
+        self._config.hook.pytest_html_results_table_row(report=report, cells=cells)
+        if not cells:
             return
-        data["resultsTableRow"] = row_cells.html
-        for sortable, value in row_cells.sortables.items():
-            data[sortable] = value
+
+        data["resultsTableRow"] = cells
+        data["extras"] = self._process_extras(report, test_id)
 
         processed_logs = _process_logs(report)
         self._config.hook.pytest_html_results_table_html(
             report=report, data=processed_logs
         )
 
-        data["result"] = _process_outcome(report)
-        data["extras"] = self._process_extras(report, test_id)
-
         if self._report.add_test(data, report, processed_logs):
             self._generate_report()
 
 
+def _format_duration(duration):
+    if duration < 1:
+        return "{} ms".format(round(duration * 1000))
+
+    hours = math.floor(duration / 3600)
+    remaining_seconds = duration % 3600
+    minutes = math.floor(remaining_seconds / 60)
+    remaining_seconds = remaining_seconds % 60
+    seconds = round(remaining_seconds)
+
+    return f"{hours:02d}:{minutes:02d}:{seconds:02d}"
+
+
 def _is_error(report):
     return report.when in ["setup", "teardown"] and report.outcome == "failed"
 

src/pytest_html/report_data.py

@@ -10,14 +10,26 @@
 class ReportData:
     def __init__(self, config):
         self._config = config
+
+        default_headers = [
+            '<th class="sortable" data-column-type="result">Result</th>',
+            '<th class="sortable" data-column-type="testId">Test</th>',
+            '<th class="sortable" data-column-type="duration">Duration</th>',
+            "<th>Links</th>",
+        ]
+
         self._data = {
             "title": "",
             "collectedItems": 0,
+            "totalDuration": {
+                "total": 0,
+                "formatted": "",
+            },
             "runningState": "not_started",
             "environment": {},
             "tests": defaultdict(list),
-            "resultsTableHeader": {},
             "additionalSummary": defaultdict(list),
+            "resultsTableHeader": default_headers,
         }
 
         collapsed = config.getini("render_collapsed")

src/pytest_html/resources/index.jinja2

@@ -32,10 +32,6 @@
     <template id="template_results-table__tbody">
       <tbody class="results-table-row">
         <tr class="collapsible">
-          <td class="col-result"></td>
-          <td class="col-name"></td>
-          <td class="col-duration"></td>
-          <td class="col-links"></td>
         </tr>
         <tr class="extras-row">
           <td class="extra" colspan="4">
@@ -62,10 +58,6 @@
     <template id="template_results-table__head">
       <thead id="results-table-head">
         <tr>
-          <th class="sortable" data-column-type="result">Result</th>
-          <th class="sortable" data-column-type="testId">Test</th>
-          <th class="sortable" data-column-type="duration">Duration</th>
-          <th>Links</th>
         </tr>
       </thead>
     </template>

src/pytest_html/resources/style.css

@@ -50,6 +50,9 @@ class DataManager {
     get isFinished() {
         return this.data.runningState === 'Finished'
     }
+    get formattedDuration() {
+        return this.data.totalDuration.formatted
+    }
 }
 
 module.exports = {

src/pytest_html/scripts/datamanager.js

@@ -1,5 +1,4 @@
 const storageModule = require('./storage.js')
-const { formatDuration, transformTableObj } = require('./utils.js')
 const mediaViewer = require('./mediaviewer.js')
 const templateEnvRow = document.querySelector('#template_environment_row')
 const templateCollGroup = document.querySelector('#template_table-colgroup')
@@ -28,12 +27,6 @@ const findAll = (selector, elem) => {
     return [...elem.querySelectorAll(selector)]
 }
 
-const insertAdditionalHTML = (html, element, selector, position = 'beforebegin') => {
-    Object.keys(html).map((key) => {
-        element.querySelectorAll(selector).item(key).insertAdjacentHTML(position, html[key])
-    })
-}
-
 const dom = {
     getStaticRow: (key, value) => {
         const envRow = templateEnvRow.content.cloneNode(true)
@@ -53,29 +46,14 @@ const dom = {
         const sortAttr = storageModule.getSort()
         const sortAsc = JSON.parse(storageModule.getSortDirection())
 
-        const regex = /data-column-type="(\w+)/
-        const cols = Object.values(resultsTableHeader).reduce((result, value) => {
-            if (value.includes('sortable')) {
-                const matches = regex.exec(value)
-                if (matches) {
-                    result.push(matches[1])
-                }
-            }
-            return result
-        }, [])
-        const sortables = ['result', 'testId', 'duration', ...cols]
-
-        // Add custom html from the pytest_html_results_table_header hook
-        const headers = transformTableObj(resultsTableHeader)
-        insertAdditionalHTML(headers.inserts, header, 'th')
-        insertAdditionalHTML(headers.appends, header, 'tr', 'beforeend')
-
-        sortables.forEach((sortCol) => {
-            if (sortCol === sortAttr) {
-                header.querySelector(`[data-column-type="${sortCol}"]`).classList.add(sortAsc ? 'desc' : 'asc')
-            }
+        resultsTableHeader.forEach((html) => {
+            const t = document.createElement('template')
+            t.innerHTML = html
+            header.querySelector('#results-table-head > tr').appendChild(t.content)
         })
 
+        header.querySelector(`.sortable[data-column-type="${sortAttr}"]`).classList.add(sortAsc ? 'desc' : 'asc')
+
         return header
     },
     getListHeaderEmpty: () => listHeaderEmpty.content.cloneNode(true),
@@ -86,12 +64,13 @@ const dom = {
         resultBody.querySelector('tbody').classList.add(resultLower)
         resultBody.querySelector('tbody').id = testId
         resultBody.querySelector('.collapsible').dataset.id = id
-        resultBody.querySelector('.col-result').innerText = result
-        resultBody.querySelector('.col-result').classList.add(`${collapsed ? 'expander' : 'collapser'}`)
-        resultBody.querySelector('.col-name').innerText = testId
 
-        const formattedDuration = duration < 1 ? formatDuration(duration).ms : formatDuration(duration).formatted
-        resultBody.querySelector('.col-duration').innerText = formattedDuration
+        resultsTableRow.forEach((html) => {
+            const t = document.createElement('template')
+            t.innerHTML = html
+            resultBody.querySelector('.collapsible').appendChild(t.content)
+        })
+        resultBody.querySelector('.collapsible > td')?.classList.add(`${collapsed ? 'expander' : 'collapser'}`)
 
         if (log) {
             // Wrap lines starting with "E" with span.error to color those lines red
@@ -107,7 +86,7 @@ const dom = {
 
         const media = []
         extras?.forEach(({ name, format_type, content }) => {
-            if (['json', 'text', 'url'].includes(format_type)) {
+            if (resultBody.querySelector('.col-links') && ['json', 'text', 'url'].includes(format_type)) {
                 const extraLink = aTag.content.cloneNode(true)
                 const extraLinkItem = extraLink.querySelector('a')
 
@@ -127,11 +106,6 @@ const dom = {
         })
         mediaViewer.setUp(resultBody, media)
 
-        // Add custom html from the pytest_html_results_table_row hook
-        const rows = transformTableObj(resultsTableRow)
-        resultsTableRow && insertAdditionalHTML(rows.inserts, resultBody, 'td')
-        resultsTableRow && insertAdditionalHTML(rows.appends, resultBody, 'tr', 'beforeend')
-
         // Add custom html from the pytest_html_results_table_html hook
         tableHtml?.forEach((item) => {
             resultBody.querySelector('td[class="extra"]').insertAdjacentHTML('beforeend', item)

src/pytest_html/scripts/dom.js

@@ -1,4 +1,3 @@
-const { formatDuration } = require('./utils.js')
 const { dom, findAll } = require('./dom.js')
 const { manager } = require('./datamanager.js')
 const { doSort } = require('./sort.js')
@@ -52,18 +51,6 @@ const renderContent = (tests) => {
         item.colSpan = document.querySelectorAll('th').length
     })
 
-    const { headerPops } = manager.renderData
-    if (headerPops > 0) {
-        // remove 'headerPops' number of header columns
-        findAll('#results-table-head th').splice(-headerPops).forEach((column) => column.remove())
-
-        // remove 'headerPops' number of row columns
-        const resultRows = findAll('.results-table-row')
-        resultRows.forEach((elem) => {
-            findAll('td:not(.extra)', elem).splice(-headerPops).forEach((column) => column.remove())
-        })
-    }
-
     findAll('.sortable').forEach((elem) => {
         elem.addEventListener('click', (evt) => {
             const { target: element } = evt
@@ -80,7 +67,7 @@ const renderContent = (tests) => {
     })
 }
 
-const renderDerived = (tests, collectedItems, isFinished) => {
+const renderDerived = (tests, collectedItems, isFinished, formattedDuration) => {
     const currentFilter = getVisible()
     possibleResults.forEach(({ result, label }) => {
         const count = tests.filter((test) => test.result.toLowerCase() === result).length
@@ -100,12 +87,8 @@ const renderDerived = (tests, collectedItems, isFinished) => {
         ['Passed', 'Failed', 'XPassed', 'XFailed'].includes(result)).length
 
     if (isFinished) {
-        const accTime = tests.reduce((prev, { duration }) => prev + duration, 0)
-        const formattedAccTime = formatDuration(accTime)
         const testWord = numberOfTests > 1 ? 'tests' : 'test'
-        const durationText = formattedAccTime.hasOwnProperty('ms') ? formattedAccTime.ms : formattedAccTime.formatted
-
-        document.querySelector('.run-count').innerText = `${numberOfTests} ${testWord} took ${durationText}.`
+        document.querySelector('.run-count').innerText = `${numberOfTests} ${testWord} took ${formattedDuration}.`
         document.querySelector('.summary__reload__button').classList.add('hidden')
     } else {
         document.querySelector('.run-count').innerText = `${numberOfTests} / ${collectedItems} tests done`
@@ -135,11 +118,11 @@ const bindEvents = () => {
 }
 
 const redraw = () => {
-    const { testSubset, allTests, collectedItems, isFinished } = manager
+    const { testSubset, allTests, collectedItems, isFinished, formattedDuration } = manager
 
     renderStatic()
     renderContent(testSubset)
-    renderDerived(allTests, collectedItems, isFinished)
+    renderDerived(allTests, collectedItems, isFinished, formattedDuration )
 }
 
 exports.redraw = redraw