From 1be05f5d9e1f0075d17fdfde4ccc182d10de5aac Mon Sep 17 00:00:00 2001 From: Mikhail Golubev Date: Thu, 19 Dec 2024 16:42:59 +0200 Subject: [PATCH] PY-78189 Flip expected and actual types in warnings about __set__ argument (cherry picked from commit 65f8c92b852539908c71715756ec407781a9fb6b) IJ-CR-152050 GitOrigin-RevId: 2672bc7f5d39dd2bec1a9b2c2d322dffb3d66062 --- .../inspections/PyTypeCheckerInspection.java | 3 ++- .../Py3TypeCheckerInspectionTest.java | 20 +++++++++---------- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/python/python-psi-impl/src/com/jetbrains/python/inspections/PyTypeCheckerInspection.java b/python/python-psi-impl/src/com/jetbrains/python/inspections/PyTypeCheckerInspection.java index 3bd925ad34ff..4e3fecc75bc9 100644 --- a/python/python-psi-impl/src/com/jetbrains/python/inspections/PyTypeCheckerInspection.java +++ b/python/python-psi-impl/src/com/jetbrains/python/inspections/PyTypeCheckerInspection.java @@ -230,7 +230,8 @@ public class PyTypeCheckerInspection extends PyInspection { String expectedName = PythonDocumentationProvider.getVerboseTypeName(expected, myTypeEvalContext); String actualName = PythonDocumentationProvider.getTypeName(actual, myTypeEvalContext); registerProblem(value, descriptor ? - PyPsiBundle.message("INSP.type.checker.expected.type.from.dunder.set.got.type.instead", actualName, expectedName) : + PyPsiBundle.message("INSP.type.checker.expected.type.from.dunder.set.got.type.instead", + expectedName, actualName) : PyPsiBundle.message("INSP.type.checker.expected.type.got.type.instead", expectedName, actualName)); } } diff --git a/python/testSrc/com/jetbrains/python/inspections/Py3TypeCheckerInspectionTest.java b/python/testSrc/com/jetbrains/python/inspections/Py3TypeCheckerInspectionTest.java index ac1a7adc0f35..8b6f1b027cd9 100644 --- a/python/testSrc/com/jetbrains/python/inspections/Py3TypeCheckerInspectionTest.java +++ b/python/testSrc/com/jetbrains/python/inspections/Py3TypeCheckerInspectionTest.java @@ -2227,8 +2227,8 @@ def foo(param: str | int) -> TypeGuard[str]: t = Test() t.member = "str" - t.member = 123 - t.member = list + t.member = 123 + t.member = list """); } @@ -2245,8 +2245,8 @@ def foo(param: str | int) -> TypeGuard[str]: t = Test() t.member = "str" - t.member = 123 - t.member = list + t.member = 123 + t.member = list """); } @@ -2278,10 +2278,10 @@ def foo(param: str | int) -> TypeGuard[str]: t = Test() t.member = "abc" - t.member = 42 + t.member = 42 p = Prod() - p.member = "abc" - p.member = 42 + p.member = "abc" + p.member = 42 """); } @@ -2300,8 +2300,8 @@ def foo(param: str | int) -> TypeGuard[str]: t = Test() t.member = 42 - t.member = 43 - t.member = "42" + t.member = 43 + t.member = "42" """); } @@ -2320,7 +2320,7 @@ def foo(param: str | int) -> TypeGuard[str]: x = Test("foo") - x.member = 42 + x.member = 42 """); }