Skip to content

Commit 484c0ce

Browse files
authored
Merge pull request #277 from christiansandberg/fix-missing-image
Fix image missing when using Base64 content
2 parents 51f50f5 + fe7b622 commit 484c0ce

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

pytest_html/plugin.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,12 @@ def _make_media_html_div(
319319
href = src = self.create_asset(
320320
content, extra_index, test_index, extra.get("extension"), "wb"
321321
)
322-
html_div = html.a(class_=base_extra_class, target="_blank", href=href)
322+
html_div = html.a(
323+
raw(base_extra_string.format(src)),
324+
class_=base_extra_class,
325+
target="_blank",
326+
href=href,
327+
)
323328
return html_div
324329

325330
def _append_image(self, extra, extra_index, test_index):

testing/test_pytest_html.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,9 @@ def pytest_runtest_makereport(item, call):
502502
assert result.ret == 0
503503
src = f"assets/test_extra_image_separated.py__test_pass_0_0.{file_extension}"
504504
link = f'<a class="image" href="{src}" target="_blank">'
505+
img = f'<img src="{src}"/>'
505506
assert link in html
507+
assert img in html
506508
assert os.path.exists(src)
507509

508510
@pytest.mark.parametrize(
@@ -536,8 +538,10 @@ def test_fail():
536538
asset_name = "test_extra_image_separated_rerun.py__test_fail"
537539
src = f"assets/{asset_name}_0_{i}.{file_extension}"
538540
link = f'<a class="image" href="{src}" target="_blank">'
541+
img = f'<img src="{src}"/>'
539542
assert result.ret
540543
assert link in html
544+
assert img in html
541545
assert os.path.exists(src)
542546

543547
@pytest.mark.parametrize("src_type", ["https://", "file://", "image.png"])

0 commit comments

Comments
 (0)