Skip to content

Commit 874ca55

Browse files
committed
address review
1 parent 0b1d8b3 commit 874ca55

2 files changed

Lines changed: 5 additions & 8 deletions

File tree

conformance/results/results.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -895,7 +895,7 @@ <h3>Python Type System Conformance Test Results</h3>
895895
<th class="column col2 conformant">Pass</th>
896896
<th class="column col2 conformant">Pass</th>
897897
<th class="column col2 conformant">Pass</th>
898-
<th class="column col2 conformant"><div class="hover-text">Pass*<span class="tooltip-text" id="bottom"><p>Does not pass all assertions in the test file, due to limitations in ty's generics solver.</p><p></p><p>Incorrectly rejects a converter of `dict` combined with a default of `()`.</p><p>Incorrectly rejects passing a sequence of two-element string tuples to the `dict` constructor.</p></span></div></th>
898+
<th class="column col2 partially-conformant"><div class="hover-text">Partial<span class="tooltip-text" id="bottom"><p>Does not pass all assertions in the test file, due to limitations in ty's generics solver.</p><p></p><p>Incorrectly rejects a converter of `dict` combined with a default of `()`.</p><p>Incorrectly rejects passing a sequence of two-element string tuples to the `dict` constructor.</p></span></div></th>
899899
</tr>
900900
<tr><th class="column col1">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;dataclasses_transform_field</th>
901901
<th class="column col2 partially-conformant"><div class="hover-text">Partial<span class="tooltip-text" id="bottom"><p>Does not properly handle field constructor that has default value for `kw_only` or `init` parameter.</p></span></div></th>

conformance/results/ty/dataclasses_transform_converter.toml

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,14 @@
1-
conformance_automated = "Pass"
2-
conformant = "Pass"
1+
conformance_automated = "Fail"
2+
conformant = "Partial"
33
notes = """
44
Does not pass all assertions in the test file, due to limitations in ty's generics solver.
55
66
Incorrectly rejects a converter of `dict` combined with a default of `()`.
77
Incorrectly rejects passing a sequence of two-element string tuples to the `dict` constructor.
88
"""
9-
ignore_errors = [
10-
"Object of type `dict[str, str] | dict[bytes, bytes]` is not assignable to `dict[str, str]`",
11-
"Argument to function `model_field` is incorrect: Expected `(Iterable[list[str]] | Iterable[list[bytes]], /) -> dict[str, str] | dict[bytes, bytes]`, found `<class 'dict'>`",
12-
"Argument is incorrect: Expected `dict[str, str]`, found `tuple[tuple[Literal[\"a\"], Literal[\"1\"]], tuple[Literal[\"b\"], Literal[\"2\"]]]`",
13-
]
149
errors_diff = """
10+
Line 104: Unexpected errors ['dataclasses_transform_converter.py:104:30: error[invalid-assignment] Object of type `dict[str, str] | dict[bytes, bytes]` is not assignable to `dict[str, str]`', "dataclasses_transform_converter.py:104:42: error[invalid-argument-type] Argument to function `model_field` is incorrect: Expected `(Iterable[list[str]] | Iterable[list[bytes]], /) -> dict[str, str] | dict[bytes, bytes]`, found `<class 'dict'>`"]
11+
Line 121: Unexpected errors ['dataclasses_transform_converter.py:121:40: error[invalid-argument-type] Argument is incorrect: Expected `dict[str, str]`, found `tuple[tuple[Literal["a"], Literal["1"]], tuple[Literal["b"], Literal["2"]]]`']
1512
"""
1613
output = """
1714
dataclasses_transform_converter.py:48:31: error[invalid-argument-type] Argument to function `model_field` is incorrect: Expected `(Unknown, /) -> Unknown`, found `def bad_converter1() -> int`

0 commit comments

Comments
 (0)