Skip to content

Commit 5dfb83b

Browse files
committed
Update tests
1 parent 4011a35 commit 5dfb83b

File tree

4 files changed

+28
-29
lines changed

4 files changed

+28
-29
lines changed

src/sanescansrv/htmlgen.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -473,6 +473,8 @@ def jinja_arg_tag(
473473
if args:
474474
args = f" {args}"
475475
if kwargs:
476+
if not args:
477+
args += " "
476478
args += " ".join(_generate_html_attributes(kwargs))
477479
return f"<{type_}{args}>"
478480

@@ -494,12 +496,9 @@ def jinja_radio_select(
494496
}
495497
jinja_properties: tuple[str, ...] = ()
496498
if default is not None:
497-
default_tag = " ".join(
498-
_generate_html_attributes({"checked": "checked"}),
499-
)
500499
jinja_properties = (
501500
jinja_if_block(
502-
{f"value == {default}": default_tag},
501+
{f"value == {default}": 'checked="checked" '},
503502
block=False,
504503
),
505504
)

src/sanescansrv/server.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -335,11 +335,11 @@ class DeviceOptionDataClass:
335335
py_name: str = field(repr=False)
336336
active: bool
337337
settable: bool
338-
default: None | str | int | float | bool = field(
338+
default: str | int | float | bool | None = field(
339339
init=False,
340340
default=None,
341341
)
342-
value: None | str | int | float = field(
342+
value: str | int | float | bool | None = field(
343343
init=False,
344344
default=None,
345345
)

tests/test_htmlgen.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ def test_jinja_for_loop_else_content() -> None:
502502
[
503503
("p", ("jinja",), {}, "<p jinja>"),
504504
("p", (), {"fish": "false"}, '<p fish="false">'),
505-
("p", ("jinja",), {"fish": "false"}, '<p jinja fish="false">'),
505+
("p", ("jinja ",), {"fish": "false"}, '<p jinja fish="false">'),
506506
("i", (), {}, "<i>"),
507507
(
508508
"input",
@@ -512,7 +512,7 @@ def test_jinja_for_loop_else_content() -> None:
512512
),
513513
(
514514
"input",
515-
("jinja",),
515+
("jinja ",),
516516
{"type": "radio", "id": "0", "name": "test", "value_": "Example"},
517517
'<input jinja type="radio" id="0" name="test" value="Example">',
518518
),
@@ -542,7 +542,7 @@ def test_jinja_radio_select_default() -> None:
542542
assert (
543543
htmlgen.jinja_radio_select("submits", "option_data", "default text")
544544
== """{% for display, value in option_data.items() %}
545-
<input {% if value == default text %}checked="checked"{% endif %} type="radio" id="submits_{{ loop.index0 }}" name="submits" value="{{ value }}">
545+
<input {% if value == default text %}checked="checked" {% endif %}type="radio" id="submits_{{ loop.index0 }}" name="submits" value="{{ value }}">
546546
<label for="submits_{{ loop.index0 }}">{{ display }}</label>
547547
<br>
548548
{% endfor %}"""

uv.lock

Lines changed: 20 additions & 20 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)