feat: validate that at least one check is enabled (#95)

* chore: empty commit to sync branch

* feat: validate that at least one check is enabled

Add validation to ensure at least one of 'check-commits' or 'check-pull-request' is enabled.
If both are set to false, the action will fail with a clear error message.

Changes:
- Add validation check in src/index.js (lines 42-48)
- Add 4 comprehensive test cases in __tests__/index.test.js
- All 55 JavaScript tests + 29 bash tests passing

Co-authored-by: joshjohanning <19912012+joshjohanning@users.noreply.github.com>

* chore: bump version to 3.0.6

* fix: improve error message for validation checks

* fix: update coverage percentage in SVG badge

---------

Co-authored-by: Josh Johanning <joshjohanning@github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: joshjohanning <19912012+joshjohanning@users.noreply.github.com>

Author: Copilot

__tests__/index.test.js

@@ -128,6 +128,84 @@ describe('Azure DevOps Commit Validator', () => {
       // Restore context
       mockContext.payload.pull_request = originalPR;
     });
+
+    it('should fail if both check-commits and check-pull-request are false', async () => {
+      mockGetInput.mockImplementation(name => {
+        if (name === 'check-commits') return 'false';
+        if (name === 'check-pull-request') return 'false';
+        if (name === 'github-token') return 'github-token';
+        return 'false';
+      });
+
+      await run();
+
+      expect(mockSetFailed).toHaveBeenCalledWith(
+        "At least one of 'check-commits' or 'check-pull-request' must be set to true. Both are currently set to false."
+      );
+    });
+
+    it('should pass when only check-commits is enabled', async () => {
+      mockGetInput.mockImplementation(name => {
+        if (name === 'check-commits') return 'true';
+        if (name === 'check-pull-request') return 'false';
+        if (name === 'github-token') return 'github-token';
+        if (name === 'fail-if-missing-workitem-commit-link') return 'false';
+        return 'false';
+      });
+
+      mockOctokit.rest.pulls.listCommits.mockResolvedValue({
+        data: []
+      });
+
+      await run();
+
+      expect(mockSetFailed).not.toHaveBeenCalled();
+    });
+
+    it('should pass when only check-pull-request is enabled', async () => {
+      mockGetInput.mockImplementation(name => {
+        if (name === 'check-commits') return 'false';
+        if (name === 'check-pull-request') return 'true';
+        if (name === 'github-token') return 'github-token';
+        return 'false';
+      });
+
+      mockOctokit.rest.pulls.get.mockResolvedValue({
+        data: {
+          title: 'Test PR AB#123',
+          body: 'Test body'
+        }
+      });
+
+      await run();
+
+      expect(mockSetFailed).not.toHaveBeenCalled();
+    });
+
+    it('should pass when both checks are enabled', async () => {
+      mockGetInput.mockImplementation(name => {
+        if (name === 'check-commits') return 'true';
+        if (name === 'check-pull-request') return 'true';
+        if (name === 'github-token') return 'github-token';
+        if (name === 'fail-if-missing-workitem-commit-link') return 'false';
+        return 'false';
+      });
+
+      mockOctokit.rest.pulls.listCommits.mockResolvedValue({
+        data: []
+      });
+
+      mockOctokit.rest.pulls.get.mockResolvedValue({
+        data: {
+          title: 'Test PR AB#123',
+          body: 'Test body'
+        }
+      });
+
+      await run();
+
+      expect(mockSetFailed).not.toHaveBeenCalled();
+    });
   });
 
   describe('Commit validation', () => {

badges/coverage.svg

@@ -1,6 +1,6 @@
 {
   "name": "azure-devops-work-item-link-enforcer-and-linker",
-  "version": "3.0.5",
+  "version": "3.0.6",
   "private": true,
   "type": "module",
   "description": "GitHub Action to enforce that each commit in a pull request be linked to an Azure DevOps work item and automatically link the pull request to each work item ",

package-lock.json

@@ -39,6 +39,14 @@ export async function run() {
     const commentOnFailure = core.getInput('comment-on-failure') === 'true';
     const validateWorkItemExistsFlag = core.getInput('validate-work-item-exists') === 'true';
 
+    // Validate that at least one check is enabled
+    if (!checkPullRequest && !checkCommits) {
+      core.setFailed(
+        `At least one of 'check-commits' or 'check-pull-request' must be set to true. Both are currently set to false.`
+      );
+      return;
+    }
+
     // Get context
     const context = github.context;
     const pullNumber = context.payload.pull_request?.number;