Merge lp:~lifeless/bzr/bug-413957 into lp:~bzr/bzr/trunk-old

[Robert Collins]

[Robert Collins (lifeless)]

This 'bug' was simply a test that wants to test Pack repo's packing
logic specifically assuming that the default format was pack-0.92.

-Rob

--

[Andrew Bennetts (spiv)]

review: Approve

[John A Meinel (jameinel)]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Robert Collins wrote:
> Robert Collins has proposed merging lp:~lifeless/bzr/bug-413957 into lp:bzr.
>
> Requested reviews:
> bzr-core (bzr-core)
>
> This 'bug' was simply a test that wants to test Pack repo's packing
> logic specifically assuming that the default format was pack-0.92.
>
> -Rob
>
>

I should mention that RepositoryFormat2a *also* sorts revisions in
reverse topological order. I didn't look at the details of the test
case, but my guess is that it might be failing because it is poking at
the details of the index. Certainly we had a similar test in
"test_pack_repository" that was updated to work with both CHK and
KnitPack repos.

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkqJXLUACgkQJdeBCYSNAANbagCgh9oozLWoqrLgDL4JgWFFLcFd
qdgAniCX8YDIlk90AOzPARYsf0xE8kgy
=RKs7
-----END PGP SIGNATURE-----

=== modified file 'bzrlib/tests/test_repository.py'
--- bzrlib/tests/test_repository.py	2009-08-14 04:11:42 +0000
+++ bzrlib/tests/test_repository.py	2009-08-17 20:36:00 +0000
@@ -1349,7 +1349,7 @@
     """Tests for the packs repository Packer class."""
 
     def test_pack_optimizes_pack_order(self):
-        builder = self.make_branch_builder('.')
+        builder = self.make_branch_builder('.', format="1.9")
         builder.start_series()
         builder.build_snapshot('A', None, [
             ('add', ('', 'root-id', 'directory', None)),