Skip to content

bpo-24132: Add direct subclassing of PurePath/Path in pathlib #26906

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
kfollstad wants to merge 7 commits into from
Closed

bpo-24132: Add direct subclassing of PurePath/Path in pathlib #26906

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
710f577
bpo-24132: Add tests for class properties in pathlib's PurePath/Path
kfollstad Jun 11, 2021
a373969
bpo-24132: Add test for direct subclassing of PurePath/Path in pathlib
kfollstad Jun 14, 2021
35d8f78
bpo-24132: Fix inconsistent organization of is_mount in pathlib
kfollstad May 27, 2021
bab67b4
bpo-24132: Replace additional hardcoded references in pathlib
kfollstad May 3, 2021
67faae0
bpo-24132: Replace factory functionality with masquerade in pathlib
kfollstad Jun 14, 2021
610f41e
bpo-24132: Add tests for comparing subclasses of PurePath in pathlib
kfollstad Jun 17, 2021
a0c4998
bpo-24132: Document direct subclassing of PurePath/Path in pathlib
kfollstad Jun 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
bpo-24132: Add tests for comparing subclasses of PurePath in pathlib 
Add tests mirroring existing comparison tests for PurePosixPath
and PureWindowsPath but extending them to direct subclasses of PurePath.
  • Loading branch information
@kfollstad
kfollstad committed Jun 24, 2021
commit 610f41ed81e96b525b6ac230117527255e79b73d
29 changes: 26 additions & 3 deletions Lib/test/test_pathlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1335,6 +1335,10 @@ def _get_class_to_generate(self):
is_posix = os.name == 'posix'
return pathlib.PurePosixPath if is_posix else pathlib.PureWindowsPath

def _get_anti_system_flavour_class(self):
is_posix = os.name == 'posix'
return pathlib.PureWindowsPath if is_posix else pathlib.PurePosixPath

def test_instance_class_properties(self):
p = self.cls('placeholder')
correct_cls = self._get_class_to_generate()
Expand Down Expand Up @@ -1369,9 +1373,15 @@ def test_different_flavours_unequal(self):
q = pathlib.PureWindowsPath('a')
self.assertNotEqual(p, q)

def test_different_flavours_unordered(self):
p = pathlib.PurePosixPath('a')
q = pathlib.PureWindowsPath('a')
def test_subclass_different_flavours_unequal(self):
class Derived(pathlib.PurePath):
pass
p = Derived('a')
PureAntiFlavourPath = self._get_anti_system_flavour_class()
q = PureAntiFlavourPath('a')
self.assertNotEqual(p, q)

def _test_different_flavours_unordered(self, p, q):
with self.assertRaises(TypeError):
p < q
with self.assertRaises(TypeError):
Expand All @@ -1381,6 +1391,19 @@ def test_different_flavours_unordered(self):
with self.assertRaises(TypeError):
p >= q

def test_different_flavours_unordered(self):
p = pathlib.PurePosixPath('a')
q = pathlib.PureWindowsPath('a')
self._test_different_flavours_unordered(p, q)

def test_subclass_different_flavours_unordered(self):
class Derived(pathlib.PurePath):
pass
p = Derived('a')
PureAntiFlavourPath = self._get_anti_system_flavour_class()
q = PureAntiFlavourPath('a')
self._test_different_flavours_unordered(p, q)


#
# Tests for the concrete classes.
Expand Down