From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: [PATCH 3/3] patch-id-test: test new --stable and --unstable flags Date: Thu, 27 Mar 2014 11:25:22 +0200 Message-ID: <1395912239-29663-3-git-send-email-mst@redhat.com> References: <1395912239-29663-1-git-send-email-mst@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: jrnieder@gmail.com, peff@peff.net, gitster@pobox.com To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Thu Mar 27 10:25:45 2014 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1WT6ZD-00023e-Sq for gcvg-git-2@plane.gmane.org; Thu, 27 Mar 2014 10:25:44 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754059AbaC0JZN (ORCPT ); Thu, 27 Mar 2014 05:25:13 -0400 Received: from mx1.redhat.com ([209.132.183.28]:52147 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752043AbaC0JZE (ORCPT ); Thu, 27 Mar 2014 05:25:04 -0400 Received: from int-mx13.intmail.prod.int.phx2.redhat.com (int-mx13.intmail.prod.int.phx2.redhat.com [10.5.11.26]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s2R9Ox2K017264 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Thu, 27 Mar 2014 05:25:00 -0400 Received: from redhat.com (vpn1-6-224.ams2.redhat.com [10.36.6.224]) by int-mx13.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with SMTP id s2R9OvZ4001401; Thu, 27 Mar 2014 05:24:58 -0400 Content-Disposition: inline In-Reply-To: <1395912239-29663-1-git-send-email-mst@redhat.com> X-Mutt-Fcc: =sent X-Scanned-By: MIMEDefang 2.68 on 10.5.11.26 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Verify that patch ID is now stable against hunk reordering. Signed-off-by: Michael S. Tsirkin --- t/t4204-patch-id.sh | 68 +++++++++++++++++++++++++++++++++++++++++++++++++---- 1 file changed, 63 insertions(+), 5 deletions(-) diff --git a/t/t4204-patch-id.sh b/t/t4204-patch-id.sh index d2c930d..75f77ef 100755 --- a/t/t4204-patch-id.sh +++ b/t/t4204-patch-id.sh @@ -5,12 +5,27 @@ test_description='git patch-id' . ./test-lib.sh test_expect_success 'setup' ' - test_commit initial foo a && - test_commit first foo b && + test_commit initial-foo foo a && + test_commit initial-bar bar a && + echo b > foo && + echo b > bar && + git commit -a -m first && git checkout -b same HEAD^ && - test_commit same-msg foo b && + echo b > foo && + echo b > bar && + git commit -a -m same-msg && git checkout -b notsame HEAD^ && - test_commit notsame-msg foo c + echo c > foo && + echo c > bar && + git commit -a -m notsame-msg && + cat > bar-then-foo < foo-then-bar < patch-id_"$1" } +calc_patch_id_unstable () { + git patch-id --unstable | + sed "s# .*##" > patch-id_"$1" +} + +calc_patch_id_stable () { + git patch-id --stable | + sed "s# .*##" > patch-id_"$1" +} + + get_patch_id () { - git log -p -1 "$1" | git patch-id | + git log -p -1 "$1" -O bar-then-foo -- | git patch-id | + sed "s# .*##" > patch-id_"$1" +} + +get_patch_id_stable () { + git log -p -1 "$1" -O bar-then-foo | git patch-id --stable | + sed "s# .*##" > patch-id_"$1" +} + +get_patch_id_unstable () { + git log -p -1 "$1" -O bar-then-foo | git patch-id --unstable | sed "s# .*##" > patch-id_"$1" } + test_expect_success 'patch-id detects equality' ' get_patch_id master && get_patch_id same && @@ -56,6 +93,27 @@ test_expect_success 'whitespace is irrelevant in footer' ' test_cmp patch-id_master patch-id_same ' +test_expect_success 'file order is irrelevant by default' ' + get_patch_id master && + git checkout same && + git format-patch -1 --stdout -O foo-then-bar | calc_patch_id same && + test_cmp patch-id_master patch-id_same +' + +test_expect_success 'file order is irrelevant with --stable' ' + get_patch_id_stable master && + git checkout same && + git format-patch -1 --stdout -O foo-then-bar | calc_patch_id_stable same && + test_cmp patch-id_master patch-id_same +' + +test_expect_success 'file order is relevant with --unstable' ' + get_patch_id_unstable master && + git checkout same && + git format-patch -1 --stdout -O foo-then-bar | calc_patch_id_unstable notsame && + ! test_cmp patch-id_master patch-id_notsame +' + test_expect_success 'patch-id supports git-format-patch MIME output' ' get_patch_id master && git checkout same && -- MST