1 files changed, 60 insertions, 8 deletions
diff --git a/tests/cp/preserve-link b/tests/cp/preserve-link
index d0da87356..e3c31f9e4 100755
--- a/tests/cp/preserve-link
+++ b/tests/cp/preserve-link
@@ -26,15 +26,67 @@ same_inode()
     v=$(stat --format %i "$2") && test "$u" = "$v"
 }
 
-mkdir -p s t/s || framework_failure_
-touch s/f t/s/f || framework_failure_
-ln s/f s/link || framework_failure_
+create_source_tree()
+{
+  rm -Rf s
+  mkdir s || framework_failure_
+
+  # a missing link in dest will be created
+  touch s/f || framework_failure_
+  ln s/f s/linkm || framework_failure_
+
+  # an existing link in dest will be maintained
+  ln s/f s/linke || framework_failure_
+
+  # a separate older file in dest will be overwritten
+  ln s/f s/fileo || framework_failure_
+
+  # a separate newer file in dest will be overwritten!
+  ln s/f s/fileu || framework_failure_
+}
+
+create_target_tree()
+{
+  f=$1 # which of f or linkm to create in t/
+
+  rm -Rf t
+  mkdir -p t/s/ || framework_failure_
+
+  # a missing link in dest must be created
+  touch t/s/$f || framework_failure_
+
+  # an existing link must be maintained
+  ln t/s/$f t/s/linke || framework_failure_
+
+  # a separate older file in dest will be overwritten
+  touch -d '-1 hour' t/s/fileo || framework_failure_
+
+  # a separate newer file in dest will be overwritten!
+  touch -d '+1 hour' t/s/fileu || framework_failure_
+}
+
+
+# Note we repeat this, creating either one of
+# two hard linked files from source in the dest, so as to
+# test both paths in `cp` for creating the hard links.
+# The path taken by cp is dependent on which cp encounters
+# first in the source, which is non deterministic currently
+# (I'm guessing that results are sorted by inode and
+# beauses they're the same here, and due to the sort
+# being unstable, either can be processed first).
+create_source_tree
+
+for f in f linkm; do
+  create_target_tree $f
 
-# This must create a hard link, t/s/link, to the existing file, t/s/f.
-# With cp from coreutils-8.12 and prior, it would mistakenly copy
-# the file rather than creating the link.
-cp -au s t || fail=1
+  # Copy all the hard links across.  With cp from coreutils-8.12
+  # and prior, it would sometimes mistakenly copy rather than link.
+  cp -au s t || fail=1
 
-same_inode t/s/f t/s/link || fail=1
+  same_inode t/s/f t/s/linkm || fail=1
+  same_inode t/s/f t/s/linke || fail=1
+  same_inode t/s/f t/s/fileo || fail=1
+  same_inode t/s/f t/s/fileu || fail=1
+done
 
 Exit $fail