Skip to content

Commit 5b52766

Browse files
committed
test(progress-api): update progress messages and notification settings
- Update progress notification title to "Diff Commit" - Make progress notification cancellable - Standardise progress message text: - Change "Checking API key" to "Validating API key" - Update "Generating message" to "Generating commit message"
1 parent c3033d5 commit 5b52766

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

test/withProgressAPI.test.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@ describe("Progress API Integration", () => {
6565
expect(window.withProgress).toHaveBeenCalledWith(
6666
{
6767
location: ProgressLocation.Notification,
68-
title: "Generating commit message...",
69-
cancellable: false,
68+
title: "Diff Commit",
69+
cancellable: true,
7070
},
7171
expect.any(Function),
7272
)
@@ -88,8 +88,8 @@ describe("Progress API Integration", () => {
8888
// Verify progress messages in order
8989
expect(mockProgress.report.mock.calls).toEqual([
9090
[{ message: "Getting git diff..." }],
91-
[{ message: "Checking API key..." }],
92-
[{ message: "Generating message..." }],
91+
[{ message: "Validating API key..." }],
92+
[{ message: "Generating commit message..." }],
9393
])
9494
})
9595

@@ -120,7 +120,7 @@ describe("Progress API Integration", () => {
120120
// Should show first two progress messages
121121
expect(mockProgress.report.mock.calls).toEqual([
122122
[{ message: "Getting git diff..." }],
123-
[{ message: "Checking API key..." }],
123+
[{ message: "Validating API key..." }],
124124
])
125125

126126
// Should show error message
@@ -139,7 +139,7 @@ describe("Progress API Integration", () => {
139139
// Should show first two progress messages
140140
expect(mockProgress.report.mock.calls).toEqual([
141141
[{ message: "Getting git diff..." }],
142-
[{ message: "Checking API key..." }],
142+
[{ message: "Validating API key..." }],
143143
])
144144

145145
// Should show error message
@@ -166,8 +166,8 @@ describe("Progress API Integration", () => {
166166
// Verify all progress messages shown
167167
expect(mockProgress.report.mock.calls).toEqual([
168168
[{ message: "Getting git diff..." }],
169-
[{ message: "Checking API key..." }],
170-
[{ message: "Generating message..." }],
169+
[{ message: "Validating API key..." }],
170+
[{ message: "Generating commit message..." }],
171171
])
172172

173173
// Verify commit message was set

0 commit comments

Comments
 (0)