docs: add comment with TODO for refactoring global init() starting te…

…st containers
Layr-Labs · Oct 16, 2024 · a9ae57b · a9ae57b
1 parent bbd81b0
commit a9ae57b
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/e2e/setup.go b/e2e/setup.go
@@ -47,6 +47,10 @@ var (
 	redisEndpoint = ""
 )
 
+// TODO: we shouldn't start the containers in the init function like this.
+// Need to find a better way to start the containers and set the endpoints.
+// Even better would be for the endpoints not to be global variables injected into the test configs.
+// Starting the containers on init like this also makes it harder to import this file into other tests.
 func init() {
 	err := startMinIOContainer()
 	if err != nil {