X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/7466d0c1af1f7d9a0c1b4ae54e9f1bbc951f2711..3950ffc9481c25262f2db2b08a0f74664c433734:/services/crunch-run/copier_test.go

diff --git a/services/crunch-run/copier_test.go b/services/crunch-run/copier_test.go
index 41c2acdf90..a2b560889f 100644
--- a/services/crunch-run/copier_test.go
+++ b/services/crunch-run/copier_test.go
@@ -108,16 +108,35 @@ func (s *copierSuite) TestSecretInOutputDir(c *check.C) {
 }
 
 func (s *copierSuite) TestSymlinkToMountedCollection(c *check.C) {
+	// simulate mounted read-only collection
 	s.cp.mounts["/mnt"] = arvados.Mount{
 		Kind:             "collection",
 		PortableDataHash: arvadostest.FooPdh,
 	}
+
+	// simulate mounted writable collection
+	bindtmp, err := ioutil.TempDir("", "crunch-run.test.")
+	c.Assert(err, check.IsNil)
+	defer os.RemoveAll(bindtmp)
+	f, err := os.OpenFile(bindtmp+"/.arvados#collection", os.O_CREATE|os.O_WRONLY, 0644)
+	c.Assert(err, check.IsNil)
+	_, err = io.WriteString(f, `{"manifest_text":". 37b51d194a7513e45b56f6524f2d51f2+3 0:3:bar\n"}`)
+	c.Assert(err, check.IsNil)
+	c.Assert(f.Close(), check.IsNil)
+	s.cp.mounts["/mnt-w"] = arvados.Mount{
+		Kind:             "collection",
+		PortableDataHash: arvadostest.FooPdh,
+		Writable:         true,
+	}
+	s.cp.binds = append(s.cp.binds, bindtmp+":/mnt-w")
+
 	c.Assert(os.Symlink("../../mnt", s.cp.hostOutputDir+"/l_dir"), check.IsNil)
 	c.Assert(os.Symlink("/mnt/foo", s.cp.hostOutputDir+"/l_file"), check.IsNil)
+	c.Assert(os.Symlink("/mnt-w/bar", s.cp.hostOutputDir+"/l_file_w"), check.IsNil)
 
-	err := s.cp.walkMount("", s.cp.ctrOutputDir, 10, true)
+	err = s.cp.walkMount("", s.cp.ctrOutputDir, 10, true)
 	c.Check(err, check.IsNil)
-	c.Check(s.cp.manifest, check.Matches, `(?ms)\./l_dir acbd\S+ 0:3:foo\n\. acbd\S+ 0:3:l_file\n`)
+	c.Check(s.cp.manifest, check.Matches, `(?ms)\./l_dir acbd\S+ 0:3:foo\n\. acbd\S+ 0:3:l_file\n\. 37b5\S+ 0:3:l_file_w\n`)
 }
 
 func (s *copierSuite) TestSymlink(c *check.C) {
@@ -134,6 +153,8 @@ func (s *copierSuite) TestSymlink(c *check.C) {
 		os.MkdirAll(s.cp.hostOutputDir+"/morelinks", 0755),
 		os.Symlink("../dir1/dir2", s.cp.hostOutputDir+"/morelinks/l_rel_dir2"),
 		os.Symlink("dir1/dir2/dir3", s.cp.hostOutputDir+"/l_rel_dir3"),
+		// rel. symlink -> rel. symlink -> regular file
+		os.Symlink("../dir1/dir2/l_rel_file", s.cp.hostOutputDir+"/morelinks/l_rel_l_rel_file"),
 	} {
 		c.Assert(err, check.IsNil)
 	}
@@ -157,6 +178,7 @@ func (s *copierSuite) TestSymlink(c *check.C) {
 		{dst: "/l_rel_file", src: hostfile, size: 4},
 		{dst: "/morelinks/l_rel_dir2/dir3/.keep", src: os.DevNull},
 		{dst: "/morelinks/l_rel_dir2/l_rel_file", src: hostfile, size: 4},
+		{dst: "/morelinks/l_rel_l_rel_file", src: hostfile, size: 4},
 	})
 }