Rev 4681: Take spiv's suggestion and move the '/' test to another test case. in http://bazaar.launchpad.net/~jameinel/bzr/2.0.1-322807-branch-at-root

John Arbash Meinel john at arbash-meinel.com
Thu Oct 8 04:58:21 BST 2009


At http://bazaar.launchpad.net/~jameinel/bzr/2.0.1-322807-branch-at-root

------------------------------------------------------------
revno: 4681
revision-id: john at arbash-meinel.com-20091008035802-gfdaqxgrxym64pqs
parent: john at arbash-meinel.com-20091008035530-xa6kmr9toz56i62x
committer: John Arbash Meinel <john at arbash-meinel.com>
branch nick: 2.0.1-322807-branch-at-root
timestamp: Wed 2009-10-07 22:58:02 -0500
message:
  Take spiv's suggestion and move the '/' test to another test case.
-------------- next part --------------
=== modified file 'bzrlib/tests/test_osutils.py'
--- a/bzrlib/tests/test_osutils.py	2009-10-06 17:36:03 +0000
+++ b/bzrlib/tests/test_osutils.py	2009-10-08 03:58:02 +0000
@@ -481,12 +481,15 @@
         self.assertRelpath('MixedCaseParent/not_child', base,
                            'MixedCaseParent/not_child')
 
-    def test_at_root(self):
-        # This path probably does not exist
+    def test_at_slash(self):
         # We can't test this on Windows, because it has a 'MIN_ABS_PATHLENGTH'
         # check...
-        # self.assertRelpath('foo', '/', '/foo')
+        if osutils.MIN_ABS_PATHLENGTH > 1:
+            return TestSkipped('relpath requires %d chars'
+                               % osutils.MIN_ABS_PATHLENGTH)
+        self.assertRelpath('foo', '/', '/foo')
 
+    def test_at_root(self):
         # see bug #322807
         # The specific issue is that when at the root of a drive, 'abspath'
         # returns "C:/" or just "/". However, the code assumes that abspath



More information about the bazaar-commits mailing list