Add validation for Azure DevOps org and PAT when link-commits-to-pull-request is true

Co-authored-by: joshjohanning <[email protected]>

Author: Copilot

__tests__/action.test.sh

@@ -270,6 +270,65 @@ test_comment_id_logic() {
     fi
 }
 
+# Test Suite: Azure DevOps Configuration Validation
+test_azdo_config_validation() {
+    echo ""
+    echo -e "${YELLOW}Testing: Azure DevOps Configuration Validation${NC}"
+    echo "========================================"
+    
+    # Test when both org and PAT are set (valid)
+    ORG="my-org"
+    PAT="my-pat-token"
+    if [ -z "$ORG" ] || [ -z "$PAT" ]; then
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_FAILED=$((TESTS_FAILED + 1))
+        echo -e "${RED}✗${NC} Should pass when both org and PAT are set"
+    else
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_PASSED=$((TESTS_PASSED + 1))
+        echo -e "${GREEN}✓${NC} Should pass when both org and PAT are set"
+    fi
+    
+    # Test when org is empty (invalid)
+    ORG=""
+    PAT="my-pat-token"
+    if [ -z "$ORG" ] || [ -z "$PAT" ]; then
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_PASSED=$((TESTS_PASSED + 1))
+        echo -e "${GREEN}✓${NC} Should fail when org is empty"
+    else
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_FAILED=$((TESTS_FAILED + 1))
+        echo -e "${RED}✗${NC} Should fail when org is empty"
+    fi
+    
+    # Test when PAT is empty (invalid)
+    ORG="my-org"
+    PAT=""
+    if [ -z "$ORG" ] || [ -z "$PAT" ]; then
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_PASSED=$((TESTS_PASSED + 1))
+        echo -e "${GREEN}✓${NC} Should fail when PAT is empty"
+    else
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_FAILED=$((TESTS_FAILED + 1))
+        echo -e "${RED}✗${NC} Should fail when PAT is empty"
+    fi
+    
+    # Test when both are empty (invalid)
+    ORG=""
+    PAT=""
+    if [ -z "$ORG" ] || [ -z "$PAT" ]; then
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_PASSED=$((TESTS_PASSED + 1))
+        echo -e "${GREEN}✓${NC} Should fail when both org and PAT are empty"
+    else
+        TESTS_RUN=$((TESTS_RUN + 1))
+        TESTS_FAILED=$((TESTS_FAILED + 1))
+        echo -e "${RED}✗${NC} Should fail when both org and PAT are empty"
+    fi
+}
+
 # Run all tests
 main() {
     echo "========================================"
@@ -283,6 +342,7 @@ main() {
     test_short_sha
     test_env_checks
     test_comment_id_logic
+    test_azdo_config_validation
 
     # Print summary
     echo ""

action.yml

@@ -86,7 +86,11 @@ runs:
               
               if ${{ inputs.link-commits-to-pull-request }}; then
                 # make the call to main.js to do the linking
-                # TODO: check to see if org/pat are set
+                # Check to see if org/pat are set
+                if [ -z "${{ inputs.azure-devops-organization }}" ] || [ -z "${{ inputs.azure-devops-token }}" ]; then
+                  echo "::error title=Missing Azure DevOps Configuration::Azure DevOps organization and token are required when link-commits-to-pull-request is true"
+                  exit 1
+                fi
                 echo "Attempting to link work item ${WORKITEM} to pull request ${PULL_NUMBER}..."
                 REPO_TOKEN=${{ inputs.github-token }} AZURE_DEVOPS_ORG=${{ inputs.azure-devops-organization }} AZURE_DEVOPS_PAT=${{ inputs.azure-devops-token }} WORKITEMID=$WORKITEM PULLREQUESTID=${{ github.event.number }} REPO=${{ github.repository }} node $main
                 echo "...PR linked to work item"