README.md

@@ -26,6 +26,10 @@ Name of the environment variable that contains the Test Analytics API token.
 
 Default value: `BUILDKITE_ANALYTICS_TOKEN`
 
+#### `api-url` (string)
+
+Full URL for the API to upload to. Defaults to `https://analytics-api.buildkite.com/v1/uploads`
+
 #### `branches` (string)
 
 String containing a regex to only do an upload in branches that match it (using the case-insensitive bash `=~` operator against the `BUILDKITE_BRANCH` environment variable).
@@ -79,7 +83,7 @@ steps:
   - label: "🔨 Test"
     command: "make test"
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "test/junit-*.xml"
           format: "junit"
 ```
@@ -93,7 +97,7 @@ steps:
   - label: "🔨 Test"
     command: "make test"
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "test-data-*.json"
           format: "json"
 ```
@@ -114,7 +118,7 @@ steps:
   - label: "🔍 Test Analytics"
     command: buildkite-agent artifact download tests-*.xml
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "tests-*.xml"
           format: "junit"
 ```
@@ -128,7 +132,7 @@ steps:
   - label: "🔨 Test"
     command: "make test"
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "test-data-*.json"
           format: "json"
           branches: "-qa$"
@@ -141,7 +145,7 @@ steps:
   - label: "🔨 Test"
     command: "make test"
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "test-data-*.json"
           format: "json"
           exclude-branches: "^legacy$"
@@ -154,7 +158,7 @@ steps:
   - label: "🔨 Test"
     command: "make test"
     plugins:
-      - test-collector#v1.4.0:
+      - test-collector#v1.5.0:
           files: "test-data-*.json"
           format: "json"
           branches: "^stage-"

docker-compose.yml

@@ -1,6 +1,6 @@
 version: '2'
 services:
   tests:
-    image: buildkite/plugin-tester:v3.0.1
+    image: buildkite/plugin-tester:v4.0.0
     volumes:
       - ".:/plugin:ro"

hooks/pre-exit

@@ -64,7 +64,7 @@ upload() {
     curl_args+=("--form" "run_env[version]=$PLUGIN_VERSION")
   fi
 
-  curl_args+=("https://analytics-api.buildkite.com/v1/uploads")
+  curl_args+=("${BUILDKITE_PLUGIN_TEST_COLLECTOR_API_URL:-https://analytics-api.buildkite.com/v1/uploads}")
 
   # Print debugging output before we add the token, so it doesn't ever get
   # printed to output
@@ -105,14 +105,15 @@ done < <("${FIND_CMD[@]}" . -path "./${FILES_PATTERN}")
 
 if [[ "${#matching_files[@]}" -eq "0" ]]; then
   echo "No files found matching '${FILES_PATTERN}'"
-  if [[ -z "${BUILDKITE_COMMAND_EXIT_STATUS+x}" ]] || [[ "${BUILDKITE_COMMAND_EXIT_STATUS}" -eq "0" ]]; then
+  if [[ "${BUILDKITE_COMMAND_EXIT_STATUS:-0}" -eq "0" ]]; then
     exit 1
   fi
-fi
-
-for file in "${matching_files[@]}"; do
-  echo "Uploading '$file'..."
-  if ! upload "$TOKEN_VALUE" "$FORMAT" "${file}"; then
-    echo "Error uploading, will continue"
-  fi
-done
+else
+  # needs to be part of else for bash4.3 compatibility
+  for file in "${matching_files[@]}"; do
+    echo "Uploading '$file'..."
+    if ! upload "$TOKEN_VALUE" "$FORMAT" "${file}"; then
+      echo "Error uploading, will continue"
+    fi
+  done
+fi

plugin.yml

@@ -7,6 +7,8 @@ configuration:
   properties:
     api-token-env-name:
       type: string
+    api-url:
+      type: string
     branches:
       type: string
     debug:

tests/pre-exit-success.bats

@@ -152,3 +152,18 @@ COMMON_CURL_OPTIONS='--form \* --form \* --form \* --form \* --form \* --form \*
   assert_success
   assert_output --partial "curl success"
 }
+
+@test "API can change" {
+  export BUILDKITE_PLUGIN_TEST_COLLECTOR_API_URL='https://test-api.example.com/v2'
+
+  stub curl "-X POST --silent --show-error --max-time 30 --form format=junit ${COMMON_CURL_OPTIONS} \* -H \* : echo \"curl success against \${29}\""
+
+  run "$PWD/hooks/pre-exit"
+
+  assert_success
+  assert_output --partial "Uploading './tests/fixtures/junit-1.xml'..."
+  assert_output --partial "curl success"
+  assert_output --partial "against https://test-api.example.com/v2"
+
+  unstub curl
+}