[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-19 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: These parameters were added in d99ef9a9df093d3443996725cd9dcac5f113f176, but they were not tested nor documented. An indirect test was added later in issue12376. If it is an official feature it needs documentation and tests. Otherwise we can remove these

[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-18 Thread dhruv
dhruv added the comment: "Required" meaning that we cannot remove them from the constructor signature even though they aren't used within it. Hope that clears it up! -- ___ Python tracker

[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-16 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: What do you mean they are required? All parameters of TestResult() are optional. -- nosy: +ezio.melotti, michael.foord, rbcollins, serhiy.storchaka status: pending -> open ___ Python tracker

[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-15 Thread dhruv
Change by dhruv : -- status: open -> pending ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-15 Thread dhruv
Change by dhruv : -- keywords: +patch pull_requests: +28351 stage: -> patch review pull_request: https://github.com/python/cpython/pull/30132 ___ Python tracker ___

[issue46094] Missing unit test on unittest.TestResult to check for required arguments

2021-12-15 Thread dhruv
New submission from dhruv : stream, descriptions, and verbosity seem like they aren't needed at first glance but are required to work with multiple inheritance. A simple test can clarify this. -- components: Tests messages: 408674 nosy: DreamSh0t priority: normal severity: normal