Skip to content

Commit 8c1e9bd

Browse files
committed
Merge pull request #1969 from johanneswuerbach/database-tests
fix(tests): data containers should not be shared
2 parents b7eff72 + 640892a commit 8c1e9bd

3 files changed

Lines changed: 9 additions & 6 deletions

File tree

database/tests/database_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ func TestDatabase(t *testing.T) {
1717
cli, stdout, stdoutPipe := dockercli.NewClient()
1818
dockercli.RunTestEtcd(t, etcdName, etcdPort)
1919
defer cli.CmdRm("-f", etcdName)
20-
dockercli.RunDeisDataTest(t, "--name", "deis-database-data",
20+
dataName := "deis-database-data-" + tag
21+
dockercli.RunDeisDataTest(t, "--name", dataName,
2122
"-v", "/var/lib/postgresql", "deis/base", "true")
2223
host, port := utils.HostAddress(), utils.RandomPort()
2324
fmt.Printf("--- Run deis/database:%s at %s:%s\n", tag, host, port)
@@ -32,7 +33,7 @@ func TestDatabase(t *testing.T) {
3233
"-e", "EXTERNAL_PORT="+port,
3334
"-e", "HOST="+host,
3435
"-e", "ETCD_PORT="+etcdPort,
35-
"--volumes-from", "deis-database-data",
36+
"--volumes-from", dataName,
3637
"deis/database:"+tag)
3738
}()
3839
dockercli.PrintToStdout(t, stdout, stdoutPipe, "deis-database running")

logger/tests/logger_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ func TestLogger(t *testing.T) {
1717
cli, stdout, stdoutPipe := dockercli.NewClient()
1818
dockercli.RunTestEtcd(t, etcdName, etcdPort)
1919
defer cli.CmdRm("-f", etcdName)
20-
dockercli.RunDeisDataTest(t, "--name", "deis-logger-data",
20+
dataName := "deis-logger-data-" + tag
21+
dockercli.RunDeisDataTest(t, "--name", dataName,
2122
"-v", "/var/log/deis", "deis/base", "/bin/true")
2223
host, port := utils.HostAddress(), utils.RandomPort()
2324
fmt.Printf("--- Run deis/logger:%s at %s:%s\n", tag, host, port)
@@ -32,7 +33,7 @@ func TestLogger(t *testing.T) {
3233
"-e", "EXTERNAL_PORT="+port,
3334
"-e", "HOST="+host,
3435
"-e", "ETCD_PORT="+etcdPort,
35-
"--volumes-from", "deis-logger-data",
36+
"--volumes-from", dataName,
3637
"deis/logger:"+tag)
3738
}()
3839
dockercli.PrintToStdout(t, stdout, stdoutPipe, "deis-logger running")

registry/tests/registry_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ func TestRegistry(t *testing.T) {
2626
defer cli.CmdRm("-f", etcdName)
2727
handler := etcdutils.InitEtcd(setdir, setkeys, etcdPort)
2828
etcdutils.PublishEtcd(t, handler)
29-
dockercli.RunDeisDataTest(t, "--name", "deis-registry-data",
29+
dataName := "deis-registry-data-" + tag
30+
dockercli.RunDeisDataTest(t, "--name", dataName,
3031
"-v", "/data", "deis/base", "/bin/true")
3132
host, port := utils.HostAddress(), utils.RandomPort()
3233
fmt.Printf("--- Run deis/registry:%s at %s:%s\n", tag, host, port)
@@ -41,7 +42,7 @@ func TestRegistry(t *testing.T) {
4142
"-e", "EXTERNAL_PORT="+port,
4243
"-e", "HOST="+host,
4344
"-e", "ETCD_PORT="+etcdPort,
44-
"--volumes-from", "deis-registry-data",
45+
"--volumes-from", dataName,
4546
"deis/registry:"+tag)
4647
}()
4748
dockercli.PrintToStdout(t, stdout, stdoutPipe, "Booting")

0 commit comments

Comments
 (0)