check label name when set label action (#36590)

Author: raych1

.github/workflows/src/sdk-breaking-change-labels.js

@@ -99,8 +99,8 @@ export async function getLabelAndActionImpl({ details_url, core, retryOptions =
     }
   }
 
-  if (!labelAction) {
-    core.info("No label action found, defaulting to None");
+  if (!labelAction || !labelName) {
+    core.info("No label action or name found, defaulting to None");
     labelAction = LabelAction.None;
   }
 

.github/workflows/test/sdk-breaking-change-labels.test.js

@@ -138,6 +138,62 @@ describe("sdk-breaking-change-labels", () => {
         issueNumber: 123,
       });
     });
+    it("should correctly set labelAction to none when label name is empty", async () => {
+      // Setup inputs
+      const inputs = {
+        details_url: "https://dev.azure.com/project/_build/results?buildId=12345",
+        head_sha: "abc123",
+      };
+
+      // Setup mock for extractInputs
+      const { extractInputs } = await import("../src/context.js");
+      extractInputs.mockResolvedValue(inputs);
+
+      // Mock artifact responses with 'remove' action
+      const mockArtifactResponse = {
+        ok: true,
+        json: vi.fn().mockResolvedValue({
+          resource: {
+            downloadUrl: "https://dev.azure.com/download?format=zip",
+          },
+        }),
+      };
+
+      const language = "azure-sdk-for-java";
+      const mockContentResponse = {
+        ok: true,
+        text: vi.fn().mockResolvedValue(
+          JSON.stringify({
+            labelAction: false,
+            language,
+            prNumber: "123",
+          }),
+        ),
+      };
+
+      // Setup fetch to return different responses for each call
+      global.fetch.mockImplementation((url) => {
+        if (url.includes("artifacts?artifactName=")) {
+          return mockArtifactResponse;
+        } else {
+          return mockContentResponse;
+        }
+      });
+
+      // Call function
+      const result = await getLabelAndAction({
+        github: mockGithub,
+        context: mockContext,
+        core: mockCore,
+      });
+
+      // Verify result has none action
+      expect(result).toEqual({
+        labelName: sdkLabels[language].breakingChange,
+        labelAction: LabelAction.None,
+        issueNumber: 123,
+      });
+    });
     it("should throw error with invalid inputs", async () => {
       // Setup inputs
       const inputs = {