On Thu, 7 Dec 2017 11:18:52 -0800 Brandon Williams <bmw...@google.com> wrote:
> Instead of requiring that every test first removes 'repo', maybe you > want to have each test do its own cleanup by adding in > 'test_when_finished' lines to do the removals? Just a thought. If "test_when_finished" is the style that we plan to use in the project, we can do that. But I think the "rm -rf" at the beginning of a test method is better than "test_when_finished", though. It makes the test independent (for example, the addition or removal of tests before such a test is less likely to affect that test), and makes it clear if (and how) the test does its own setup as opposed to requiring the setup from another test block.
