acc: fix -logrequests option (#3731)

It was only enabled where there was a dedicated server started for this
test.

Author: denik

acceptance/acceptance_test.go

@@ -228,7 +228,7 @@ func testAccept(t *testing.T, inprocessMode bool, singleTest string) int {
 	cloudEnv := os.Getenv("CLOUD_ENV")
 
 	if cloudEnv == "" {
-		internal.StartDefaultServer(t)
+		internal.StartDefaultServer(t, LogRequests)
 		if os.Getenv("TEST_DEFAULT_WAREHOUSE_ID") == "" {
 			t.Setenv("TEST_DEFAULT_WAREHOUSE_ID", "8ec9edc1-db0c-40df-af8d-7580020fe61e")
 		}

acceptance/internal/prepare_server.go

@@ -25,10 +25,15 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-func StartDefaultServer(t *testing.T) {
+func StartDefaultServer(t *testing.T, logRequests bool) {
 	s := testserver.New(t)
 	testserver.AddDefaultHandlers(s)
 
+	// Log API responses if the -logrequests flag is set.
+	if logRequests {
+		s.ResponseCallback = logResponseCallback(t)
+	}
+
 	t.Setenv("DATABRICKS_DEFAULT_HOST", s.URL)
 
 	// Do not read user's ~/.databrickscfg.