only create links for text, url and json (#22)

Author: BeyondEvil

src/pytest_html/nextgen.py

@@ -206,6 +206,10 @@ def pytest_sessionfinish(self, session):
         self._report.data["runningState"] = "Finished"
         self._generate_report()
 
+    @pytest.hookimpl(trylast=True)
+    def pytest_terminal_summary(self, terminalreporter):
+        terminalreporter.write_sep("-", f"Generated html report: file://{self._report_path}")
+
     @pytest.hookimpl(trylast=True)
     def pytest_collection_finish(self, session):
         self._report.data["collectedItems"] = len(session.items)

src/pytest_html/resources/index.jinja2

@@ -73,7 +73,7 @@
     </template>
     <template id="template_results-table__head--empty">
       <tr id="not-found-message">
-        <th colspan="4">No results found. Try to check the filters</th>
+        <th colspan="4">No results found. Check the filters.</th>
       </tr>
     </template>
     <!-- END TEMPLATES -->

src/pytest_html/scripts/dom.js

@@ -82,20 +82,22 @@ const dom = {
 
         const media = []
         extras?.forEach(({ name, format_type, content }) => {
-            const extraLink = aTag.content.cloneNode(true)
-            const extraLinkItem = extraLink.querySelector('a')
-
-            extraLinkItem.href = content
-            extraLinkItem.className = `col-links__extra ${format_type}`
-            extraLinkItem.innerText = name
-            resultBody.querySelector('.col-links').appendChild(extraLinkItem)
+            if (['json', 'text', 'url'].includes(format_type)) {
+                const extraLink = aTag.content.cloneNode(true)
+                const extraLinkItem = extraLink.querySelector('a')
+
+                extraLinkItem.href = content
+                extraLinkItem.className = `col-links__extra ${format_type}`
+                extraLinkItem.innerText = name
+                resultBody.querySelector('.col-links').appendChild(extraLinkItem)
+            }
 
             if (['image', 'video'].includes(format_type)) {
                 media.push({ path: content, name, format_type })
             }
 
             if (format_type === 'html') {
-              resultBody.querySelector('.extraHTML').insertAdjacentHTML('beforeend', `<div>${content}</div>`)
+                resultBody.querySelector('.extraHTML').insertAdjacentHTML('beforeend', `<div>${content}</div>`)
             }
         })
         mediaViewer.setUp(resultBody, media)