[MERGE] Convert tests.blackbox.test_d* to use internals where appropriate

Alexander Belchenko bialix at ukr.net
Mon Aug 13 23:16:08 BST 2007


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

Daniel Watkins пишет:
> Attached is a fix for tests.blackbox.test_diff (the only 'd' in need of
> attention).

bb:comment

 class TestDiff(DiffBase):

     def test_diff(self):
- -        self.make_example_branch()
- -        file('hello', 'wt').write('hello world!')
- -        self.run_bzr('commit -m fixing hello')
+        tree = super(TestDiff, self).make_example_branch()

^-- super() there

     def test_diff_across_rename(self):
         """The working tree path should always be considered for diffing"""
- -        self.make_example_branch()
+        tree = super(TestDiff, self).make_example_branch()

^-- and super() here

they both seems odd.

IIUC, these tests both use DiffBase.make_example_branch() but they present in
TestDiff and TestCheckoutDiff. It means that these two tests runs twice for the same
data. If it's correct, I'd like to see them extracted in separate TestDiff* class
and run them only once. I don't see any reason to waste selftest time for
duplicates.
- --
[µ]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGwNgozYr338mxwCURAsyXAJ4z2eb/6ZMMzJ3FjfxgGgDIQDUUTQCgi4O5
409YUWxzSekAbbXl4ygbZMI=
=ZJQS
-----END PGP SIGNATURE-----



More information about the bazaar mailing list