From: Eric Sunshine <sunshine@sunshineco.com>
Date: Mon, 12 Jan 2015 23:40:28 GMT
Subject: Re: [PATCHv12 10/10] t5543-atomic-push.sh: add basic tests for atomic pushes
Message-ID: <CAPig+cT4SvvO=KCuRNaEZ7Vn1AeBy19mMeKtC=XJ1Mqw+zYYzA@mail.gmail.com>
In-Reply-To: <1420687404-13997-11-git-send-email-sbeller@google.com>

On Wed, Jan 7, 2015 at 10:23 PM, Stefan Beller <sbeller@google.com> wrote:
> This adds tests for the atomic push option.
> The first four tests check if the atomic option works in
> good conditions and the last three patches check if the atomic

s/patches/tests/

> option prevents any change to be pushed if just one ref cannot
> be updated.

The commit message talks about 7 tests, but I count 8.

> Signed-off-by: Stefan Beller <sbeller@google.com>
> ---
> diff --git a/t/t5543-atomic-push.sh b/t/t5543-atomic-push.sh
> new file mode 100755
> index 0000000..3480b33
> --- /dev/null
> +++ b/t/t5543-atomic-push.sh
> @@ -0,0 +1,194 @@
> +test_expect_success 'atomic push is not advertised if configured' '
> +       mk_repo_pair &&
> +       (
> +               cd upstream

Broken &&-chain.

> +               git config receive.advertiseatomic 0
> +       ) &&
> +       (
> +               cd workbench &&
> +               test_commit one &&
> +               git push --mirror up &&
> +               test_commit two &&
> +               test_must_fail git push --atomic up master
> +       ) &&
> +       test_refs master HEAD@{1}
> +'
> +
> +test_done
> --
> 2.2.1.62.g3f15098