Add a one-second delay before the mock HTTP server starts in tests

amerkurev · Oct 31, 2023 · 917cc2e · 917cc2e
1 parent 6b3603f
commit 917cc2e
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 4 deletions.
diff --git a/app/bindmount/size_test.go b/app/bindmount/size_test.go
@@ -47,7 +47,7 @@ func Test_Run(t *testing.T) {
 	addr := fmt.Sprintf("127.0.0.1:%d", port)
 	mock := docker.NewMockServer(addr, version, logFile, mountDir)
 	mock.Start(t)
-	time.Sleep(10 * time.Millisecond)
+	time.Sleep(time.Second)
 
 	ctx, cancel := context.WithCancel(context.Background())
 	d, err := docker.NewClient(ctx, "http://"+addr, "", version, false)
@@ -82,7 +82,7 @@ func Test_Run_NoSuchFileOrDir(t *testing.T) {
 	version := "v1.22"
 	mock := docker.NewMockServer(addr, version, "incorrect-path", "incorrect-path")
 	mock.Start(t)
-	time.Sleep(10 * time.Millisecond)
+	time.Sleep(time.Second)
 
 	ctx, cancel := context.WithCancel(context.Background())
 	d, err := docker.NewClient(ctx, "http://"+addr, "", version, false)
@@ -106,7 +106,7 @@ func Test_Run_Failed(t *testing.T) {
 	addr := fmt.Sprintf("127.0.0.1:%d", port)
 	mock := docker.NewMockServer(addr, "", "", "")
 	mock.Start(t)
-	time.Sleep(10 * time.Millisecond)
+	time.Sleep(time.Second)
 
 	ctx, cancel := context.WithCancel(context.Background())
 	d, err := docker.NewClient(ctx, "http://"+addr, "", "", false)

diff --git a/app/docker/docker_test.go b/app/docker/docker_test.go
@@ -22,8 +22,8 @@ func Test_Client(t *testing.T) {
 	version := "v1.22"
 	mock := NewMockServer(addr, version, "", "")
 	mock.Start(t)
+	time.Sleep(time.Second)
 
-	time.Sleep(10 * time.Millisecond)
 	ctx := context.Background()
 
 	// bad host

diff --git a/app/http/handler/handler_test.go b/app/http/handler/handler_test.go
@@ -54,6 +54,8 @@ func Test_Handler(t *testing.T) {
 	version := "v1.22"
 	mock := docker.NewMockServer(dockerMockAddr, version, logFile, mountDir)
 	mock.Start(t)
+	time.Sleep(time.Second)
+
 	d, err := docker.NewClient(ctx, "http://"+dockerMockAddr, "", version, false)
 	require.NoError(t, err)
 
@@ -131,6 +133,8 @@ func Test_FailedToGetLogFile(t *testing.T) {
 	version := "v1.22"
 	mock := docker.NewMockServer(dockerMockAddr, version, "incorrect-path", "incorrect-path")
 	mock.Start(t)
+	time.Sleep(time.Second)
+
 	d, err := docker.NewClient(ctx, "http://"+dockerMockAddr, "", version, false)
 	require.NoError(t, err)
 

diff --git a/app/http/server_test.go b/app/http/server_test.go
@@ -78,6 +78,8 @@ func Test_Server_Run(t *testing.T) {
 	version := "v1.22"
 	mock := docker.NewMockServer(dockerMockAddr, version, logFile, mountDir)
 	mock.Start(t)
+	time.Sleep(time.Second)
+
 	d, err := docker.NewClient(ctx, "http://"+dockerMockAddr, "", version, false)
 	require.NoError(t, err)
 
@@ -152,6 +154,8 @@ func Test_Server_RunFailed(t *testing.T) {
 	version := "v1.22"
 	mock := docker.NewMockServer(dockerMockAddr, version, "", "")
 	mock.Start(t)
+	time.Sleep(time.Second)
+
 	d, err := docker.NewClient(ctx, "http://"+addr, "", version, false)
 	require.NoError(t, err)