[patch] exclude .bzrignore from export, this time with tests

[Jamie Wilkinson]

This one time, at band camp, John A Meinel wrote:
>Jamie Wilkinson wrote:
>> I noticed yesterday that the exporters were split apart, and in doing so
>> they no longer excluded '.bzrignore' from the export.
>> 
>> I've repatched it now, and added blackbox tests.
>> 
>> http://repo.spacepants.org/bzr.jaq/
>> 
>> revno 1524 and 1525
>> 
>> 
>
>The tests are appreciated, but need some cleanup before they can be merged.
>
>First, you have tab characters in your code. bzrlib code is expected to
>only have space characters for indentation. There have been some threads
>about how to configure vim so that it sets 'expandtab' based on the
>working directory. I'm not sure how to do emacs. But I can repost if you
>want.

Oops, I thought I converted them all.

Fixed and committed to 1526 in the same branch.

>Thanks for the tests here, but you forgot to test the 'dir exporter'.

If you've got an idea on how to do that, I'd love to hear it :)

>It hasn't been obvious, and may have only happened recently, but all of
>the tests are now sorted in alphabetical order. So this should go
>between test_diff and test_fetch.

Well, _missing and _too_much aren't in the right order, either :)

I've reordered them and committed to 1527 in the same branch.

Actually, I only read your first sentence.  test_diff and test_fetch don't
appear in the blackbox suite -- test_export isn't an internal test.