ppkarwasz commented on code in PR #418:
URL: https://github.com/apache/logging-parent/pull/418#discussion_r2150406188


##########
.github/actions/generate-dependabot-changelog/src/pull_request.test.ts:
##########
@@ -0,0 +1,102 @@
+// SPDX-License-Identifier: Apache-2.0
+import { getAssociatedPullRequest } from "./pull_request";
+import { graphql } from "@octokit/graphql";
+
+jest.mock("@octokit/graphql");
+
+describe("getAssociatedPullRequest", () => {
+  const mockGraphql = graphql.defaults as jest.Mock;
+
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it("should return the associated pull request when the query is successful", 
async () => {
+    const mockResponse = {
+      repository: {
+        object: {
+          associatedPullRequests: {
+            nodes: [
+              {
+                number: 123,
+                url: "https://github.com/owner/repo/pull/123";,
+              },
+            ],
+          },
+        },
+      },
+    };
+
+    mockGraphql.mockReturnValueOnce(jest.fn().mockResolvedValue(mockResponse));
+
+    const owner = "owner";
+    const repo = "repo";
+    const sha = "abc123";
+    const token = "test-token";
+
+    const result = await getAssociatedPullRequest(owner, repo, sha, token);
+
+    expect(result).toEqual({
+      number: 123,
+      url: new URL("https://github.com/owner/repo/pull/123";),
+    });
+  });
+
+  it("should throw an error when the query fails", async () => {
+    const mockError = new Error("GraphQL query failed");
+    mockGraphql.mockReturnValueOnce(jest.fn().mockRejectedValue(mockError));
+
+    const owner = "owner";
+    const repo = "repo";
+    const sha = "abc123";
+    const token = "test-token";
+
+    await expect(
+      getAssociatedPullRequest(owner, repo, sha, token),
+    ).rejects.toThrow("GraphQL query failed");
+  });
+
+  it("should return null if there are no PRs associated", async () => {
+    [
+      null,
+      // Wrong `owner` or `repo` values
+      {
+        repository: null,
+      },
+      // Wrong `sha` value
+      {
+        repository: {
+          object: null,
+        },
+      },
+      // Sha is not a commit
+      {
+        repository: {
+          object: {},
+        },
+      },
+      // No associated PRs
+      {
+        repository: {
+          object: {
+            associatedPullRequests: {
+              nodes: [],
+            },
+          },
+        },
+      },
+    ].forEach(async (mockResponse) => {

Review Comment:
   Fixed in 
https://github.com/apache/logging-parent/pull/418/commits/a125ff476bf60e2e124dc32d85c477c143b84174



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to