← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_groupcompress.py

Add bzrlib.pyutils, which has get_named_object, a wrapper around __import__.

This is used to replace various ad hoc implementations of the same logic,
notably the version used in registry's _LazyObjectGetter which had a bug when
getting a module without also getting a member.  And of course, this new
function has unit tests, unlike the replaced code.

This also adds a KnownHooksRegistry subclass to provide a more natural home for
some other logic.

I'm not thrilled about the name of the new module or the new functions, but it's
hard to think of good names for such generic functionality.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_groupcompress.py
1066
1066
        # consumption
1067
1067
        self.add_key_to_manager(('key4',), locations, block, manager)
1068
1068
        self.assertTrue(manager.check_is_well_utilized())
 
1069
 
 
1070
 
 
1071
class Test_GCBuildDetails(tests.TestCase):
 
1072
 
 
1073
    def test_acts_like_tuple(self):
 
1074
        # _GCBuildDetails inlines some of the data that used to be spread out
 
1075
        # across a bunch of tuples
 
1076
        bd = groupcompress._GCBuildDetails((('parent1',), ('parent2',)),
 
1077
            ('INDEX', 10, 20, 0, 5))
 
1078
        self.assertEqual(4, len(bd))
 
1079
        self.assertEqual(('INDEX', 10, 20, 0, 5), bd[0])
 
1080
        self.assertEqual(None, bd[1]) # Compression Parent is always None
 
1081
        self.assertEqual((('parent1',), ('parent2',)), bd[2])
 
1082
        self.assertEqual(('group', None), bd[3]) # Record details
 
1083
 
 
1084
    def test__repr__(self):
 
1085
        bd = groupcompress._GCBuildDetails((('parent1',), ('parent2',)),
 
1086
            ('INDEX', 10, 20, 0, 5))
 
1087
        self.assertEqual("_GCBuildDetails(('INDEX', 10, 20, 0, 5),"
 
1088
                         " (('parent1',), ('parent2',)))",
 
1089
                         repr(bd))
 
1090