From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-3.7 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_PASS, SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by dcvr.yhbt.net (Postfix) with ESMTP id 1D5491F4B4 for ; Fri, 23 Oct 2020 14:00:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750242AbgJWOAP (ORCPT ); Fri, 23 Oct 2020 10:00:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53238 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750236AbgJWOAM (ORCPT ); Fri, 23 Oct 2020 10:00:12 -0400 Received: from mail-wr1-x443.google.com (mail-wr1-x443.google.com [IPv6:2a00:1450:4864:20::443]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6063CC0613CE for ; Fri, 23 Oct 2020 07:00:12 -0700 (PDT) Received: by mail-wr1-x443.google.com with SMTP id x7so1979455wrl.3 for ; Fri, 23 Oct 2020 07:00:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:in-reply-to:references:from:date:subject:fcc :content-transfer-encoding:mime-version:to:cc; bh=/EnYefy9FtBroImVDVpDBZju4wSiIiy3qgJ3y+deKaY=; b=hgNlqnIEs3nyj3AHDHEDUZdemRSAfHTcUiI6SlqiLNBGJY8sdd+IOi2Y3JduiJFpY3 9dOxOzNEhxqiksZGNSSK6RY6uC8danqpOLlecuh5i/z3YHwpoFdchbCMBLLdUBKhg1jK tK1SvQKJrFRf3ajzfJPCmb3GvU3xaCK1ES3eIQjxoBGhmenF+KWOUKeOV/5OWmFpVM8p hDspZhfGvSVRdPe+GgeirBEDsQBmDp8BLGQhVX+cvwIesnpNpMFHc+rjsAIgBeDNCh5H 4nkEyISK2XoHkFBcz9IwRvgAoojnhWDNVd5wlD8BwXB9dBk7EiHwx7uO+jIw1FyWQR1P EPjA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:in-reply-to:references:from:date :subject:fcc:content-transfer-encoding:mime-version:to:cc; bh=/EnYefy9FtBroImVDVpDBZju4wSiIiy3qgJ3y+deKaY=; b=gT/blyGhWoGRLLTsVYcMyWTsCvdyY32ujIxqNYw/aVfT/xO3OwqCKdSFHHPyweYizb UZx2rv1lKp68uR5yP8v/HHwh0gSqaM/4HxKHAxBFLlHA32tSHM3bbtXtIX3fbuuhe5Lc Uw3W+7I96XmYAE532N8zjf3nAYCDwhIyoZkCokaCk6L1gTo0aWSvfQ6rEbiDP16TWFJY yylzeyHPrBhCrb/MXCzbyiCgWFAJUp6CUxd179WKvjlYFkBsuNLrG59+7O5THiI2hNoJ C60v6OJSWLZyX4tAAItAIQeXqaSVcm5PG+sU/yRjsGE9t7hS/PRcrpb84Gg4x6VHV3hL 2WeQ== X-Gm-Message-State: AOAM533ON81jptsId+qVihJ77+XakrnouJ6ZlhiRpdAllC/kuYTnmNiY wpwfudi7ha0SnCOHiMsSnaslzzFPhro= X-Google-Smtp-Source: ABdhPJzbRAHSgmBL3MOSgTyL/X+75v1YLmdw3G0pZ58ZqLA6HsdjIO1MtzWufdDdTR+H2w4wiwiEeQ== X-Received: by 2002:adf:a405:: with SMTP id d5mr2759652wra.421.1603461610916; Fri, 23 Oct 2020 07:00:10 -0700 (PDT) Received: from [127.0.0.1] ([13.74.141.28]) by smtp.gmail.com with ESMTPSA id b15sm3577364wrm.65.2020.10.23.07.00.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Oct 2020 07:00:10 -0700 (PDT) Message-Id: <9c608c311e6f6c22eaff6934f2048ca5e563a6cc.1603461606.git.gitgitgadget@gmail.com> In-Reply-To: References: From: "Johannes Schindelin via GitGitGadget" Date: Fri, 23 Oct 2020 14:00:00 +0000 Subject: [PATCH v2 3/9] tests: start moving to a different default main branch name Fcc: Sent Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit MIME-Version: 1.0 To: git@vger.kernel.org Cc: Johannes Schindelin , Johannes Schindelin , Johannes Schindelin Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Johannes Schindelin To allow for an incremental conversion to a new default main branch name, let's introduce `GIT_TEST_DEFAULT_MAIN_BRANCH_NAME`. This environment variable can be set at the top of each converted test script, overriding the default main branch name to use when initializing new repositories (or cloning empty repositories). Note: the `GIT_TEST_DEFAULT_MAIN_BRANCH_NAME` is _not_ intended to be used manually; many tests require a specific main branch name and cannot simply work with another one. This `GIT_TEST_*` variable is meant purely for the transitional period while the entire test suite is converted to use `main` as the initial branch name by default. We also introduce the `PREPARE_FOR_MAIN_BRANCH` prereq that determines whether the default main branch name is `main`, and adjust a couple of test functions to use it. This prereq will be used to temporarily disable a couple test cases to allow for adjusting the test script incrementally. Once an entire test is adjusted, we will adjust the test so that it is run with `GIT_TEST_DEFAULT_MAIN_BRANCH_NAME=main`. Signed-off-by: Johannes Schindelin --- refs.c | 5 ++++- t/lib-submodule-update.sh | 2 +- t/t0001-init.sh | 13 ++++++++++--- t/t5606-clone-options.sh | 3 +++ t/test-lib.sh | 7 +++++++ 5 files changed, 25 insertions(+), 5 deletions(-) diff --git a/refs.c b/refs.c index fa01153151..392f0bbf68 100644 --- a/refs.c +++ b/refs.c @@ -567,8 +567,11 @@ char *repo_default_branch_name(struct repository *r) const char *config_key = "init.defaultbranch"; const char *config_display_key = "init.defaultBranch"; char *ret = NULL, *full_ref; + const char *env = getenv("GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME"); - if (repo_config_get_string(r, config_key, &ret) < 0) + if (env && *env) + ret = xstrdup(env); + else if (repo_config_get_string(r, config_key, &ret) < 0) die(_("could not retrieve `%s`"), config_display_key); if (!ret) diff --git a/t/lib-submodule-update.sh b/t/lib-submodule-update.sh index 87a759149f..bd3fa3c6da 100644 --- a/t/lib-submodule-update.sh +++ b/t/lib-submodule-update.sh @@ -144,7 +144,7 @@ create_lib_submodule_repo () { git checkout -b valid_sub1 && git revert HEAD && - git checkout master + git checkout "${GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME-master}" ) } diff --git a/t/t0001-init.sh b/t/t0001-init.sh index 2f7c3dcd0f..69a320489f 100755 --- a/t/t0001-init.sh +++ b/t/t0001-init.sh @@ -553,14 +553,21 @@ test_expect_success '--initial-branch' ' test_expect_success 'overridden default initial branch name (config)' ' test_config_global init.defaultBranch nmb && - git init initial-branch-config && + GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME= git init initial-branch-config && git -C initial-branch-config symbolic-ref HEAD >actual && grep nmb actual ' +test_expect_success 'overridden default main branch name (env)' ' + test_config_global init.defaultBranch nmb && + GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=env git init main-branch-env && + git -C main-branch-env symbolic-ref HEAD >actual && + grep env actual +' + test_expect_success 'invalid default branch name' ' - test_config_global init.defaultBranch "with space" && - test_must_fail git init initial-branch-invalid 2>err && + test_must_fail env GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME="with space" \ + git init initial-branch-invalid 2>err && test_i18ngrep "invalid branch name" err ' diff --git a/t/t5606-clone-options.sh b/t/t5606-clone-options.sh index e69427f881..856eebf222 100755 --- a/t/t5606-clone-options.sh +++ b/t/t5606-clone-options.sh @@ -37,6 +37,7 @@ test_expect_success 'redirected clone -v does show progress' ' test_expect_success 'chooses correct default initial branch name' ' git init --bare empty && + GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME= \ git -c init.defaultBranch=up clone empty whats-up && test refs/heads/up = $(git -C whats-up symbolic-ref HEAD) && test refs/heads/up = $(git -C whats-up config branch.up.merge) @@ -51,9 +52,11 @@ test_expect_success 'guesses initial branch name correctly' ' git -c init.defaultBranch=none init --bare no-head && git -C initial-branch push ../no-head guess abc && + GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME= \ git clone no-head is-it2 && test_must_fail git -C is-it2 symbolic-ref refs/remotes/origin/HEAD && git -C no-head update-ref --no-deref HEAD refs/heads/guess && + GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME= \ git -c init.defaultBranch=guess clone no-head is-it3 && test refs/remotes/origin/guess = \ $(git -C is-it3 symbolic-ref refs/remotes/origin/HEAD) diff --git a/t/test-lib.sh b/t/test-lib.sh index ef31f40037..6b35070d22 100644 --- a/t/test-lib.sh +++ b/t/test-lib.sh @@ -1702,3 +1702,10 @@ test_lazy_prereq SHA1 ' test_lazy_prereq REBASE_P ' test -z "$GIT_TEST_SKIP_REBASE_P" ' +# Special-purpose prereq for transitioning to a new default branch name: +# Some tests need more than just a mindless (case-preserving) s/master/main/g +# replacement. The non-trivial adjustments are guarded behind this +# prerequisite, acting kind of as a feature flag +test_lazy_prereq PREPARE_FOR_MAIN_BRANCH ' + test "$GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME" = main +' -- gitgitgadget