Skip to content

Commit c8ac2d8

Browse files
committed
update tests to raise NotADirectoryError
1 parent 7571f11 commit c8ac2d8

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

Lib/test/test_os/test_os.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1863,14 +1863,10 @@ def test_walk_bad_dir2(self):
18631863
self.assertRaises(StopIteration, next, walk_it)
18641864

18651865
walk_it = self.walk(self.tmp1_path)
1866-
self.assertRaises(StopIteration, next, walk_it)
1866+
self.assertRaises(NotADirectoryError, next, walk_it)
18671867

18681868
walk_it = self.walk(self.tmp1_path, follow_symlinks=True)
1869-
if self.is_fwalk:
1870-
with self.assertRaises(OSError) as cm:
1871-
next(walk_it)
1872-
self.assertIn(cm.exception.errno, (errno.ENOTDIR, errno.EINVAL))
1873-
self.assertRaises(StopIteration, next, walk_it)
1869+
self.assertRaises(NotADirectoryError, next, walk_it)
18741870

18751871
@unittest.skipUnless(hasattr(os, "mkfifo"), 'requires os.mkfifo()')
18761872
@unittest.skipIf(sys.platform == "vxworks",
@@ -1881,12 +1877,10 @@ def test_walk_named_pipe(self):
18811877
self.addCleanup(os.unlink, path)
18821878

18831879
walk_it = self.walk(path)
1884-
self.assertRaises(StopIteration, next, walk_it)
1880+
self.assertRaises(NotADirectoryError, next, walk_it)
18851881

18861882
walk_it = self.walk(path, follow_symlinks=True)
1887-
if self.is_fwalk:
1888-
self.assertRaises(NotADirectoryError, next, walk_it)
1889-
self.assertRaises(StopIteration, next, walk_it)
1883+
self.assertRaises(NotADirectoryError, next, walk_it)
18901884

18911885
@unittest.skipUnless(hasattr(os, "mkfifo"), 'requires os.mkfifo()')
18921886
@unittest.skipIf(sys.platform == "vxworks",

0 commit comments

Comments
 (0)