From adaf73b538b9704d7a4c45d132bc8e20127f498a Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 17:56:50 +0100
Subject: accomodate older bash-es

---
 links-chmod  | 3 ++-
 links-remove | 3 ++-
 links-update | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/links-chmod b/links-chmod
index f722b95..c0cf71e 100755
--- a/links-chmod
+++ b/links-chmod
@@ -8,7 +8,8 @@
 # usage: ./links-chmod [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run] MODE
 
 set -o errexit -o nounset -o pipefail
-shopt -s inherit_errexit lastpipe
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
 
 script_name="$( basename -- "${BASH_SOURCE[0]}" )"
 readonly script_name
diff --git a/links-remove b/links-remove
index c378485..2efc46d 100755
--- a/links-remove
+++ b/links-remove
@@ -8,7 +8,8 @@
 # usage: ./links-remove [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run]
 
 set -o errexit -o nounset -o pipefail
-shopt -s inherit_errexit lastpipe
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
 
 script_name="$( basename -- "${BASH_SOURCE[0]}" )"
 readonly script_name
diff --git a/links-update b/links-update
index 5b15f1b..322cd49 100755
--- a/links-update
+++ b/links-update
@@ -20,7 +20,8 @@
 # usage: ./links-update [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-m|--mode MODE] [-n|--dry-run]
 
 set -o errexit -o nounset -o pipefail
-shopt -s inherit_errexit lastpipe
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
 
 script_name="$( basename -- "${BASH_SOURCE[0]}" )"
 readonly script_name
-- 
cgit v1.2.3


From 7cca99fd7ad119c27665488e28ceb86800afd574 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 18:23:17 +0100
Subject: add Docker tests for older bash-es

---
 .github/workflows/ci.yml | 18 ++++++++++++++++--
 Makefile                 | 45 +++++++++++++++++++++++++++++++++++++++++++++
 test/docker-compose.yml  |  7 +++++++
 test/test.sh             |  3 ++-
 4 files changed, 70 insertions(+), 3 deletions(-)
 create mode 100644 Makefile
 create mode 100644 test/docker-compose.yml

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index f4954c4..2bfb964 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -6,7 +6,7 @@ on:
   workflow_dispatch:
 
 jobs:
-  test:
+  test_local:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest]
@@ -26,4 +26,18 @@ jobs:
         # with ancient utilities that come with the actual macOS.
 
       - name: Test
-        run: ./test/test.sh
+        run: make test
+
+  test_docker:
+    strategy:
+      matrix:
+        # Keep in sync with those in Makefile:
+        distro: [xenial, focal]
+    runs-on: ubuntu-latest
+    name: 'Test / Docker / ${{ matrix.distro }}'
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Test
+        run: make 'test/docker/${{ matrix.distro }}'
diff --git a/Makefile b/Makefile
new file mode 100644
index 0000000..cf78947
--- /dev/null
+++ b/Makefile
@@ -0,0 +1,45 @@
+MAKEFLAGS += --no-builtin-rules --no-builtin-variables --warn-undefined-variables
+unexport MAKEFLAGS
+.DEFAULT_GOAL := all
+.DELETE_ON_ERROR:
+.SUFFIXES:
+SHELL := bash
+.SHELLFLAGS := -eu -o pipefail -c
+
+escape = $(subst ','\'',$(1))
+
+define noexpand
+ifeq ($$(origin $(1)),environment)
+    $(1) := $$(value $(1))
+endif
+ifeq ($$(origin $(1)),environment override)
+    $(1) := $$(value $(1))
+endif
+ifeq ($$(origin $(1)),command line)
+    override $(1) := $$(value $(1))
+endif
+endef
+
+.PHONY: DO
+DO:
+
+.PHONY: all
+all: test
+
+.PHONY: test
+test: test/local
+
+.PHONY: test/all
+test/all: test/local test/docker
+
+.PHONY: test/local
+test/local:
+	./test/test.sh
+
+test/docker/%: DO
+	cd test && \
+		DISTRO='$*' docker-compose run --rm test && \
+		docker-compose down -v
+
+.PHONY: test/docker
+test/docker: test/docker/xenial test/docker/focal
diff --git a/test/docker-compose.yml b/test/docker-compose.yml
new file mode 100644
index 0000000..2d7f094
--- /dev/null
+++ b/test/docker-compose.yml
@@ -0,0 +1,7 @@
+version: '3'
+services:
+  test:
+    image: "ubuntu:${DISTRO:-xenial}"
+    volumes:
+      - ../:/src
+    command: /src/test/test.sh
diff --git a/test/test.sh b/test/test.sh
index 05c5d65..e28f112 100755
--- a/test/test.sh
+++ b/test/test.sh
@@ -1,7 +1,8 @@
 #!/usr/bin/env bash
 
 set -o errexit -o nounset -o pipefail
-shopt -s inherit_errexit lastpipe
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
 
 script_dir="$( dirname -- "${BASH_SOURCE[0]}" )"
 script_dir="$( cd -- "$script_dir" && pwd )"
-- 
cgit v1.2.3


From e2e618894ed86470c89ab78c9e032a809ce7c227 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 18:28:18 +0100
Subject: move unit tests to test/unit/

---
 Makefile                           |   2 +-
 test/dest/1.txt                    |   1 -
 test/dest/bar/3.txt                |   1 -
 test/docker-compose.yml            |   2 +-
 test/src/%DEST%/1.txt              |   1 -
 test/src/%DEST%/bar/3.txt          |   1 -
 test/src/%DEST%/bar/baz/4.txt      |   1 -
 test/src/%DEST%/foo/2.txt          |   1 -
 test/test.sh                       | 435 -------------------------------------
 test/unit/dest/1.txt               |   1 +
 test/unit/dest/bar/3.txt           |   1 +
 test/unit/src/%DEST%/1.txt         |   1 +
 test/unit/src/%DEST%/bar/3.txt     |   1 +
 test/unit/src/%DEST%/bar/baz/4.txt |   1 +
 test/unit/src/%DEST%/foo/2.txt     |   1 +
 test/unit/test.sh                  | 435 +++++++++++++++++++++++++++++++++++++
 16 files changed, 443 insertions(+), 443 deletions(-)
 delete mode 100644 test/dest/1.txt
 delete mode 100644 test/dest/bar/3.txt
 delete mode 100644 test/src/%DEST%/1.txt
 delete mode 100644 test/src/%DEST%/bar/3.txt
 delete mode 100644 test/src/%DEST%/bar/baz/4.txt
 delete mode 100644 test/src/%DEST%/foo/2.txt
 delete mode 100755 test/test.sh
 create mode 100644 test/unit/dest/1.txt
 create mode 100644 test/unit/dest/bar/3.txt
 create mode 100644 test/unit/src/%DEST%/1.txt
 create mode 100644 test/unit/src/%DEST%/bar/3.txt
 create mode 100644 test/unit/src/%DEST%/bar/baz/4.txt
 create mode 100644 test/unit/src/%DEST%/foo/2.txt
 create mode 100755 test/unit/test.sh

diff --git a/Makefile b/Makefile
index cf78947..ddc4349 100644
--- a/Makefile
+++ b/Makefile
@@ -34,7 +34,7 @@ test/all: test/local test/docker
 
 .PHONY: test/local
 test/local:
-	./test/test.sh
+	./test/unit/test.sh
 
 test/docker/%: DO
 	cd test && \
diff --git a/test/dest/1.txt b/test/dest/1.txt
deleted file mode 100644
index 3a2e3f4..0000000
--- a/test/dest/1.txt
+++ /dev/null
@@ -1 +0,0 @@
--1
diff --git a/test/dest/bar/3.txt b/test/dest/bar/3.txt
deleted file mode 100644
index a83d1d5..0000000
--- a/test/dest/bar/3.txt
+++ /dev/null
@@ -1 +0,0 @@
--3
diff --git a/test/docker-compose.yml b/test/docker-compose.yml
index 2d7f094..1a4374a 100644
--- a/test/docker-compose.yml
+++ b/test/docker-compose.yml
@@ -4,4 +4,4 @@ services:
     image: "ubuntu:${DISTRO:-xenial}"
     volumes:
       - ../:/src
-    command: /src/test/test.sh
+    command: /src/test/unit/test.sh
diff --git a/test/src/%DEST%/1.txt b/test/src/%DEST%/1.txt
deleted file mode 100644
index d00491f..0000000
--- a/test/src/%DEST%/1.txt
+++ /dev/null
@@ -1 +0,0 @@
-1
diff --git a/test/src/%DEST%/bar/3.txt b/test/src/%DEST%/bar/3.txt
deleted file mode 100644
index 00750ed..0000000
--- a/test/src/%DEST%/bar/3.txt
+++ /dev/null
@@ -1 +0,0 @@
-3
diff --git a/test/src/%DEST%/bar/baz/4.txt b/test/src/%DEST%/bar/baz/4.txt
deleted file mode 100644
index b8626c4..0000000
--- a/test/src/%DEST%/bar/baz/4.txt
+++ /dev/null
@@ -1 +0,0 @@
-4
diff --git a/test/src/%DEST%/foo/2.txt b/test/src/%DEST%/foo/2.txt
deleted file mode 100644
index 0cfbf08..0000000
--- a/test/src/%DEST%/foo/2.txt
+++ /dev/null
@@ -1 +0,0 @@
-2
diff --git a/test/test.sh b/test/test.sh
deleted file mode 100755
index e28f112..0000000
--- a/test/test.sh
+++ /dev/null
@@ -1,435 +0,0 @@
-#!/usr/bin/env bash
-
-set -o errexit -o nounset -o pipefail
-shopt -s inherit_errexit 2> /dev/null || true
-shopt -s lastpipe
-
-script_dir="$( dirname -- "${BASH_SOURCE[0]}" )"
-script_dir="$( cd -- "$script_dir" && pwd )"
-readonly script_dir
-script_name="$( basename -- "${BASH_SOURCE[0]}" )"
-readonly script_name
-
-readonly src_dir_name='src'
-readonly dest_dir_name='dest'
-readonly alt_dest_dir_name='alt_dest'
-
-src_dir_path="$script_dir/$src_dir_name"
-readonly src_dir_path
-dest_dir_path="$script_dir/$dest_dir_name"
-readonly dest_dir_path
-
-test_root_dir=
-test_src_dir=
-test_dest_dir=
-test_alt_dest_dir=
-
-new_test() {
-    local test_name=
-    [ "${#FUNCNAME[@]}" -gt 1 ] && test_name="${FUNCNAME[1]}"
-    [ "$#" -gt 0 ] && test_name="$1"
-
-    echo
-    echo ======================================================================
-    echo "New test: $test_name"
-
-    test_root_dir="$( mktemp -d )"
-    # mktemp returns /var/..., which is actually in /private/var/... on macOS.
-    test_root_dir="$( readlink -e -- "$test_root_dir" )"
-    test_src_dir="$test_root_dir/$src_dir_name"
-    test_dest_dir="$test_root_dir/$dest_dir_name"
-    test_alt_dest_dir="$test_root_dir/$alt_dest_dir_name"
-
-    echo "Root directory: $test_root_dir"
-    echo "Shared directory: $test_src_dir"
-    echo "%DEST% directory: $test_dest_dir"
-    echo "%ALT_DEST% directory: $test_alt_dest_dir"
-    echo ======================================================================
-
-    cp -r -- "$src_dir_path" "$test_src_dir"
-    cp -r -- "$dest_dir_path" "$test_dest_dir"
-    cp -r -- "$dest_dir_path" "$test_alt_dest_dir"
-}
-
-call_bin_script() {
-    echo
-    echo -n 'Executing script:'
-
-    printf -- ' %q' "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
-    printf -- '\n'
-
-    echo
-    DEST="$test_dest_dir" ALT_DEST="$test_alt_dest_dir" "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
-}
-
-call_update() {
-    call_bin_script "$script_dir/../links-update" "$@"
-}
-
-call_remove() {
-    call_bin_script "$script_dir/../links-remove"
-}
-
-call_chmod() {
-    call_bin_script "$script_dir/../links-chmod" "$@"
-}
-
-verify_output() {
-    if [ "$#" -lt 1 ] || [ "$#" -gt 2 ]; then
-        echo "usage: ${FUNCNAME[0]} EXPECTED_OUTPUT [DEST_DIR]" >&2
-        return 1
-    fi
-
-    local expected_output="$1"
-    echo
-    echo 'Expected directory structure:'
-    echo "$expected_output"
-
-    local dest_dir="$test_dest_dir"
-    [ "$#" -ge 2 ] && dest_dir="$2"
-
-    local actual_output
-    actual_output="$( find "$dest_dir" -printf '%h/%f->%l\n' | sort )"
-    echo
-    echo 'Actual directory structure:'
-    echo "$actual_output"
-    echo
-
-    if [ "$actual_output" = "$expected_output" ]; then
-        echo "... They match!"
-    else
-        echo "... The actual directory structure does not match the expected directory structure!" >&2
-        return 1
-    fi
-}
-
-verify_mode() {
-    if [ "$#" -ne 2 ]; then
-        echo "usage: ${FUNCNAME[0]} EXPECTED_MODE FILE" >&2
-        return 1
-    fi
-
-    local expected_mode="$1"
-    local path="$2"
-
-    echo
-    echo "Checking permissions for file: $path"
-    echo "Expected mode: $expected_mode"
-
-    local actual_mode
-    actual_mode="$( stat -c '%a' -- "$path" )"
-    actual_mode="0$actual_mode"
-
-    echo "Actual mode: $actual_mode"
-
-    if [ "$actual_mode" = "$expected_mode" ]; then
-        echo "... They match!"
-    else
-        echo "... They don't match."
-        return 1
-    fi
-}
-
-test_update_works() {
-    # Basic test to make sure that links-update actually creates the proper
-    # symlinks.
-
-    new_test
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-
-    verify_output "$expected_output"
-}
-
-test_remove_works() {
-    # Basic test to make sure links-remove actually removes the created
-    # symlinks.
-
-    new_test
-    call_update
-    call_remove
-
-    local expected_output="$test_dest_dir->"
-    verify_output "$expected_output"
-}
-
-test_remove_does_not_overwrite_files() {
-    # Check that if a user overwrites a symlink with his own file, links-remove
-    # keeps it.
-
-    new_test
-    call_update
-
-    # Simulate a user overwriting one of the symlinks with his own file.
-    rm -- "$test_dest_dir/bar/3.txt"
-    echo 'User content' > "$test_dest_dir/bar/3.txt"
-
-    call_remove
-
-    # 3.txt must be kept:
-    local expected_output="$test_dest_dir->
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->"
-
-    verify_output "$expected_output"
-}
-
-test_remove_does_not_delete_used_dirs() {
-    # Check that if a user adds a file to a destination directory, it's not
-    # deleted by links-remove.
-
-    new_test
-    call_update
-
-    # User adds his own file to the directory:
-    echo 'User content' > "$test_dest_dir/bar/my-file"
-
-    call_remove
-
-    # bar/ and bar/my-file must be kept:
-    local expected_output="$test_dest_dir->
-$test_dest_dir/bar->
-$test_dest_dir/bar/my-file->"
-
-    verify_output "$expected_output"
-}
-
-new_test_dir_symlink() {
-    new_test "${FUNCNAME[1]}"
-
-    # Files will get symlinks in the directory pointed to by $DEST, as well as
-    # by $ALT_DEST.
-    ln -s -- '%DEST%' "$test_src_dir/%ALT_DEST%"
-}
-
-test_dir_symlink_update_works() {
-    # We can symlink files to multiple directories by creating symlinks inside
-    # --shared-dir.
-
-    new_test_dir_symlink
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-
-    verify_output "$expected_output"
-
-    expected_output="$test_alt_dest_dir->
-$test_alt_dest_dir/1.txt->$test_src_dir/%ALT_DEST%/1.txt
-$test_alt_dest_dir/bar->
-$test_alt_dest_dir/bar/3.txt->$test_src_dir/%ALT_DEST%/bar/3.txt
-$test_alt_dest_dir/bar/baz->
-$test_alt_dest_dir/bar/baz/4.txt->$test_src_dir/%ALT_DEST%/bar/baz/4.txt
-$test_alt_dest_dir/foo->
-$test_alt_dest_dir/foo/2.txt->$test_src_dir/%ALT_DEST%/foo/2.txt"
-
-    verify_output "$expected_output" "$test_alt_dest_dir"
-}
-
-test_dir_symlink_remove_works() {
-    # Test that links-remove works for directory symlinks inside --shared-dir.
-
-    new_test_dir_symlink
-    call_update
-    call_remove
-
-    local expected_output="$test_dest_dir->"
-    verify_output "$expected_output"
-
-    expected_output="$test_alt_dest_dir->"
-    verify_output "$expected_output" "$test_alt_dest_dir"
-}
-
-test_dir_symlink_remove_shared_file() {
-    # If we remove a shared file, both of the symlinks should be removed.
-
-    new_test_dir_symlink
-    call_update
-
-    # Remove a random shared file:
-    rm -- "$test_src_dir/%DEST%/bar/3.txt"
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-
-    verify_output "$expected_output"
-
-    expected_output="$test_alt_dest_dir->
-$test_alt_dest_dir/1.txt->$test_src_dir/%ALT_DEST%/1.txt
-$test_alt_dest_dir/bar->
-$test_alt_dest_dir/bar/baz->
-$test_alt_dest_dir/bar/baz/4.txt->$test_src_dir/%ALT_DEST%/bar/baz/4.txt
-$test_alt_dest_dir/foo->
-$test_alt_dest_dir/foo/2.txt->$test_src_dir/%ALT_DEST%/foo/2.txt"
-
-    verify_output "$expected_output" "$test_alt_dest_dir"
-}
-
-test_dir_symlink_remove_dir_symlink() {
-    # If we remove a directory symlink in --shared-dir, all the symlinks
-    # accessible through this directory symlink should be removed.
-
-    new_test_dir_symlink
-    call_update
-
-    # Remove the directory symlink:
-    rm -- "$test_src_dir/%ALT_DEST%"
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-
-    verify_output "$expected_output"
-
-    expected_output="$test_alt_dest_dir->"
-    verify_output "$expected_output" "$test_alt_dest_dir"
-}
-
-new_test_symlink() {
-    new_test "${FUNCNAME[1]}"
-
-    # Create a stupid symlink.
-    ln -s -- 'bar/3.txt' "$test_src_dir/%DEST%/3_copy.txt"
-}
-
-test_symlink_update_works() {
-    # Shared files can also be symlinks, pointing to something else.
-
-    new_test_symlink
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/3_copy.txt->$test_src_dir/%DEST%/3_copy.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-
-    verify_output "$expected_output"
-
-    echo
-    echo 'Verifying 3_copy.txt (the symlink) is valid...'
-
-    local copy_target
-    copy_target="$( readlink -- "$test_dest_dir/3_copy.txt" )"
-    test "$copy_target" = "$test_src_dir/%DEST%/3_copy.txt"
-
-    local copy_content
-    copy_content="$( cat -- "$test_dest_dir/3_copy.txt" )"
-    test "$copy_content" = '3'
-}
-
-test_symlink_remove_works() {
-    # Verify that links-remove doesn't delete shared symlinks.
-
-    new_test_symlink
-    call_update
-    call_remove
-
-    local expected_output="$test_dest_dir->"
-    verify_output "$expected_output"
-
-    echo
-    echo 'Verifying 3_copy.txt (the shared file) is valid...'
-
-    local copy_target
-    copy_target="$( readlink -e -- "$test_src_dir/%DEST%/3_copy.txt" )"
-    test "$copy_target" = "$test_src_dir/%DEST%/bar/3.txt"
-
-    local copy_content
-    copy_content="$( cat -- "$test_src_dir/%DEST%/3_copy.txt" )"
-    test "$copy_content" = '3'
-}
-
-test_chmod_works() {
-    # Test that links-chmod works.
-
-    new_test
-    call_update
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-    verify_output "$expected_output"
-
-    echo
-    echo 'Verifying 1.txt (the shared file) permissions...'
-
-    verify_mode 0644 "$test_src_dir/%DEST%/1.txt"
-    call_chmod 0600
-    verify_mode 0600 "$test_src_dir/%DEST%/1.txt"
-}
-
-test_update_chmod() {
-    # Test that links-update --mode works.
-
-    new_test
-    local expected_mode='0622'
-    call_update --mode "$expected_mode"
-
-    local expected_output="$test_dest_dir->
-$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
-$test_dest_dir/bar->
-$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
-$test_dest_dir/bar/baz->
-$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
-$test_dest_dir/foo->
-$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
-    verify_output "$expected_output"
-
-    verify_mode "$expected_mode" "$test_src_dir/%DEST%/1.txt"
-}
-
-main() {
-    test_update_works
-    test_remove_works
-    test_remove_does_not_overwrite_files
-    test_remove_does_not_delete_used_dirs
-
-    test_dir_symlink_update_works
-    test_dir_symlink_remove_works
-    test_dir_symlink_remove_shared_file
-    test_dir_symlink_remove_dir_symlink
-
-    test_symlink_update_works
-    test_symlink_remove_works
-
-    test_chmod_works
-    test_update_chmod
-}
-
-main
diff --git a/test/unit/dest/1.txt b/test/unit/dest/1.txt
new file mode 100644
index 0000000..3a2e3f4
--- /dev/null
+++ b/test/unit/dest/1.txt
@@ -0,0 +1 @@
+-1
diff --git a/test/unit/dest/bar/3.txt b/test/unit/dest/bar/3.txt
new file mode 100644
index 0000000..a83d1d5
--- /dev/null
+++ b/test/unit/dest/bar/3.txt
@@ -0,0 +1 @@
+-3
diff --git a/test/unit/src/%DEST%/1.txt b/test/unit/src/%DEST%/1.txt
new file mode 100644
index 0000000..d00491f
--- /dev/null
+++ b/test/unit/src/%DEST%/1.txt
@@ -0,0 +1 @@
+1
diff --git a/test/unit/src/%DEST%/bar/3.txt b/test/unit/src/%DEST%/bar/3.txt
new file mode 100644
index 0000000..00750ed
--- /dev/null
+++ b/test/unit/src/%DEST%/bar/3.txt
@@ -0,0 +1 @@
+3
diff --git a/test/unit/src/%DEST%/bar/baz/4.txt b/test/unit/src/%DEST%/bar/baz/4.txt
new file mode 100644
index 0000000..b8626c4
--- /dev/null
+++ b/test/unit/src/%DEST%/bar/baz/4.txt
@@ -0,0 +1 @@
+4
diff --git a/test/unit/src/%DEST%/foo/2.txt b/test/unit/src/%DEST%/foo/2.txt
new file mode 100644
index 0000000..0cfbf08
--- /dev/null
+++ b/test/unit/src/%DEST%/foo/2.txt
@@ -0,0 +1 @@
+2
diff --git a/test/unit/test.sh b/test/unit/test.sh
new file mode 100755
index 0000000..e28f112
--- /dev/null
+++ b/test/unit/test.sh
@@ -0,0 +1,435 @@
+#!/usr/bin/env bash
+
+set -o errexit -o nounset -o pipefail
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
+
+script_dir="$( dirname -- "${BASH_SOURCE[0]}" )"
+script_dir="$( cd -- "$script_dir" && pwd )"
+readonly script_dir
+script_name="$( basename -- "${BASH_SOURCE[0]}" )"
+readonly script_name
+
+readonly src_dir_name='src'
+readonly dest_dir_name='dest'
+readonly alt_dest_dir_name='alt_dest'
+
+src_dir_path="$script_dir/$src_dir_name"
+readonly src_dir_path
+dest_dir_path="$script_dir/$dest_dir_name"
+readonly dest_dir_path
+
+test_root_dir=
+test_src_dir=
+test_dest_dir=
+test_alt_dest_dir=
+
+new_test() {
+    local test_name=
+    [ "${#FUNCNAME[@]}" -gt 1 ] && test_name="${FUNCNAME[1]}"
+    [ "$#" -gt 0 ] && test_name="$1"
+
+    echo
+    echo ======================================================================
+    echo "New test: $test_name"
+
+    test_root_dir="$( mktemp -d )"
+    # mktemp returns /var/..., which is actually in /private/var/... on macOS.
+    test_root_dir="$( readlink -e -- "$test_root_dir" )"
+    test_src_dir="$test_root_dir/$src_dir_name"
+    test_dest_dir="$test_root_dir/$dest_dir_name"
+    test_alt_dest_dir="$test_root_dir/$alt_dest_dir_name"
+
+    echo "Root directory: $test_root_dir"
+    echo "Shared directory: $test_src_dir"
+    echo "%DEST% directory: $test_dest_dir"
+    echo "%ALT_DEST% directory: $test_alt_dest_dir"
+    echo ======================================================================
+
+    cp -r -- "$src_dir_path" "$test_src_dir"
+    cp -r -- "$dest_dir_path" "$test_dest_dir"
+    cp -r -- "$dest_dir_path" "$test_alt_dest_dir"
+}
+
+call_bin_script() {
+    echo
+    echo -n 'Executing script:'
+
+    printf -- ' %q' "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
+    printf -- '\n'
+
+    echo
+    DEST="$test_dest_dir" ALT_DEST="$test_alt_dest_dir" "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
+}
+
+call_update() {
+    call_bin_script "$script_dir/../links-update" "$@"
+}
+
+call_remove() {
+    call_bin_script "$script_dir/../links-remove"
+}
+
+call_chmod() {
+    call_bin_script "$script_dir/../links-chmod" "$@"
+}
+
+verify_output() {
+    if [ "$#" -lt 1 ] || [ "$#" -gt 2 ]; then
+        echo "usage: ${FUNCNAME[0]} EXPECTED_OUTPUT [DEST_DIR]" >&2
+        return 1
+    fi
+
+    local expected_output="$1"
+    echo
+    echo 'Expected directory structure:'
+    echo "$expected_output"
+
+    local dest_dir="$test_dest_dir"
+    [ "$#" -ge 2 ] && dest_dir="$2"
+
+    local actual_output
+    actual_output="$( find "$dest_dir" -printf '%h/%f->%l\n' | sort )"
+    echo
+    echo 'Actual directory structure:'
+    echo "$actual_output"
+    echo
+
+    if [ "$actual_output" = "$expected_output" ]; then
+        echo "... They match!"
+    else
+        echo "... The actual directory structure does not match the expected directory structure!" >&2
+        return 1
+    fi
+}
+
+verify_mode() {
+    if [ "$#" -ne 2 ]; then
+        echo "usage: ${FUNCNAME[0]} EXPECTED_MODE FILE" >&2
+        return 1
+    fi
+
+    local expected_mode="$1"
+    local path="$2"
+
+    echo
+    echo "Checking permissions for file: $path"
+    echo "Expected mode: $expected_mode"
+
+    local actual_mode
+    actual_mode="$( stat -c '%a' -- "$path" )"
+    actual_mode="0$actual_mode"
+
+    echo "Actual mode: $actual_mode"
+
+    if [ "$actual_mode" = "$expected_mode" ]; then
+        echo "... They match!"
+    else
+        echo "... They don't match."
+        return 1
+    fi
+}
+
+test_update_works() {
+    # Basic test to make sure that links-update actually creates the proper
+    # symlinks.
+
+    new_test
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+
+    verify_output "$expected_output"
+}
+
+test_remove_works() {
+    # Basic test to make sure links-remove actually removes the created
+    # symlinks.
+
+    new_test
+    call_update
+    call_remove
+
+    local expected_output="$test_dest_dir->"
+    verify_output "$expected_output"
+}
+
+test_remove_does_not_overwrite_files() {
+    # Check that if a user overwrites a symlink with his own file, links-remove
+    # keeps it.
+
+    new_test
+    call_update
+
+    # Simulate a user overwriting one of the symlinks with his own file.
+    rm -- "$test_dest_dir/bar/3.txt"
+    echo 'User content' > "$test_dest_dir/bar/3.txt"
+
+    call_remove
+
+    # 3.txt must be kept:
+    local expected_output="$test_dest_dir->
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->"
+
+    verify_output "$expected_output"
+}
+
+test_remove_does_not_delete_used_dirs() {
+    # Check that if a user adds a file to a destination directory, it's not
+    # deleted by links-remove.
+
+    new_test
+    call_update
+
+    # User adds his own file to the directory:
+    echo 'User content' > "$test_dest_dir/bar/my-file"
+
+    call_remove
+
+    # bar/ and bar/my-file must be kept:
+    local expected_output="$test_dest_dir->
+$test_dest_dir/bar->
+$test_dest_dir/bar/my-file->"
+
+    verify_output "$expected_output"
+}
+
+new_test_dir_symlink() {
+    new_test "${FUNCNAME[1]}"
+
+    # Files will get symlinks in the directory pointed to by $DEST, as well as
+    # by $ALT_DEST.
+    ln -s -- '%DEST%' "$test_src_dir/%ALT_DEST%"
+}
+
+test_dir_symlink_update_works() {
+    # We can symlink files to multiple directories by creating symlinks inside
+    # --shared-dir.
+
+    new_test_dir_symlink
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+
+    verify_output "$expected_output"
+
+    expected_output="$test_alt_dest_dir->
+$test_alt_dest_dir/1.txt->$test_src_dir/%ALT_DEST%/1.txt
+$test_alt_dest_dir/bar->
+$test_alt_dest_dir/bar/3.txt->$test_src_dir/%ALT_DEST%/bar/3.txt
+$test_alt_dest_dir/bar/baz->
+$test_alt_dest_dir/bar/baz/4.txt->$test_src_dir/%ALT_DEST%/bar/baz/4.txt
+$test_alt_dest_dir/foo->
+$test_alt_dest_dir/foo/2.txt->$test_src_dir/%ALT_DEST%/foo/2.txt"
+
+    verify_output "$expected_output" "$test_alt_dest_dir"
+}
+
+test_dir_symlink_remove_works() {
+    # Test that links-remove works for directory symlinks inside --shared-dir.
+
+    new_test_dir_symlink
+    call_update
+    call_remove
+
+    local expected_output="$test_dest_dir->"
+    verify_output "$expected_output"
+
+    expected_output="$test_alt_dest_dir->"
+    verify_output "$expected_output" "$test_alt_dest_dir"
+}
+
+test_dir_symlink_remove_shared_file() {
+    # If we remove a shared file, both of the symlinks should be removed.
+
+    new_test_dir_symlink
+    call_update
+
+    # Remove a random shared file:
+    rm -- "$test_src_dir/%DEST%/bar/3.txt"
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+
+    verify_output "$expected_output"
+
+    expected_output="$test_alt_dest_dir->
+$test_alt_dest_dir/1.txt->$test_src_dir/%ALT_DEST%/1.txt
+$test_alt_dest_dir/bar->
+$test_alt_dest_dir/bar/baz->
+$test_alt_dest_dir/bar/baz/4.txt->$test_src_dir/%ALT_DEST%/bar/baz/4.txt
+$test_alt_dest_dir/foo->
+$test_alt_dest_dir/foo/2.txt->$test_src_dir/%ALT_DEST%/foo/2.txt"
+
+    verify_output "$expected_output" "$test_alt_dest_dir"
+}
+
+test_dir_symlink_remove_dir_symlink() {
+    # If we remove a directory symlink in --shared-dir, all the symlinks
+    # accessible through this directory symlink should be removed.
+
+    new_test_dir_symlink
+    call_update
+
+    # Remove the directory symlink:
+    rm -- "$test_src_dir/%ALT_DEST%"
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+
+    verify_output "$expected_output"
+
+    expected_output="$test_alt_dest_dir->"
+    verify_output "$expected_output" "$test_alt_dest_dir"
+}
+
+new_test_symlink() {
+    new_test "${FUNCNAME[1]}"
+
+    # Create a stupid symlink.
+    ln -s -- 'bar/3.txt' "$test_src_dir/%DEST%/3_copy.txt"
+}
+
+test_symlink_update_works() {
+    # Shared files can also be symlinks, pointing to something else.
+
+    new_test_symlink
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/3_copy.txt->$test_src_dir/%DEST%/3_copy.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+
+    verify_output "$expected_output"
+
+    echo
+    echo 'Verifying 3_copy.txt (the symlink) is valid...'
+
+    local copy_target
+    copy_target="$( readlink -- "$test_dest_dir/3_copy.txt" )"
+    test "$copy_target" = "$test_src_dir/%DEST%/3_copy.txt"
+
+    local copy_content
+    copy_content="$( cat -- "$test_dest_dir/3_copy.txt" )"
+    test "$copy_content" = '3'
+}
+
+test_symlink_remove_works() {
+    # Verify that links-remove doesn't delete shared symlinks.
+
+    new_test_symlink
+    call_update
+    call_remove
+
+    local expected_output="$test_dest_dir->"
+    verify_output "$expected_output"
+
+    echo
+    echo 'Verifying 3_copy.txt (the shared file) is valid...'
+
+    local copy_target
+    copy_target="$( readlink -e -- "$test_src_dir/%DEST%/3_copy.txt" )"
+    test "$copy_target" = "$test_src_dir/%DEST%/bar/3.txt"
+
+    local copy_content
+    copy_content="$( cat -- "$test_src_dir/%DEST%/3_copy.txt" )"
+    test "$copy_content" = '3'
+}
+
+test_chmod_works() {
+    # Test that links-chmod works.
+
+    new_test
+    call_update
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+    verify_output "$expected_output"
+
+    echo
+    echo 'Verifying 1.txt (the shared file) permissions...'
+
+    verify_mode 0644 "$test_src_dir/%DEST%/1.txt"
+    call_chmod 0600
+    verify_mode 0600 "$test_src_dir/%DEST%/1.txt"
+}
+
+test_update_chmod() {
+    # Test that links-update --mode works.
+
+    new_test
+    local expected_mode='0622'
+    call_update --mode "$expected_mode"
+
+    local expected_output="$test_dest_dir->
+$test_dest_dir/1.txt->$test_src_dir/%DEST%/1.txt
+$test_dest_dir/bar->
+$test_dest_dir/bar/3.txt->$test_src_dir/%DEST%/bar/3.txt
+$test_dest_dir/bar/baz->
+$test_dest_dir/bar/baz/4.txt->$test_src_dir/%DEST%/bar/baz/4.txt
+$test_dest_dir/foo->
+$test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
+    verify_output "$expected_output"
+
+    verify_mode "$expected_mode" "$test_src_dir/%DEST%/1.txt"
+}
+
+main() {
+    test_update_works
+    test_remove_works
+    test_remove_does_not_overwrite_files
+    test_remove_does_not_delete_used_dirs
+
+    test_dir_symlink_update_works
+    test_dir_symlink_remove_works
+    test_dir_symlink_remove_shared_file
+    test_dir_symlink_remove_dir_symlink
+
+    test_symlink_update_works
+    test_symlink_remove_works
+
+    test_chmod_works
+    test_update_chmod
+}
+
+main
-- 
cgit v1.2.3


From 6e595f7cc304af5cef6c33d70928e4948f6d9bcb Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 18:35:13 +0100
Subject: test/unit: fix path to links-* tools

---
 test/unit/test.sh | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/test/unit/test.sh b/test/unit/test.sh
index e28f112..c7645ea 100755
--- a/test/unit/test.sh
+++ b/test/unit/test.sh
@@ -10,6 +10,9 @@ readonly script_dir
 script_name="$( basename -- "${BASH_SOURCE[0]}" )"
 readonly script_name
 
+root_dir="$( git -C "$script_dir" rev-parse --show-toplevel )"
+readonly root_dir
+
 readonly src_dir_name='src'
 readonly dest_dir_name='dest'
 readonly alt_dest_dir_name='alt_dest'
@@ -63,15 +66,15 @@ call_bin_script() {
 }
 
 call_update() {
-    call_bin_script "$script_dir/../links-update" "$@"
+    call_bin_script "$root_dir/links-update" "$@"
 }
 
 call_remove() {
-    call_bin_script "$script_dir/../links-remove"
+    call_bin_script "$root_dir/links-remove"
 }
 
 call_chmod() {
-    call_bin_script "$script_dir/../links-chmod" "$@"
+    call_bin_script "$root_dir/links-chmod" "$@"
 }
 
 verify_output() {
-- 
cgit v1.2.3


From ba19f5478917cd8b92c81f22e425a7407f98e995 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 18:46:31 +0100
Subject: fix Docker tests

---
 test/docker-compose.yml |  5 ++++-
 test/docker/Dockerfile  |  9 +++++++++
 test/unit/test.sh       | 11 +++++++++++
 3 files changed, 24 insertions(+), 1 deletion(-)
 create mode 100644 test/docker/Dockerfile

diff --git a/test/docker-compose.yml b/test/docker-compose.yml
index 1a4374a..ec55b25 100644
--- a/test/docker-compose.yml
+++ b/test/docker-compose.yml
@@ -1,7 +1,10 @@
 version: '3'
 services:
   test:
-    image: "ubuntu:${DISTRO:-xenial}"
+    build:
+      context: docker/
+      args:
+        DISTRO: "${DISTRO:-xenial}"
     volumes:
       - ../:/src
     command: /src/test/unit/test.sh
diff --git a/test/docker/Dockerfile b/test/docker/Dockerfile
new file mode 100644
index 0000000..b67e6f0
--- /dev/null
+++ b/test/docker/Dockerfile
@@ -0,0 +1,9 @@
+ARG DISTRO=xenial
+
+FROM ubuntu:$DISTRO
+
+RUN apt-get update && \
+    DEBIAN_FRONTEND=noninteractive \
+        apt-get install -y --no-install-recommends git
+
+RUN git config --global --add safe.directory /src
diff --git a/test/unit/test.sh b/test/unit/test.sh
index c7645ea..f693aa1 100755
--- a/test/unit/test.sh
+++ b/test/unit/test.sh
@@ -417,7 +417,18 @@ $test_dest_dir/foo/2.txt->$test_src_dir/%DEST%/foo/2.txt"
     verify_mode "$expected_mode" "$test_src_dir/%DEST%/1.txt"
 }
 
+show_env() {
+    echo
+    echo ======================================================================
+    echo Environment
+    echo ======================================================================
+
+    bash --version
+}
+
 main() {
+    show_env
+
     test_update_works
     test_remove_works
     test_remove_does_not_overwrite_files
-- 
cgit v1.2.3


From 1f1d4af82552e8a5af4d6305d38dfe0f65383f92 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 19:08:00 +0100
Subject: properly rebuild Docker images

Before, if you ran

    make test/docker/xenial
    make test/docker/focal

It would still run tests against a Xenial image.
---
 Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index ddc4349..b5c8269 100644
--- a/Makefile
+++ b/Makefile
@@ -38,7 +38,8 @@ test/local:
 
 test/docker/%: DO
 	cd test && \
-		DISTRO='$*' docker-compose run --rm test && \
+		DISTRO='$*' docker-compose build --pull && \
+		docker-compose run --rm test && \
 		docker-compose down -v
 
 .PHONY: test/docker
-- 
cgit v1.2.3


From d5b36cc5f268d2cf53554268b345d28b9aa4aa11 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 19:20:31 +0100
Subject: README: update

---
 README.md | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/README.md b/README.md
index 4247989..8317081 100644
--- a/README.md
+++ b/README.md
@@ -53,16 +53,16 @@ To remove all symlinks, use `links-remove`.
 usage: links-remove [-h|--help] [-d|--database PATH] [-s|--shared-dir DIR] [-n|--dry-run]
 ```
 
-In this example, symlinks to files in "../src" must appear in "/test/dest".
+In this example, symlinks to files in "/test/src" must appear in "/test/dest".
 
 ```
-> tree /test/dest/
+$ tree /test/dest/
 /test/dest/
 
 0 directories, 0 files
 
-> tree ../src/
-../src/
+$ tree /test/src/
+/test/src/
 └── %DEST%
     ├── a
     │   └── b
@@ -74,21 +74,21 @@ In this example, symlinks to files in "../src" must appear in "/test/dest".
 
 6 directories, 2 files
 
-> echo "$DEST"
+$ echo "$DEST"
 /test/dest
 
-> ./links-update --shared-dir ../src/
+$ ./links-update --shared-dir /test/src/
 ...
 
-> tree /test/dest/
+$ tree /test/dest/
 /test/dest/
 ├── a
 │   └── b
 │       └── c
-│           └── test.txt -> /cygdrive/d/workspace/personal/src/%DEST%/a/b/c/test.txt
+│           └── test.txt -> /test/src/%DEST%/a/b/c/test.txt
 └── foo
     └── bar
-        └── baz -> /cygdrive/d/workspace/personal/src/%DEST%/foo/bar/baz
+        └── baz -> /test/src/%DEST%/foo/bar/baz
 
 5 directories, 2 files
 ```
-- 
cgit v1.2.3


From 68cdc9f03044f7664e7bbe584f1f3acb9c547878 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 19:23:33 +0100
Subject: README: update

---
 README.md | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/README.md b/README.md
index 8317081..9c04c05 100644
--- a/README.md
+++ b/README.md
@@ -101,6 +101,21 @@ For my personal real-life usage examples, see
 [my dotfiles]: https://github.com/egor-tensin/linux-home
 [Windows apps]: https://github.com/egor-tensin/windows-home
 
+Development
+-----------
+
+Run local tests using
+
+    make test
+
+Run Docker tests using
+
+    make test/docker
+
+Run all tests using
+
+    make test/all
+
 Limitations
 -----------
 
-- 
cgit v1.2.3


From 4f7d1526ad46a5f02a9e782d80ace5b7ea09c83b Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Thu, 2 Feb 2023 19:25:55 +0100
Subject: Makefile: add a comment

---
 Makefile | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Makefile b/Makefile
index b5c8269..054e75b 100644
--- a/Makefile
+++ b/Makefile
@@ -42,5 +42,9 @@ test/docker/%: DO
 		docker-compose run --rm test && \
 		docker-compose down -v
 
+# Xenial has bash 4.3, which doesn't support inherit_errexit, which is a good
+# thing to test against.
+#
+# Keep the list repositories synced with the GitHub actions workflow.
 .PHONY: test/docker
 test/docker: test/docker/xenial test/docker/focal
-- 
cgit v1.2.3


From 8f6582160a6dce81d6c6bc080d05aaa0b85d9537 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Sat, 4 Feb 2023 12:08:22 +0100
Subject: add an "integration" test

I noticed that doing links-update for my linux-home repository didn't
work on an old bash version. Building a test case to reproduce...

You should be able to check out this commit later and run

    make test/docker/xenial

and it's supposed to fail with something like

    /src/src/db.sh: line 194: wait: pid 1771 is not a child of this shell
---
 test/docker-compose.yml  |  5 ++-
 test/docker/Dockerfile   |  2 +-
 test/integration/test.sh | 89 ++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 94 insertions(+), 2 deletions(-)
 create mode 100755 test/integration/test.sh

diff --git a/test/docker-compose.yml b/test/docker-compose.yml
index ec55b25..76546c4 100644
--- a/test/docker-compose.yml
+++ b/test/docker-compose.yml
@@ -7,4 +7,7 @@ services:
         DISTRO: "${DISTRO:-xenial}"
     volumes:
       - ../:/src
-    command: /src/test/unit/test.sh
+    command:
+      - sh
+      - -c
+      - /src/test/unit/test.sh && /src/test/integration/test.sh
diff --git a/test/docker/Dockerfile b/test/docker/Dockerfile
index b67e6f0..5d4d016 100644
--- a/test/docker/Dockerfile
+++ b/test/docker/Dockerfile
@@ -4,6 +4,6 @@ FROM ubuntu:$DISTRO
 
 RUN apt-get update && \
     DEBIAN_FRONTEND=noninteractive \
-        apt-get install -y --no-install-recommends git
+        apt-get install -y --no-install-recommends ca-certificates git
 
 RUN git config --global --add safe.directory /src
diff --git a/test/integration/test.sh b/test/integration/test.sh
new file mode 100755
index 0000000..3bb481f
--- /dev/null
+++ b/test/integration/test.sh
@@ -0,0 +1,89 @@
+#!/usr/bin/env bash
+
+set -o errexit -o nounset -o pipefail
+shopt -s inherit_errexit 2> /dev/null || true
+shopt -s lastpipe
+
+script_dir="$( dirname -- "${BASH_SOURCE[0]}" )"
+script_dir="$( cd -- "$script_dir" && pwd )"
+readonly script_dir
+script_name="$( basename -- "${BASH_SOURCE[0]}" )"
+readonly script_name
+
+root_dir="$( git -C "$script_dir" rev-parse --show-toplevel )"
+readonly root_dir
+
+src_name='linux-home'
+readonly src_name
+src_url="https://github.com/egor-tensin/$src_name.git"
+readonly src_url
+
+test_root_dir=
+test_src_dir=
+
+new_test() {
+    local test_name=
+    [ "${#FUNCNAME[@]}" -gt 1 ] && test_name="${FUNCNAME[1]}"
+    [ "$#" -gt 0 ] && test_name="$1"
+
+    echo
+    echo ======================================================================
+    echo "New test: $test_name"
+
+    test_root_dir="$( mktemp -d )"
+    # mktemp returns /var/..., which is actually in /private/var/... on macOS.
+    test_root_dir="$( readlink -e -- "$test_root_dir" )"
+    test_src_dir="$test_root_dir/$src_name"
+
+    echo "Root directory: $test_root_dir"
+    echo "Shared directory: $test_src_dir"
+    echo ======================================================================
+
+    git clone -q -- "$src_url" "$test_src_dir"
+    cd -- "$test_src_dir"
+}
+
+call_bin_script() {
+    echo
+    echo -n 'Executing script:'
+
+    printf -- ' %q' "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
+    printf -- '\n'
+
+    echo
+    "$@" --shared-dir "$test_src_dir" --database "$test_root_dir/links.bin"
+}
+
+call_update() {
+    call_bin_script "$root_dir/links-update" "$@"
+}
+
+call_remove() {
+    call_bin_script "$root_dir/links-remove"
+}
+
+call_chmod() {
+    call_bin_script "$root_dir/links-chmod" "$@"
+}
+
+test_my_dotfiles_work() {
+    new_test
+    call_update
+}
+
+show_env() {
+    echo
+    echo ======================================================================
+    echo Environment
+    echo ======================================================================
+
+    bash --version
+}
+
+main() {
+    show_env
+
+    test_my_dotfiles_work
+}
+
+main
-- 
cgit v1.2.3


From 46349b5e772b40bb0dda614ad4317200408f2e67 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Sat, 4 Feb 2023 12:27:50 +0100
Subject: db.sh: fix Xenial tests

Please see the parent commit for the description of the problem.
---
 src/db.sh | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/db.sh b/src/db.sh
index 201ad25..94a4a7d 100644
--- a/src/db.sh
+++ b/src/db.sh
@@ -192,6 +192,7 @@ shared_file_present() {
 }
 
 link_all_entries() {
+    local -a shared_var_dirs=()
     local shared_var_dir
 
     find "$shared_root_dir" \
@@ -201,6 +202,10 @@ link_all_entries() {
         -regex ".*/$var_name_regex\$" \
         -printf '%P\0' |
     while IFS= read -d '' -r shared_var_dir; do
+        shared_var_dirs+=("$shared_var_dir")
+    done
+
+    for shared_var_dir in ${shared_var_dirs[@]+"${shared_var_dirs[@]}"}; do
         dump "shared directory: $shared_root_dir$shared_var_dir"
         local shared_path
 
-- 
cgit v1.2.3


From cff4f3426970192e18f63395e10289e5f13f4e48 Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Sat, 4 Feb 2023 12:29:44 +0100
Subject: t/i: call links-update twice, just in case

---
 test/integration/test.sh | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/integration/test.sh b/test/integration/test.sh
index 3bb481f..f4b1028 100755
--- a/test/integration/test.sh
+++ b/test/integration/test.sh
@@ -69,6 +69,8 @@ call_chmod() {
 test_my_dotfiles_work() {
     new_test
     call_update
+    # Again:
+    call_update
 }
 
 show_env() {
-- 
cgit v1.2.3