[brisbane:MERGE] GroupCHKStreamSource
John Arbash Meinel
john at arbash-meinel.com
Wed Apr 1 03:06:08 BST 2009
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Robert Collins wrote:
> bb:tweak
>
> In get_source, please upcall for the fallback; that prevents some
> duplication, and this isn't a high volume codepath.
>
Sure.
> This comment
> + # Can we get this from the search?
>
> in get_stream seems redundant. Please either expand this to something
> concrete or remove it :)
Sorry, I actually did end up getting it from the search and forgot to
remove the comment. I was using:
self.from_repo._get_revision_outside_set()
However, the recipe has 'exclude_keys', which is what I used instead. I
assume that is correct.
>
> For testing, we do test fetch in per_repository; I think a better and
> more relevant test is to test the source type that is constructed - add
> an expected source type to the parameterisation for per_repository, and
> assert they match in a per_repository.test_fetch fetch.
>
> -Rob
Is this 'per_repository' or something more of an 'interrepository' test?
I agree that a simple test in 'test_fetch' is probably not entirely
relevant, though it was certainly the easiest to get at (at the time, at
least).
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iEYEARECAAYFAknSzA8ACgkQJdeBCYSNAAPYqgCgyYTKm8Uq74vEaRrAPv1lnFjm
TPAAn2ntZoeG1cMGdRMiY6ZvjmwOF9dm
=Ujhk
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list