add test to ensure that concurrent file uploads don't result in an error

Signed-off-by: Callum Styan <callumstyan@gmail.com>
coder · cstyan · Jul 30, 2025 · Jul 22, 2025 · Jul 22, 2025 · Jul 22, 2025
commit df947efe57e6bf66b36ff741d27979120a5190cf
diff --git a/coderd/files_test.go b/coderd/files_test.go
@@ -5,6 +5,7 @@
 	"bytes"
 	"context"
 	"net/http"
+	"sync"
 	"testing"
 
 	"github.com/google/uuid"
@@ -69,6 +70,34 @@
 		_, err = client.Upload(ctx, codersdk.ContentTypeTar, bytes.NewReader(data))
 		require.NoError(t, err)
 	})
+	t.Run("InsertConcurrent", func(t *testing.T) {
+		t.Parallel()
+		client := coderdtest.New(t, nil)
+		_ = coderdtest.CreateFirstUser(t, client)
+
+		ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitLong)
+		defer cancel()
+
+		var wg sync.WaitGroup
+		var end sync.WaitGroup
+		wg.Add(1)
+		end.Add(3)
+		for range 3 {
+			go func() {
+				wg.Wait()
+				data := make([]byte, 1024)
+				_, err := client.Upload(ctx, codersdk.ContentTypeTar, bytes.NewReader(data))
+				end.Done()
+				require.NoError(t, err)
+			}()
+
+		}
+		wg.Done()
+		end.Wait()
+
+		// _, err = client.Upload(ctx, codersdk.ContentTypeTar, bytes.NewReader(data))
+		// require.NoError(t, err)
+	})
 }
 
 func TestDownload(t *testing.T) {