From 4661af99c3cb3a2b1e70f92b798b82ddd8d26190 Mon Sep 17 00:00:00 2001 From: Terrance Date: Sun, 11 Dec 2022 18:07:09 +0000 Subject: [PATCH] Annotate `unittest.TestCase.skipTest()` as no-return (#9352) Annotate unittest.TestCase.skipTest() as no-return This method unconditionally raises unittest.SkipTest, which ends a test method early. unittest.TestCase.fail() works similarly, and is already annotated with NoReturn to indicate this behaviour. --- stdlib/unittest/case.pyi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/stdlib/unittest/case.pyi b/stdlib/unittest/case.pyi index c75539a97..42633ed13 100644 --- a/stdlib/unittest/case.pyi +++ b/stdlib/unittest/case.pyi @@ -104,7 +104,7 @@ class TestCase: def tearDownClass(cls) -> None: ... def run(self, result: unittest.result.TestResult | None = ...) -> unittest.result.TestResult | None: ... def __call__(self, result: unittest.result.TestResult | None = ...) -> unittest.result.TestResult | None: ... - def skipTest(self, reason: Any) -> None: ... + def skipTest(self, reason: Any) -> NoReturn: ... def subTest(self, msg: Any = ..., **params: Any) -> AbstractContextManager[None]: ... def debug(self) -> None: ... if sys.version_info < (3, 11):