[BUG] latest bzr-integration has multiple test_diff functions

John A Meinel john at arbash-meinel.com
Tue Sep 20 06:01:08 BST 2005


I ran across this earlier, when I was trying to update a test, but in
blackbox.py there are 2 copies of "test_diff"

I don't remember what the other test was that was duplicated, but they
were not next to eachother as these were. Doing the following grep:

for f in *.py; do echo $f; grep "def test_.*" $f |  sort | uniq -d; done

I found that blackbox.py has 2 test_diff() functions, testlog.py has 2
test_cur_revno(), and testrevision.py has completely duplicated the
TestIntermediateRevisions class.

I'm not sure how we get these duplicates, other than a possibly borked
merge, like with the old bad base selection, causing the same code to
get merged in twice.

Anyway, I thought I would bring up the ones that I've found, so that
they can be removed.
When I came across it in one of my branches, it took me quite a while to
figure out why my new changes weren't being run (since I was editing the
first function, but the last function is the one that gets run).

John
=:->
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 253 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050920/8f164c83/attachment.pgp 


More information about the bazaar mailing list