add integration test

acceptance/telemetry/test.toml

@@ -1,5 +1,3 @@
-LocalOnly = true
-RecordRequests = true
 IncludeRequestHeaders = ["Authorization"]
 
 [[Repls]]

acceptance/telemetry/upload-command/out.upload.txt

@@ -1,3 +1,3 @@
 Telemetry logs uploaded successfully
 Response:
-{"errors":null,"numProtoSuccess":2}
+{"errors":[],"numProtoSuccess":2}

acceptance/telemetry/upload-command/test.toml

@@ -1,7 +1,11 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''
 {
+    "errors": [],
     "numProtoSuccess": 2
 }
 '''

acceptance/telemetry/upload-fails/test.toml

@@ -1,3 +1,6 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''

acceptance/telemetry/upload-skipped/test.toml

@@ -1,7 +1,11 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''
 {
+    "errors": [],
     "numProtoSuccess": 2
 }
 '''

acceptance/telemetry/upload-succeeds-integration/out.upload_process.txt

@@ -0,0 +1,3 @@
+Telemetry logs uploaded successfully
+Response:
+{"errors":[],"numProtoSuccess":2}

acceptance/telemetry/upload-succeeds-integration/output.txt

@@ -0,0 +1,5 @@
+
+>>> [CLI] selftest send-telemetry
+waiting for telemetry process to finish
+hello
+[wait_pid] process has ended

acceptance/telemetry/upload-succeeds-integration/script

@@ -0,0 +1,14 @@
+export DATABRICKS_CLI_TELEMETRY_PID_FILE=./telemetry.pid
+export DATABRICKS_CLI_TELEMETRY_UPLOAD_LOGS_FILE=./out.upload_process.txt
+
+trace $CLI selftest send-telemetry
+
+echo "waiting for telemetry process to finish"
+
+echo "hello"
+
+# Wait for the child telemetry process to finish
+wait_pid $(cat ./telemetry.pid)
+
+# cleanup the pid file
+rm -f ./telemetry.pid

acceptance/telemetry/upload-succeeds-integration/test.toml

@@ -0,0 +1,16 @@
+# RecordRequests is not supported for integration tests yet. Once we have support
+# for running integration tests with record requests we can remove this test since
+# it's a copy of acceptance/telemetry/upload-succeeds.
+
+[[Server]]
+Pattern = "POST /telemetry-ext"
+Response.Body = '''
+{
+    "errors": [],
+    "numProtoSuccess": 2
+}
+'''
+
+[[Repls]]
+Old = 'execution_time_ms\\\":\d{1,5},'
+New = 'execution_time_ms\":\"SMALL_INT\",'

acceptance/telemetry/upload-succeeds-oauth/out.upload_process.txt

@@ -1,3 +1,3 @@
 Telemetry logs uploaded successfully
 Response:
-{"errors":null,"numProtoSuccess":2}
+{"errors":[],"numProtoSuccess":2}

acceptance/telemetry/upload-succeeds-oauth/test.toml

@@ -1,7 +1,11 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''
 {
+    "errors": [],
     "numProtoSuccess": 2
 }
 '''

acceptance/telemetry/upload-succeeds/out.upload_process.txt

@@ -1,3 +1,3 @@
 Telemetry logs uploaded successfully
 Response:
-{"errors":null,"numProtoSuccess":2}
+{"errors":[],"numProtoSuccess":2}

acceptance/telemetry/upload-succeeds/test.toml

@@ -1,7 +1,11 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''
 {
+    "errors": [],
     "numProtoSuccess": 2
 }
 '''

acceptance/telemetry/upload-timeout/test.toml

@@ -1,7 +1,11 @@
+LocalOnly = true
+RecordRequests = true
+
 [[Server]]
 Pattern = "POST /telemetry-ext"
 Response.Body = '''
 {
+    "errors": [],
     "numProtoSuccess": 2
 }
 '''