Skip to content

Commit

Permalink
[fix] Links in static HTML files should be relative
Browse files Browse the repository at this point in the history
The links in the static HTML files generated by "CodeChecker parse -e html"
should be relative, so the output folder of the above command is portable.
  • Loading branch information
bruntib committed May 3, 2024
1 parent 2afffe6 commit f233107
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,7 @@ def create_index_html(self, output_dir: str):
html_report_links.append({'link': html_file, 'report': report})

table_reports = map(lambda data: {
'link': data['link'],
'link': os.path.basename(data['link']),
'file-path': data['report']['fileId'],
'report-hash': data['report']['reportHash'],
'checker-name': data['report']['checker']['name'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -186,4 +186,10 @@ def test_html_for_inclusion(self):
index_html = os.path.join(output_dir, "index.html")

with open(index_html, 'r', encoding="utf-8", errors="ignore") as f:
self.assertEqual(len(re.findall('"link": "', f.read())), 3)
content = f.read()

# There are 3 reports in the test file.
self.assertEqual(len(re.findall('"link": "', content)), 3)
# The links should be relative so the static HTML folder is
# prtable.
self.assertNotIn('"link": "/', content)

0 comments on commit f233107

Please sign in to comment.