fix: increase render interval to 30s if CI env var is true (#2440)

Fixes #2251

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Author: efekarakus

.release/buildspec_e2e.yml

@@ -155,5 +155,6 @@ phases:
          docker run --privileged -v /tmp/.aws:/home/.aws -e "HOME=/home"
          -e "TEST_SUITE=$TEST_SUITE"
          -e "DOCKERHUB_USERNAME=$DOCKERHUB_USERNAME"
-         -e "DOCKERHUB_TOKEN=$DOCKERHUB_TOKEN" 
+         -e "DOCKERHUB_TOKEN=$DOCKERHUB_TOKEN"
+         -e "CI=true"
          copilot-cli/e2e:latest

internal/pkg/term/progress/render_interval.go

@@ -5,8 +5,18 @@
 
 package progress
 
-import "time"
+import (
+	"os"
+	"time"
+)
 
-const (
+var (
 	renderInterval = 100 * time.Millisecond // How frequently Render should be invoked.
 )
+
+func init() {
+	// The CI environment variable is set to "true" by default by GitHub actions and GitLab.
+	if os.Getenv("CI") == "true" {
+		renderInterval = 30 * time.Second
+	}
+}

internal/pkg/term/progress/render_interval_windows.go

@@ -3,9 +3,18 @@
 
 package progress
 
-import "time"
+import (
+	"os"
+	"time"
+)
 
-const (
+var (
 	// Windows flickers too frequently if the interval is too short.
 	renderInterval = 500 * time.Millisecond // How frequently Render should be invoked.
 )
+
+func init() {
+	if os.Getenv("CI") == "true" {
+		renderInterval = 30 * time.Second
+	}
+}

internal/pkg/term/progress/render_test.go

@@ -62,6 +62,8 @@ func (m *mockFileWriter) Fd() uintptr {
 }
 
 func TestRender(t *testing.T) {
+	renderInterval = 100 * time.Millisecond // Ensure that even when CI=true we are testing with default interval.
+
 	t.Run("stops the renderer when context is canceled", func(t *testing.T) {
 		t.Parallel()
 		// GIVEN