diff options
| author | Serhiy Storchaka <storchaka@gmail.com> | 2019-09-01 12:16:51 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-09-01 12:16:51 +0300 |
| commit | 1f21eaa15e8a0d2b0f78d0e3f2b9e5b458eb0a70 (patch) | |
| tree | 8db3de2421c1d45d018a7a1dc03f42a0797acee2 /Lib/test/datetimetester.py | |
| parent | 5eca7f3f3836cc734dfe8dc5ec669f3b4e9333fe (diff) | |
| download | cpython-git-1f21eaa15e8a0d2b0f78d0e3f2b9e5b458eb0a70.tar.gz | |
bpo-15999: Clean up of handling boolean arguments. (GH-15610)
* Use the 'p' format unit instead of manually called PyObject_IsTrue().
* Pass boolean value instead 0/1 integers to functions that needs boolean.
* Convert some arguments to boolean only once.
Diffstat (limited to 'Lib/test/datetimetester.py')
| -rw-r--r-- | Lib/test/datetimetester.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/Lib/test/datetimetester.py b/Lib/test/datetimetester.py index d09255d79b..d5de5e91a9 100644 --- a/Lib/test/datetimetester.py +++ b/Lib/test/datetimetester.py @@ -6160,7 +6160,7 @@ class CapiTest(unittest.TestCase): def test_date_from_date(self): exp_date = date(1993, 8, 26) - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro): c_api_date = _testcapi.get_date_fromdate( macro, @@ -6173,7 +6173,7 @@ class CapiTest(unittest.TestCase): def test_datetime_from_dateandtime(self): exp_date = datetime(1993, 8, 26, 22, 12, 55, 99999) - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro): c_api_date = _testcapi.get_datetime_fromdateandtime( macro, @@ -6191,7 +6191,7 @@ class CapiTest(unittest.TestCase): exp_date = datetime(1993, 8, 26, 22, 12, 55, 99999) for fold in [0, 1]: - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro, fold=fold): c_api_date = _testcapi.get_datetime_fromdateandtimeandfold( macro, @@ -6210,7 +6210,7 @@ class CapiTest(unittest.TestCase): def test_time_from_time(self): exp_time = time(22, 12, 55, 99999) - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro): c_api_time = _testcapi.get_time_fromtime( macro, @@ -6225,7 +6225,7 @@ class CapiTest(unittest.TestCase): exp_time = time(22, 12, 55, 99999) for fold in [0, 1]: - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro, fold=fold): c_api_time = _testcapi.get_time_fromtimeandfold( macro, @@ -6241,7 +6241,7 @@ class CapiTest(unittest.TestCase): def test_delta_from_dsu(self): exp_delta = timedelta(26, 55, 99999) - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro): c_api_delta = _testcapi.get_delta_fromdsu( macro, @@ -6254,7 +6254,7 @@ class CapiTest(unittest.TestCase): def test_date_from_timestamp(self): ts = datetime(1995, 4, 12).timestamp() - for macro in [0, 1]: + for macro in False, True: with self.subTest(macro=macro): d = _testcapi.get_date_fromtimestamp(int(ts), macro) @@ -6272,7 +6272,7 @@ class CapiTest(unittest.TestCase): from_timestamp = _testcapi.get_datetime_fromtimestamp for case in cases: - for macro in [0, 1]: + for macro in False, True: with self.subTest(case=case, macro=macro): dtup, tzinfo, usetz = case dt_orig = datetime(*dtup, tzinfo=tzinfo) |