Skip to content

Commit dc12665

Browse files
committed
urlencode image URLs
The images are path safe but not URL safe. Parametrized tests can include characters such as "\" or "?" in their name which can lead to 404 errors in the HTML summary report.
1 parent 7f94aae commit dc12665

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

pytest_mpl/summary/templates/basic.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ <h2>Image test comparison</h2>
7373
{%- endif %}
7474
</td>
7575
{% macro image(file) -%}
76-
<td>{% if file %}<img src="{{ file }}">{% endif %}</td>
76+
<td>{% if file %}<img src="{{ file | urlencode }}">{% endif %}</td>
7777
{%- endmacro -%}
7878
{{ image(result.baseline_image) }}
7979
{{ image(result.diff_image) }}

pytest_mpl/summary/templates/result.html

+5-5
Original file line numberDiff line numberDiff line change
@@ -12,19 +12,19 @@
1212
<div class="hover-image">
1313
{% if r.diff_image -%}
1414
<div class="diff-image">
15-
<img src="{{ r.diff_image }}" class="card-img-top" alt="diff image">
15+
<img src="{{ r.diff_image | urlencode }}" class="card-img-top" alt="diff image">
1616
</div>
1717
<div class="result-image">
18-
<img src="{{ r.result_image }}" class="card-img-top" alt="result image">
18+
<img src="{{ r.result_image | urlencode }}" class="card-img-top" alt="result image">
1919
</div>
2020
{%- else -%}
21-
<img src="{{ r.result_image }}" class="card-img-top" alt="result image">
21+
<img src="{{ r.result_image | urlencode }}" class="card-img-top" alt="result image">
2222
{%- endif %}
2323
</div>
2424
{%- elif r.result_image -%}
25-
<img src="{{ r.result_image }}" class="card-img-top" alt="result image">
25+
<img src="{{ r.result_image | urlencode }}" class="card-img-top" alt="result image">
2626
{%- elif r.baseline_image -%}
27-
<img src="{{ r.baseline_image }}" class="card-img-top" alt="baseline image">
27+
<img src="{{ r.baseline_image | urlencode }}" class="card-img-top" alt="baseline image">
2828
{%- endif %}
2929
</a>
3030
{% filter indent(width=8) -%}

pytest_mpl/summary/templates/result_images.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ <h5 class="card-title">{{ r.name }}</h5>
1313
<div class="card h-100">
1414
<div class="card-header">{{ name }}</div>
1515
{% if file -%}
16-
<img src="{{ file }}" class="card-img-top" alt="{{ name }}">
16+
<img src="{{ file | urlencode }}" class="card-img-top" alt="{{ name }}">
1717
{%- endif %}
1818
</div>
1919
</div>

0 commit comments

Comments
 (0)