Skip to content

Commit aa1628c

Browse files
committed
fix(tests): improve clarity of Ollama model update success messages
The commit updates the expected status bar messages in the OllamaManager tests to use more consistent and clear wording. It changes "model 'x' selected successfully" and "model 'x' changed to successfully" to the more precise "model updated to 'x' successfully" format, improving readability and consistency across the test suite.
1 parent 4993f40 commit aa1628c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/ollamaManager.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ describe("OllamaManager", () => {
112112
expect(mockConfigUpdate).toHaveBeenCalledWith("provider", "ollama", true)
113113
expect(mockConfigUpdate).toHaveBeenCalledWith("ollamaHostname", "http://localhost:11434", true)
114114
expect(mockConfigUpdate).toHaveBeenCalledWith("ollamaModel", "llama3.2", true)
115-
expect(mockSetStatusBarMessage).toHaveBeenCalledWith("✓ Ollama model 'llama3.2' selected successfully", 4000)
115+
expect(mockSetStatusBarMessage).toHaveBeenCalledWith("✓ Ollama model updated to 'llama3.2' successfully", 4000)
116116
})
117117

118118
it("should return false when user cancels hostname input", async () => {
@@ -243,7 +243,7 @@ describe("OllamaManager", () => {
243243
expect(mockConfigUpdate).not.toHaveBeenCalledWith("provider", "ollama", true)
244244
expect(mockConfigUpdate).not.toHaveBeenCalledWith("ollamaHostname", expect.anything(), true)
245245
expect(mockConfigUpdate).toHaveBeenCalledWith("ollamaModel", "codellama", true)
246-
expect(mockSetStatusBarMessage).toHaveBeenCalledWith("✓ Ollama model 'codellama' changed to successfully", 4000)
246+
expect(mockSetStatusBarMessage).toHaveBeenCalledWith("✓ Ollama model updated to 'codellama' successfully", 4000)
247247
})
248248

249249
it("should handle errors during model changing", async () => {

0 commit comments

Comments
 (0)