Rev 2935: Ugly bug marking, but test suite passing for python2.6. Since 2.6 in http://code.launchpad.net/%7Ev-ladeuil/bzr/bzr.python26

Vincent Ladeuil v.ladeuil+lp at free.fr
Tue Nov 20 16:06:34 GMT 2007


At http://code.launchpad.net/%7Ev-ladeuil/bzr/bzr.python26

------------------------------------------------------------
revno: 2935
revision-id:v.ladeuil+lp at free.fr-20071120160612-hxv17ysoy0sct45g
parent: v.ladeuil+lp at free.fr-20071120141050-21w0pnh2ydu94vzh
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: bzr.python26
timestamp: Tue 2007-11-20 17:06:12 +0100
message:
  Ugly bug marking, but test suite passing for python2.6. Since 2.6
  is still developed investigating now seems a waste.
  
  * bzrlib/tests/tree_implementations/test_walkdirs.py:
  (TestWalkdirs.test_walkdir_root,
  TestWalkdirs.test_walkdir_subtree): Mark failing tests.
  
  * bzrlib/tests/tree_implementations/test_test_trees.py:
  (TestTreeShapes.test_tree_with_subdirs_and_all_content_types):
  Mark failing tests.
  
  * bzrlib/tests/tree_implementations/test_inv.py:
  (TestEntryDiffing.test_link_diff_changed,
  TestInventory.test_symlink_target, TestInventory.test_symlink):
  Mark failing tests.
  
  * bzrlib/errors.py:
  (BzrError.__str__, BzrError.__str__): Working around python2.5
  hiding'message' attribute is not needed anymore.
modified:
  bzrlib/errors.py               errors.py-20050309040759-20512168c4e14fbd
  bzrlib/tests/tree_implementations/test_inv.py test_inv.py-20070312023226-0cdvk5uwhutis9vg-1
  bzrlib/tests/tree_implementations/test_test_trees.py test_tree_trees.py-20060720091921-3nwi5h21lf06vf5p-1
  bzrlib/tests/tree_implementations/test_walkdirs.py test_walkdirs.py-20060729160421-gmjnkotqgxdh98ce-1
-------------- next part --------------
=== modified file 'bzrlib/errors.py'
--- a/bzrlib/errors.py	2007-11-20 14:10:50 +0000
+++ b/bzrlib/errors.py	2007-11-20 16:06:12 +0000
@@ -17,6 +17,8 @@
 """Exceptions for bzr, and reporting of them.
 """
 
+import sys
+
 
 from bzrlib import (
     osutils,
@@ -102,9 +104,6 @@
             fmt = self._get_format_string()
             if fmt:
                 d = dict(self.__dict__)
-                # special case: python2.5 puts the 'message' attribute in a
-                # slot, so it isn't seen in __dict__
-                d['message'] = getattr(self, 'message', 'no message')
                 s = fmt % d
                 # __str__() should always return a 'str' object
                 # never a 'unicode' object.

=== modified file 'bzrlib/tests/tree_implementations/test_inv.py'
--- a/bzrlib/tests/tree_implementations/test_inv.py	2007-11-01 09:52:45 +0000
+++ b/bzrlib/tests/tree_implementations/test_inv.py	2007-11-20 16:06:12 +0000
@@ -130,6 +130,19 @@
 
     def test_link_diff_changed(self):
         self.requireFeature(SymlinkFeature)
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import return_parameter
+        # Even more ugly: return_paramater is used for WorkingTree[2-4]
+        if (self.workingtree_to_test_tree is return_parameter
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         output = StringIO()
         self.link_1.diff(internal_diff, 
                           "/dev/null", self.tree_1, 
@@ -219,6 +232,18 @@
 
     def test_symlink_target(self):
         self.requireFeature(SymlinkFeature)
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import _dirstate_tree_from_workingtree
+        if (self.workingtree_to_test_tree is _dirstate_tree_from_workingtree
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         self._set_up()
         if isinstance(self.tree, MutableTree):
             raise TestSkipped(
@@ -228,6 +253,18 @@
 
     def test_symlink(self):
         self.requireFeature(SymlinkFeature)
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import _dirstate_tree_from_workingtree
+        if (self.workingtree_to_test_tree is _dirstate_tree_from_workingtree
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         self._set_up()
         entry = self.inv[self.inv.path2id('symlink')]
         self.assertEqual(entry.kind, 'symlink')

=== modified file 'bzrlib/tests/tree_implementations/test_test_trees.py'
--- a/bzrlib/tests/tree_implementations/test_test_trees.py	2007-11-01 09:52:45 +0000
+++ b/bzrlib/tests/tree_implementations/test_test_trees.py	2007-11-20 16:06:12 +0000
@@ -148,6 +148,18 @@
         # to have it simply stop having the single unicode file in it
         # when dealing with a non-unicode filesystem.
         self.requireFeature(SymlinkFeature)
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import _dirstate_tree_from_workingtree
+        if (self.workingtree_to_test_tree is _dirstate_tree_from_workingtree
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         tree = self.get_tree_with_subdirs_and_all_content_types()
         tree.lock_read()
         self.addCleanup(tree.unlock)

=== modified file 'bzrlib/tests/tree_implementations/test_walkdirs.py'
--- a/bzrlib/tests/tree_implementations/test_walkdirs.py	2007-04-14 11:55:51 +0000
+++ b/bzrlib/tests/tree_implementations/test_walkdirs.py	2007-11-20 16:06:12 +0000
@@ -60,6 +60,18 @@
                 ]
 
     def test_walkdir_root(self):
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import _dirstate_tree_from_workingtree
+        if (self.workingtree_to_test_tree is _dirstate_tree_from_workingtree
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         tree = self.get_tree_with_subdirs_and_all_supported_content_types(has_symlinks())
         tree.lock_read()
         expected_dirblocks = self.get_all_subdirs_expected(tree, has_symlinks())
@@ -80,6 +92,18 @@
         self.assertEqual(len(expected_dirblocks), len(result))
             
     def test_walkdir_subtree(self):
+
+        import sys
+        from bzrlib.tests import KnownFailure
+        from bzrlib.tests.tree_implementations import _dirstate_tree_from_workingtree
+        if (self.workingtree_to_test_tree is _dirstate_tree_from_workingtree
+            and sys.version_info >= (2, 6)):
+            # Furthermore the problem here is that it interacts badly with a
+            # '\0\n\0'.join(lines) where lines contains *one* unicode string
+            # where all the other strings are not unicode.
+            raise  KnownFailure("python-2.6 os.readlink returns unicode path"
+                                " if called with unicode path")
+
         tree = self.get_tree_with_subdirs_and_all_supported_content_types(has_symlinks())
         # test that its iterable by iterating
         result = []



More information about the bazaar-commits mailing list