Skip to content

Commit f0af09a

Browse files
Address code review feedback
- Use more specific error message: 'Error retrieving operation' vs 'Unable to retrieve operation' - Fix test name to accurately reflect what is being tested Co-authored-by: Andrew-Paystack <78197464+Andrew-Paystack@users.noreply.github.com>
1 parent c89d346 commit f0af09a

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/tools/get-paystack-operation.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export function registerGetPaystackOperationTool(
5454
content: [
5555
{
5656
type: "text",
57-
text: `Unable to retrieve operation: ${errorMessage}`,
57+
text: `Error retrieving operation: ${errorMessage}`,
5858
},
5959
],
6060
isError: true

test/make-paystack-request-tool.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ describe("MakePaystackRequestTool", () => {
5353
}
5454
});
5555

56-
it("should return isError: false (omitted) for successful responses", async () => {
56+
it("should omit isError for successful responses", async () => {
5757
// Mock fetch to return valid JSON
5858
const originalFetch = global.fetch;
5959
const validJsonResponse = {

0 commit comments

Comments
 (0)