git/t/t7420-submodule-set-url.sh
Patrick Steinhardt 5bf922a4e9 builtin/submodule--helper: fix leaking remote ref on errors
When `update_submodule()` fails we return with `die_message()`, which
only causes us to print the same message as `die()` would without
actually causing the process to die. We don't free memory in that case
and thus leak memory.

Fix the leak by freeing the remote ref.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2024-09-27 08:25:35 -07:00

82 lines
1.8 KiB
Bash
Executable file

#!/bin/sh
#
# Copyright (c) 2019 Denton Liu
#
test_description='Test submodules set-url subcommand
This test verifies that the set-url subcommand of git-submodule is working
as expected.
'
TEST_NO_CREATE_REPO=1
TEST_PASSES_SANITIZE_LEAK=true
. ./test-lib.sh
test_expect_success 'setup' '
git config --global protocol.file.allow always
'
test_expect_success 'submodule config cache setup' '
mkdir submodule &&
(
cd submodule &&
git init &&
echo a >file &&
git add file &&
git commit -ma
) &&
mkdir namedsubmodule &&
(
cd namedsubmodule &&
git init &&
echo 1 >file &&
git add file &&
git commit -m1
) &&
mkdir super &&
(
cd super &&
git init &&
git submodule add ../submodule &&
git submodule add --name thename ../namedsubmodule thepath &&
git commit -m "add submodules"
)
'
test_expect_success 'test submodule set-url' '
# add commits and move the submodules (change the urls)
(
cd submodule &&
echo b >>file &&
git add file &&
git commit -mb
) &&
mv submodule newsubmodule &&
(
cd namedsubmodule &&
echo 2 >>file &&
git add file &&
git commit -m2
) &&
mv namedsubmodule newnamedsubmodule &&
git -C newsubmodule show >expect &&
git -C newnamedsubmodule show >>expect &&
(
cd super &&
test_must_fail git submodule update --remote &&
git submodule set-url submodule ../newsubmodule &&
test_cmp_config ../newsubmodule -f .gitmodules submodule.submodule.url &&
git submodule set-url thepath ../newnamedsubmodule &&
test_cmp_config ../newnamedsubmodule -f .gitmodules submodule.thename.url &&
test_cmp_config "" -f .gitmodules --default "" submodule.thepath.url &&
git submodule update --remote
) &&
git -C super/submodule show >actual &&
git -C super/thepath show >>actual &&
test_cmp expect actual
'
test_done