obsidian-plugin/andy-stack_vaultkeeper-ai
1
0
Fork 0
mirror of https://github.com/andy-stack/vaultkeeper-ai.git synced 2026-07-22 06:42:03 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

refactor: remove unused import and redundant test cases

Browse source 
Remove unused AIFunctionResponse import from BaseAgent and delete
non-contentEditable element warning tests that are no longer needed
This commit is contained in:
Andrew Beal 2026-01-30 22:10:11 +00:00
parent 3d56643772
commit 6e24ba20cc
2 changed files with 0 additions and 38 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Services/AIServices/BaseAgent.ts
View file

@ -1,5 +1,4 @@
import type { AIFunctionCall } from "AIClasses/AIFunctionCall";
import { AIFunctionResponse } from "AIClasses/FunctionDefinitions/AIFunctionResponse";
import type { IAIClass } from "AIClasses/IAIClass";
import type { IPrompt } from "AIPrompts/IPrompt";
import type { Conversation } from "Conversations/Conversation";

37
__tests__/Services/InputService.test.ts
View file

@ -515,18 +515,6 @@ describe('InputService', () => {
service.insertTextAtCursor('Test', element);
expect(element.textContent).toBe(originalText);
});
it('should warn and not insert for non-contentEditable element', () => {
const nonEditable = document.createElement('div');
nonEditable.textContent = 'Not editable';
document.body.appendChild(nonEditable);
const consoleSpy = vi.spyOn(console, 'warn');
service.insertTextAtCursor('Test', nonEditable);
expect(consoleSpy).toHaveBeenCalledWith('Element must be contenteditable');
document.body.removeChild(nonEditable);
});
});
describe('insertElementAtCursor', () => {
@ -579,19 +567,6 @@ describe('InputService', () => {
service.insertElementAtCursor(span, element);
expect(element.childNodes.length).toBe(childrenBefore);
});
it('should warn and not insert for non-contentEditable element', () => {
const nonEditable = document.createElement('div');
nonEditable.textContent = 'Not editable';
document.body.appendChild(nonEditable);
const consoleSpy = vi.spyOn(console, 'warn');
const span = document.createElement('span');
service.insertElementAtCursor(span, nonEditable);
expect(consoleSpy).toHaveBeenCalledWith('Element must be contenteditable');
document.body.removeChild(nonEditable);
});
});
describe('deleteTextRange', () => {
@ -632,18 +607,6 @@ describe('InputService', () => {
expect(element.textContent).toBe('Hello ld');
});
it('should warn for non-contentEditable element', () => {
const nonEditable = document.createElement('div');
nonEditable.textContent = 'Not editable';
document.body.appendChild(nonEditable);
const consoleSpy = vi.spyOn(console, 'warn');
service.deleteTextRange(0, 3, nonEditable);
expect(consoleSpy).toHaveBeenCalledWith('Element must be contenteditable');
expect(nonEditable.textContent).toBe('Not editable'); // Unchanged
document.body.removeChild(nonEditable);
});
it('should handle invalid range gracefully', () => {
const originalText = element.textContent;