123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363 |
- package main
- import (
- "os"
- "path/filepath"
- "testing"
- "gotest.tools/v3/assert"
- )
- // Try all of the test cases from the archive package which implements the
- // internals of `docker cp` and ensure that the behavior matches when actually
- // copying to and from containers.
- // Basic assumptions about SRC and DST:
- // 1. SRC must exist.
- // 2. If SRC ends with a trailing separator, it must be a directory.
- // 3. DST parent directory must exist.
- // 4. If DST exists as a file, it must not end with a trailing separator.
- // Check that copying from a container to a local symlink copies to the symlink
- // target and does not overwrite the local symlink itself.
- // TODO: move to docker/cli and/or integration/container/copy_test.go
- func (s *DockerCLICpSuite) TestCpFromSymlinkDestination(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{addContent: true})
- tmpDir := getTestDir(c, "test-cp-from-err-dst-not-dir")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- // First, copy a file from the container to a symlink to a file. This
- // should overwrite the symlink target contents with the source contents.
- srcPath := containerCpPath(containerID, "/file2")
- dstPath := cpPath(tmpDir, "symlinkToFile1")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, symlinkTargetEquals(c, dstPath, "file1"), "The symlink should not have been modified")
- assert.NilError(c, fileContentEquals(c, cpPath(tmpDir, "file1"), "file2\n"), `The file should have the contents of "file2" now`)
- // Next, copy a file from the container to a symlink to a directory. This
- // should copy the file into the symlink target directory.
- dstPath = cpPath(tmpDir, "symlinkToDir1")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, symlinkTargetEquals(c, dstPath, "dir1"), "The symlink should not have been modified")
- assert.NilError(c, fileContentEquals(c, cpPath(tmpDir, "file2"), "file2\n"), `The file should have the contents of "file2" now`)
- // Next, copy a file from the container to a symlink to a file that does
- // not exist (a broken symlink). This should create the target file with
- // the contents of the source file.
- dstPath = cpPath(tmpDir, "brokenSymlinkToFileX")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, symlinkTargetEquals(c, dstPath, "fileX"), "The symlink should not have been modified")
- assert.NilError(c, fileContentEquals(c, cpPath(tmpDir, "fileX"), "file2\n"), `The file should have the contents of "file2" now`)
- // Next, copy a directory from the container to a symlink to a local
- // directory. This should copy the directory into the symlink target
- // directory and not modify the symlink.
- srcPath = containerCpPath(containerID, "/dir2")
- dstPath = cpPath(tmpDir, "symlinkToDir1")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, symlinkTargetEquals(c, dstPath, "dir1"), "The symlink should not have been modified")
- assert.NilError(c, fileContentEquals(c, cpPath(tmpDir, "dir1/dir2/file2-1"), "file2-1\n"), `The directory should now contain a copy of "dir2"`)
- // Next, copy a directory from the container to a symlink to a local
- // directory that does not exist (a broken symlink). This should create
- // the target as a directory with the contents of the source directory. It
- // should not modify the symlink.
- dstPath = cpPath(tmpDir, "brokenSymlinkToDirX")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, symlinkTargetEquals(c, dstPath, "dirX"), "The symlink should not have been modified")
- assert.NilError(c, fileContentEquals(c, cpPath(tmpDir, "dirX/file2-1"), "file2-1\n"), `The "dirX" directory should now be a copy of "dir2"`)
- }
- // Possibilities are reduced to the remaining 10 cases:
- //
- // case | srcIsDir | onlyDirContents | dstExists | dstIsDir | dstTrSep | action
- // ===================================================================================================
- // A | no | - | no | - | no | create file
- // B | no | - | no | - | yes | error
- // C | no | - | yes | no | - | overwrite file
- // D | no | - | yes | yes | - | create file in dst dir
- // E | yes | no | no | - | - | create dir, copy contents
- // F | yes | no | yes | no | - | error
- // G | yes | no | yes | yes | - | copy dir and contents
- // H | yes | yes | no | - | - | create dir, copy contents
- // I | yes | yes | yes | no | - | error
- // J | yes | yes | yes | yes | - | copy dir contents
- //
- // A. SRC specifies a file and DST (no trailing path separator) doesn't exist.
- //
- // This should create a file with the name DST and copy the contents of the
- // source file into it.
- func (s *DockerCLICpSuite) TestCpFromCaseA(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-a")
- defer os.RemoveAll(tmpDir)
- srcPath := containerCpPath(containerID, "/root/file1")
- dstPath := cpPath(tmpDir, "itWorks.txt")
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1\n"))
- }
- // B. SRC specifies a file and DST (with trailing path separator) doesn't exist.
- //
- // This should cause an error because the copy operation cannot create a directory
- // when copying a single file.
- func (s *DockerCLICpSuite) TestCpFromCaseB(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{addContent: true})
- tmpDir := getTestDir(c, "test-cp-from-case-b")
- defer os.RemoveAll(tmpDir)
- srcPath := containerCpPath(containerID, "/file1")
- dstDir := cpPathTrailingSep(tmpDir, "testDir")
- err := runDockerCp(c, srcPath, dstDir)
- assert.ErrorContains(c, err, "")
- assert.Assert(c, isCpDirNotExist(err), "expected DirNotExists error, but got %T: %s", err, err)
- }
- // C. SRC specifies a file and DST exists as a file.
- //
- // This should overwrite the file at DST with the contents of the source file.
- func (s *DockerCLICpSuite) TestCpFromCaseC(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-c")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcPath := containerCpPath(containerID, "/root/file1")
- dstPath := cpPath(tmpDir, "file2")
- // Ensure the local file starts with different content.
- assert.NilError(c, fileContentEquals(c, dstPath, "file2\n"))
- assert.NilError(c, runDockerCp(c, srcPath, dstPath))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1\n"))
- }
- // D. SRC specifies a file and DST exists as a directory.
- //
- // This should place a copy of the source file inside it using the basename from
- // SRC. Ensure this works whether DST has a trailing path separator or not.
- func (s *DockerCLICpSuite) TestCpFromCaseD(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{addContent: true})
- tmpDir := getTestDir(c, "test-cp-from-case-d")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcPath := containerCpPath(containerID, "/file1")
- dstDir := cpPath(tmpDir, "dir1")
- dstPath := filepath.Join(dstDir, "file1")
- // Ensure that dstPath doesn't exist.
- _, err := os.Stat(dstPath)
- assert.Assert(c, os.IsNotExist(err), "did not expect dstPath %q to exist", dstPath)
- assert.NilError(c, runDockerCp(c, srcPath, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1\n"))
- // Now try again but using a trailing path separator for dstDir.
- assert.NilError(c, os.RemoveAll(dstDir), "unable to remove dstDir")
- assert.NilError(c, os.MkdirAll(dstDir, os.FileMode(0755)), "unable to make dstDir")
- dstDir = cpPathTrailingSep(tmpDir, "dir1")
- assert.NilError(c, runDockerCp(c, srcPath, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1\n"))
- }
- // E. SRC specifies a directory and DST does not exist.
- //
- // This should create a directory at DST and copy the contents of the SRC directory
- // into the DST directory. Ensure this works whether DST has a trailing path
- // separator or not.
- func (s *DockerCLICpSuite) TestCpFromCaseE(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{addContent: true})
- tmpDir := getTestDir(c, "test-cp-from-case-e")
- defer os.RemoveAll(tmpDir)
- srcDir := containerCpPath(containerID, "dir1")
- dstDir := cpPath(tmpDir, "testDir")
- dstPath := filepath.Join(dstDir, "file1-1")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- // Now try again but using a trailing path separator for dstDir.
- assert.NilError(c, os.RemoveAll(dstDir), "unable to remove dstDir")
- dstDir = cpPathTrailingSep(tmpDir, "testDir")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- }
- // F. SRC specifies a directory and DST exists as a file.
- //
- // This should cause an error as it is not possible to overwrite a file with a
- // directory.
- func (s *DockerCLICpSuite) TestCpFromCaseF(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-f")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcDir := containerCpPath(containerID, "/root/dir1")
- dstFile := cpPath(tmpDir, "file1")
- err := runDockerCp(c, srcDir, dstFile)
- assert.ErrorContains(c, err, "")
- assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)
- }
- // G. SRC specifies a directory and DST exists as a directory.
- //
- // This should copy the SRC directory and all its contents to the DST directory.
- // Ensure this works whether DST has a trailing path separator or not.
- func (s *DockerCLICpSuite) TestCpFromCaseG(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-g")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcDir := containerCpPath(containerID, "/root/dir1")
- dstDir := cpPath(tmpDir, "dir2")
- resultDir := filepath.Join(dstDir, "dir1")
- dstPath := filepath.Join(resultDir, "file1-1")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- // Now try again but using a trailing path separator for dstDir.
- assert.NilError(c, os.RemoveAll(dstDir), "unable to remove dstDir")
- assert.NilError(c, os.MkdirAll(dstDir, os.FileMode(0755)), "unable to make dstDir")
- dstDir = cpPathTrailingSep(tmpDir, "dir2")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- }
- // H. SRC specifies a directory's contents only and DST does not exist.
- //
- // This should create a directory at DST and copy the contents of the SRC
- // directory (but not the directory itself) into the DST directory. Ensure
- // this works whether DST has a trailing path separator or not.
- func (s *DockerCLICpSuite) TestCpFromCaseH(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{addContent: true})
- tmpDir := getTestDir(c, "test-cp-from-case-h")
- defer os.RemoveAll(tmpDir)
- srcDir := containerCpPathTrailingSep(containerID, "dir1") + "."
- dstDir := cpPath(tmpDir, "testDir")
- dstPath := filepath.Join(dstDir, "file1-1")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- // Now try again but using a trailing path separator for dstDir.
- assert.NilError(c, os.RemoveAll(dstDir), "unable to remove resultDir")
- dstDir = cpPathTrailingSep(tmpDir, "testDir")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- }
- // I. SRC specifies a directory's contents only and DST exists as a file.
- //
- // This should cause an error as it is not possible to overwrite a file with a
- // directory.
- func (s *DockerCLICpSuite) TestCpFromCaseI(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-i")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcDir := containerCpPathTrailingSep(containerID, "/root/dir1") + "."
- dstFile := cpPath(tmpDir, "file1")
- err := runDockerCp(c, srcDir, dstFile)
- assert.ErrorContains(c, err, "")
- assert.Assert(c, isCpCannotCopyDir(err), "expected ErrCannotCopyDir error, but got %T: %s", err, err)
- }
- // J. SRC specifies a directory's contents only and DST exists as a directory.
- //
- // This should copy the contents of the SRC directory (but not the directory
- // itself) into the DST directory. Ensure this works whether DST has a
- // trailing path separator or not.
- func (s *DockerCLICpSuite) TestCpFromCaseJ(c *testing.T) {
- testRequires(c, DaemonIsLinux)
- containerID := makeTestContainer(c, testContainerOptions{
- addContent: true, workDir: "/root",
- })
- tmpDir := getTestDir(c, "test-cp-from-case-j")
- defer os.RemoveAll(tmpDir)
- makeTestContentInDir(c, tmpDir)
- srcDir := containerCpPathTrailingSep(containerID, "/root/dir1") + "."
- dstDir := cpPath(tmpDir, "dir2")
- dstPath := filepath.Join(dstDir, "file1-1")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- // Now try again but using a trailing path separator for dstDir.
- assert.NilError(c, os.RemoveAll(dstDir), "unable to remove dstDir")
- assert.NilError(c, os.MkdirAll(dstDir, os.FileMode(0755)), "unable to make dstDir")
- dstDir = cpPathTrailingSep(tmpDir, "dir2")
- assert.NilError(c, runDockerCp(c, srcDir, dstDir))
- assert.NilError(c, fileContentEquals(c, dstPath, "file1-1\n"))
- }
|