[PATCH 2/2] Add regression test, morph branch doesn't remove existing branch dir

Richard Maw richard.maw at codethink.co.uk
Wed Oct 30 11:09:15 GMT 2013


On Wed, Oct 30, 2013 at 11:03:35AM +0000, Pedro Alvarez wrote:
<snip>
> +Running `morph branch` when the branch directory exists don't
> +remove the existing directory.
> +
> +    SCENARIO    morph branch don't remove an existing branch directory
> +    GIVEN       a workspace
> +    AND         a git server
> +    WHEN        creating system branch foo
> +    THEN        the system branch foo is checked out
> +    WHEN        morph attempts to check out system branch foo
> +    THEN        the system branch foo is checked out
> +    AND         morph failed

s/don't/doesn't/g

Interesting bug. We definitely need this test, so you've got a +1 from
me if you do the language tweak, but I'm wondering if it may be worth
adding it to the scenario that does the checkout initially, rather than
adding a new scenario.



More information about the baserock-dev mailing list