Thanks to visit codestin.com
Credit goes to github.com

Skip to content

stubtest: check Final variables with literal values against runtime #20858

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hauntsaninja merged 5 commits into from
Feb 23, 2026
Merged

stubtest: check Final variables with literal values against runtime #20858

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
380d5d3
stubtest: check Final variables with literal values against runtime
Vikash-Kumar-23 Feb 22, 2026
2cc6b43
Update mypy/stubtest.py
Vikash-Kumar-23 Feb 22, 2026
b997e14
Refactor: Move Final check to elif branch and clean error string
Vikash-Kumar-23 Feb 22, 2026
8949fd4
Update mypy/stubtest.py
hauntsaninja Feb 22, 2026
8d63f8e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions mypy/stubtest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1279,6 +1279,14 @@ def verify_var(
yield Error(object_path, "is not present at runtime", stub, runtime)
return

if stub.final_value is not None and stub.final_value != runtime:
Copy link
Copy Markdown
Collaborator

@brianschubert brianschubert Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe move this to an elif branch of the subtype check below? That way we don't get duplicate errors when the types are different.

yield Error(
object_path,
f"is inconsistent, stub value {stub.final_value!r} differs from runtime value {runtime!r}",
Copy link
Copy Markdown
Collaborator

@brianschubert brianschubert Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The runtime object could potentially have a very long repr, which we wouldn't want included in the error message. It's also already rendered in the output (with length truncation), so including in the message isn't necessary.

stub,
runtime,
Comment thread
Vikash-Kumar-23 marked this conversation as resolved.
Outdated
)

if (
stub.is_initialized_in_class
and is_read_only_property(runtime)
Expand Down
16 changes: 16 additions & 0 deletions mypy/test/teststubtest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1171,6 +1171,22 @@ def read_write_attr(self, val): self._val = val
""",
error=None,
)
yield Case(
stub="""
from typing import Final
x_final: Final = 2
Copy link
Copy Markdown
Collaborator

@brianschubert brianschubert Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style nit: maybe use CAPS_CASE, like the other cases with Final variables?

""",
runtime="x_final = 1",
error="x_final",
)
yield Case(
stub="""
from typing import Final
x_final_ok: Final = 1
""",
runtime="x_final_ok = 1",
error=None,
)

@collect_cases
def test_type_alias(self) -> Iterator[Case]:
Expand Down