+274

tests/search-state.test.ts

reviewed

··· 1 1 + /** 2 2 + * Tests for SearchState — find-and-replace pure logic (src/docs/search-state.ts). 3 3 + * Covers findMatches, navigation (next/prev), replaceOne, replaceAll, and case sensitivity. 4 4 + */ 5 5 + import { describe, it, expect } from 'vitest'; 6 6 + import { SearchState } from '../src/docs/search-state.js'; 7 7 + 8 8 + // ===================================================================== 9 9 + // Constructor 10 10 + // ===================================================================== 11 11 + 12 12 + describe('SearchState — constructor', () => { 13 13 + it('defaults to case-insensitive', () => { 14 14 + const s = new SearchState(); 15 15 + expect(s.caseSensitive).toBe(false); 16 16 + }); 17 17 + 18 18 + it('accepts caseSensitive option', () => { 19 19 + const s = new SearchState({ caseSensitive: true }); 20 20 + expect(s.caseSensitive).toBe(true); 21 21 + }); 22 22 + 23 23 + it('starts with no matches', () => { 24 24 + const s = new SearchState(); 25 25 + expect(s.matches).toEqual([]); 26 26 + expect(s.currentIndex).toBe(-1); 27 27 + expect(s.matchCount).toBe(0); 28 28 + expect(s.currentMatch).toBeNull(); 29 29 + }); 30 30 + }); 31 31 + 32 32 + // ===================================================================== 33 33 + // findMatches 34 34 + // ===================================================================== 35 35 + 36 36 + describe('SearchState — findMatches', () => { 37 37 + it('finds single occurrence', () => { 38 38 + const s = new SearchState(); 39 39 + const matches = s.findMatches('hello world', 'world'); 40 40 + expect(matches).toEqual([{ from: 6, to: 11 }]); 41 41 + expect(s.currentIndex).toBe(0); 42 42 + }); 43 43 + 44 44 + it('finds multiple occurrences', () => { 45 45 + const s = new SearchState(); 46 46 + const matches = s.findMatches('abcabc', 'abc'); 47 47 + expect(matches.length).toBe(2); 48 48 + expect(matches[0]).toEqual({ from: 0, to: 3 }); 49 49 + expect(matches[1]).toEqual({ from: 3, to: 6 }); 50 50 + }); 51 51 + 52 52 + it('finds overlapping occurrences', () => { 53 53 + const s = new SearchState(); 54 54 + const matches = s.findMatches('aaa', 'aa'); 55 55 + expect(matches.length).toBe(2); 56 56 + expect(matches[0]).toEqual({ from: 0, to: 2 }); 57 57 + expect(matches[1]).toEqual({ from: 1, to: 3 }); 58 58 + }); 59 59 + 60 60 + it('returns empty for no match', () => { 61 61 + const s = new SearchState(); 62 62 + const matches = s.findMatches('hello', 'xyz'); 63 63 + expect(matches).toEqual([]); 64 64 + expect(s.currentIndex).toBe(-1); 65 65 + }); 66 66 + 67 67 + it('returns empty for empty term', () => { 68 68 + const s = new SearchState(); 69 69 + expect(s.findMatches('hello', '')).toEqual([]); 70 70 + }); 71 71 + 72 72 + it('case-insensitive by default', () => { 73 73 + const s = new SearchState(); 74 74 + const matches = s.findMatches('Hello HELLO hello', 'hello'); 75 75 + expect(matches.length).toBe(3); 76 76 + }); 77 77 + 78 78 + it('case-sensitive when enabled', () => { 79 79 + const s = new SearchState({ caseSensitive: true }); 80 80 + const matches = s.findMatches('Hello HELLO hello', 'hello'); 81 81 + expect(matches.length).toBe(1); 82 82 + expect(matches[0]).toEqual({ from: 12, to: 17 }); 83 83 + }); 84 84 + 85 85 + it('resets previous matches on new search', () => { 86 86 + const s = new SearchState(); 87 87 + s.findMatches('aaa', 'a'); 88 88 + expect(s.matchCount).toBe(3); 89 89 + s.findMatches('bb', 'b'); 90 90 + expect(s.matchCount).toBe(2); 91 91 + }); 92 92 + }); 93 93 + 94 94 + // ===================================================================== 95 95 + // Navigation — next / prev 96 96 + // ===================================================================== 97 97 + 98 98 + describe('SearchState — next / prev', () => { 99 99 + it('next advances currentIndex', () => { 100 100 + const s = new SearchState(); 101 101 + s.findMatches('abab', 'ab'); 102 102 + expect(s.currentIndex).toBe(0); 103 103 + s.next(); 104 104 + expect(s.currentIndex).toBe(1); 105 105 + }); 106 106 + 107 107 + it('next wraps around', () => { 108 108 + const s = new SearchState(); 109 109 + s.findMatches('abab', 'ab'); 110 110 + s.next(); // 1 111 111 + s.next(); // wraps to 0 112 112 + expect(s.currentIndex).toBe(0); 113 113 + }); 114 114 + 115 115 + it('prev goes backwards', () => { 116 116 + const s = new SearchState(); 117 117 + s.findMatches('ababab', 'ab'); 118 118 + expect(s.currentIndex).toBe(0); 119 119 + s.prev(); // wraps to last 120 120 + expect(s.currentIndex).toBe(2); 121 121 + }); 122 122 + 123 123 + it('prev wraps around from 0', () => { 124 124 + const s = new SearchState(); 125 125 + s.findMatches('abab', 'ab'); 126 126 + s.prev(); // wraps to 1 127 127 + expect(s.currentIndex).toBe(1); 128 128 + }); 129 129 + 130 130 + it('next does nothing with no matches', () => { 131 131 + const s = new SearchState(); 132 132 + s.findMatches('hello', 'xyz'); 133 133 + s.next(); 134 134 + expect(s.currentIndex).toBe(-1); 135 135 + }); 136 136 + 137 137 + it('prev does nothing with no matches', () => { 138 138 + const s = new SearchState(); 139 139 + s.findMatches('hello', 'xyz'); 140 140 + s.prev(); 141 141 + expect(s.currentIndex).toBe(-1); 142 142 + }); 143 143 + }); 144 144 + 145 145 + // ===================================================================== 146 146 + // currentMatch 147 147 + // ===================================================================== 148 148 + 149 149 + describe('SearchState — currentMatch', () => { 150 150 + it('returns the current match object', () => { 151 151 + const s = new SearchState(); 152 152 + s.findMatches('hello world', 'world'); 153 153 + expect(s.currentMatch).toEqual({ from: 6, to: 11 }); 154 154 + }); 155 155 + 156 156 + it('returns null when no matches', () => { 157 157 + const s = new SearchState(); 158 158 + expect(s.currentMatch).toBeNull(); 159 159 + }); 160 160 + 161 161 + it('updates after next()', () => { 162 162 + const s = new SearchState(); 163 163 + s.findMatches('abab', 'ab'); 164 164 + s.next(); 165 165 + expect(s.currentMatch).toEqual({ from: 2, to: 4 }); 166 166 + }); 167 167 + }); 168 168 + 169 169 + // ===================================================================== 170 170 + // replaceOne 171 171 + // ===================================================================== 172 172 + 173 173 + describe('SearchState — replaceOne', () => { 174 174 + it('replaces the current match', () => { 175 175 + const s = new SearchState(); 176 176 + s.findMatches('hello world', 'world'); 177 177 + const result = s.replaceOne('hello world', 'earth'); 178 178 + expect(result.text).toBe('hello earth'); 179 179 + expect(result.replaced).toBe(true); 180 180 + }); 181 181 + 182 182 + it('returns unchanged text when no matches', () => { 183 183 + const s = new SearchState(); 184 184 + s.findMatches('hello', 'xyz'); 185 185 + const result = s.replaceOne('hello', 'abc'); 186 186 + expect(result.text).toBe('hello'); 187 187 + expect(result.replaced).toBe(false); 188 188 + }); 189 189 + 190 190 + it('replaces with empty string', () => { 191 191 + const s = new SearchState(); 192 192 + s.findMatches('hello world', 'world'); 193 193 + const result = s.replaceOne('hello world', ''); 194 194 + expect(result.text).toBe('hello '); 195 195 + }); 196 196 + 197 197 + it('replaces with longer string', () => { 198 198 + const s = new SearchState(); 199 199 + s.findMatches('ab', 'ab'); 200 200 + const result = s.replaceOne('ab', 'abcdef'); 201 201 + expect(result.text).toBe('abcdef'); 202 202 + }); 203 203 + }); 204 204 + 205 205 + // ===================================================================== 206 206 + // replaceAll 207 207 + // ===================================================================== 208 208 + 209 209 + describe('SearchState — replaceAll', () => { 210 210 + it('replaces all occurrences', () => { 211 211 + const s = new SearchState(); 212 212 + s.findMatches('aXbXc', 'X'); 213 213 + const result = s.replaceAll('aXbXc', 'Y'); 214 214 + expect(result.text).toBe('aYbYc'); 215 215 + expect(result.count).toBe(2); 216 216 + }); 217 217 + 218 218 + it('returns original text when no matches', () => { 219 219 + const s = new SearchState(); 220 220 + s.findMatches('hello', 'xyz'); 221 221 + const result = s.replaceAll('hello', 'abc'); 222 222 + expect(result.text).toBe('hello'); 223 223 + expect(result.count).toBe(0); 224 224 + }); 225 225 + 226 226 + it('replaces all with empty string', () => { 227 227 + const s = new SearchState(); 228 228 + s.findMatches('a-b-c', '-'); 229 229 + const result = s.replaceAll('a-b-c', ''); 230 230 + expect(result.text).toBe('abc'); 231 231 + expect(result.count).toBe(2); 232 232 + }); 233 233 + 234 234 + it('replaces all with longer string', () => { 235 235 + const s = new SearchState(); 236 236 + s.findMatches('ab', 'a'); 237 237 + const result = s.replaceAll('ab', 'xyz'); 238 238 + expect(result.text).toBe('xyzb'); 239 239 + expect(result.count).toBe(1); 240 240 + }); 241 241 + 242 242 + it('handles adjacent matches correctly', () => { 243 243 + const s = new SearchState(); 244 244 + s.findMatches('aaa', 'a'); 245 245 + const result = s.replaceAll('aaa', 'bb'); 246 246 + expect(result.text).toBe('bbbbbb'); 247 247 + expect(result.count).toBe(3); 248 248 + }); 249 249 + }); 250 250 + 251 251 + // ===================================================================== 252 252 + // setCaseSensitive 253 253 + // ===================================================================== 254 254 + 255 255 + describe('SearchState — setCaseSensitive', () => { 256 256 + it('toggles case sensitivity', () => { 257 257 + const s = new SearchState(); 258 258 + expect(s.caseSensitive).toBe(false); 259 259 + s.setCaseSensitive(true); 260 260 + expect(s.caseSensitive).toBe(true); 261 261 + s.setCaseSensitive(false); 262 262 + expect(s.caseSensitive).toBe(false); 263 263 + }); 264 264 + 265 265 + it('affects subsequent findMatches calls', () => { 266 266 + const s = new SearchState(); 267 267 + s.findMatches('Hello hello', 'hello'); 268 268 + expect(s.matchCount).toBe(2); 269 269 + 270 270 + s.setCaseSensitive(true); 271 271 + s.findMatches('Hello hello', 'hello'); 272 272 + expect(s.matchCount).toBe(1); 273 273 + }); 274 274 + });

+342

tests/suggesting.test.ts

reviewed

··· 1 1 + /** 2 2 + * Tests for the suggesting/track-changes module (src/lib/suggesting.ts). 3 3 + * Covers SuggestionSession, SuggestionManager, createSuggestionAttrs, and constants. 4 4 + */ 5 5 + import { describe, it, expect } from 'vitest'; 6 6 + import { 7 7 + SUGGESTION_TYPES, 8 8 + SESSION_TIMEOUT_MS, 9 9 + createSuggestionAttrs, 10 10 + SuggestionSession, 11 11 + SuggestionManager, 12 12 + } from '../src/lib/suggesting.js'; 13 13 + 14 14 + // ===================================================================== 15 15 + // Constants 16 16 + // ===================================================================== 17 17 + 18 18 + describe('suggesting constants', () => { 19 19 + it('SUGGESTION_TYPES has INSERT and DELETE', () => { 20 20 + expect(SUGGESTION_TYPES.INSERT).toBe('suggestion-insert'); 21 21 + expect(SUGGESTION_TYPES.DELETE).toBe('suggestion-delete'); 22 22 + }); 23 23 + 24 24 + it('SESSION_TIMEOUT_MS is 2000', () => { 25 25 + expect(SESSION_TIMEOUT_MS).toBe(2000); 26 26 + }); 27 27 + }); 28 28 + 29 29 + // ===================================================================== 30 30 + // createSuggestionAttrs 31 31 + // ===================================================================== 32 32 + 33 33 + describe('createSuggestionAttrs', () => { 34 34 + it('creates attrs with generated ID', () => { 35 35 + const attrs = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 36 36 + expect(attrs.suggestionId).toBeTruthy(); 37 37 + expect(attrs.author).toBe('Alice'); 38 38 + expect(attrs.type).toBe('insert'); 39 39 + expect(attrs.timestamp).toBeTruthy(); 40 40 + }); 41 41 + 42 42 + it('generates unique IDs', () => { 43 43 + const a = createSuggestionAttrs({ type: 'insert', author: 'A' }); 44 44 + const b = createSuggestionAttrs({ type: 'insert', author: 'A' }); 45 45 + expect(a.suggestionId).not.toBe(b.suggestionId); 46 46 + }); 47 47 + 48 48 + it('uses provided timestamp', () => { 49 49 + const attrs = createSuggestionAttrs({ type: 'delete', author: 'Bob', timestamp: '2026-01-01T00:00:00Z' }); 50 50 + expect(attrs.timestamp).toBe('2026-01-01T00:00:00Z'); 51 51 + }); 52 52 + 53 53 + it('handles delete type', () => { 54 54 + const attrs = createSuggestionAttrs({ type: 'delete', author: 'Carol' }); 55 55 + expect(attrs.type).toBe('delete'); 56 56 + }); 57 57 + }); 58 58 + 59 59 + // ===================================================================== 60 60 + // SuggestionSession 61 61 + // ===================================================================== 62 62 + 63 63 + describe('SuggestionSession', () => { 64 64 + function createSession(timeMs = 0) { 65 65 + let clock = timeMs; 66 66 + return { 67 67 + session: new SuggestionSession({ timeoutMs: 2000, now: () => clock }), 68 68 + advance(ms: number) { clock += ms; }, 69 69 + }; 70 70 + } 71 71 + 72 72 + it('starts with no active session', () => { 73 73 + const { session } = createSession(); 74 74 + expect(session.currentId).toBeNull(); 75 75 + }); 76 76 + 77 77 + it('creates a new session on first getAttrs', () => { 78 78 + const { session } = createSession(); 79 79 + const attrs = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 80 80 + expect(attrs.suggestionId).toBeTruthy(); 81 81 + expect(session.currentId).toBe(attrs.suggestionId); 82 82 + }); 83 83 + 84 84 + it('reuses session ID for consecutive edits by same author/type/position', () => { 85 85 + const { session } = createSession(); 86 86 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 87 87 + session.updateCursor(6); 88 88 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 6 }); 89 89 + expect(a.suggestionId).toBe(b.suggestionId); 90 90 + }); 91 91 + 92 92 + it('starts new session when author changes', () => { 93 93 + const { session } = createSession(); 94 94 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 95 95 + const b = session.getAttrs({ type: 'insert', author: 'Bob', cursorPos: 6 }); 96 96 + expect(a.suggestionId).not.toBe(b.suggestionId); 97 97 + }); 98 98 + 99 99 + it('starts new session when type changes', () => { 100 100 + const { session } = createSession(); 101 101 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 102 102 + const b = session.getAttrs({ type: 'delete', author: 'Alice', cursorPos: 5 }); 103 103 + expect(a.suggestionId).not.toBe(b.suggestionId); 104 104 + }); 105 105 + 106 106 + it('starts new session after timeout', () => { 107 107 + const { session, advance } = createSession(); 108 108 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 109 109 + advance(2001); 110 110 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 6 }); 111 111 + expect(a.suggestionId).not.toBe(b.suggestionId); 112 112 + }); 113 113 + 114 114 + it('reuses session within timeout', () => { 115 115 + const { session, advance } = createSession(); 116 116 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 117 117 + session.updateCursor(6); 118 118 + advance(1999); 119 119 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 6 }); 120 120 + expect(a.suggestionId).toBe(b.suggestionId); 121 121 + }); 122 122 + 123 123 + it('starts new session on cursor jump (>1 position)', () => { 124 124 + const { session } = createSession(); 125 125 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 126 126 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 10 }); 127 127 + expect(a.suggestionId).not.toBe(b.suggestionId); 128 128 + }); 129 129 + 130 130 + it('allows adjacent cursor (exactly 1 position away)', () => { 131 131 + const { session } = createSession(); 132 132 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 133 133 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 6 }); 134 134 + expect(a.suggestionId).toBe(b.suggestionId); 135 135 + }); 136 136 + 137 137 + it('handles null cursor position', () => { 138 138 + const { session } = createSession(); 139 139 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: null }); 140 140 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: null }); 141 141 + expect(a.suggestionId).toBe(b.suggestionId); 142 142 + }); 143 143 + 144 144 + it('resetSession clears the active session', () => { 145 145 + const { session } = createSession(); 146 146 + session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 147 147 + expect(session.currentId).not.toBeNull(); 148 148 + session.resetSession(); 149 149 + expect(session.currentId).toBeNull(); 150 150 + }); 151 151 + 152 152 + it('new session starts after resetSession', () => { 153 153 + const { session } = createSession(); 154 154 + const a = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 155 155 + session.resetSession(); 156 156 + const b = session.getAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 157 157 + expect(a.suggestionId).not.toBe(b.suggestionId); 158 158 + }); 159 159 + }); 160 160 + 161 161 + // ===================================================================== 162 162 + // SuggestionManager 163 163 + // ===================================================================== 164 164 + 165 165 + describe('SuggestionManager', () => { 166 166 + it('starts in editing mode', () => { 167 167 + const mgr = new SuggestionManager(); 168 168 + expect(mgr.isSuggesting()).toBe(false); 169 169 + }); 170 170 + 171 171 + it('setMode to suggesting', () => { 172 172 + const mgr = new SuggestionManager(); 173 173 + mgr.setMode('suggesting'); 174 174 + expect(mgr.isSuggesting()).toBe(true); 175 175 + }); 176 176 + 177 177 + it('setMode back to editing', () => { 178 178 + const mgr = new SuggestionManager(); 179 179 + mgr.setMode('suggesting'); 180 180 + mgr.setMode('editing'); 181 181 + expect(mgr.isSuggesting()).toBe(false); 182 182 + }); 183 183 + 184 184 + it('toggleMode switches between editing and suggesting', () => { 185 185 + const mgr = new SuggestionManager(); 186 186 + mgr.toggleMode(); 187 187 + expect(mgr.isSuggesting()).toBe(true); 188 188 + mgr.toggleMode(); 189 189 + expect(mgr.isSuggesting()).toBe(false); 190 190 + }); 191 191 + 192 192 + it('addSuggestion and getSuggestion', () => { 193 193 + const mgr = new SuggestionManager(); 194 194 + const attrs = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 195 195 + mgr.addSuggestion(attrs); 196 196 + expect(mgr.getSuggestion(attrs.suggestionId)).toEqual(attrs); 197 197 + }); 198 198 + 199 199 + it('getSuggestion returns null for unknown ID', () => { 200 200 + const mgr = new SuggestionManager(); 201 201 + expect(mgr.getSuggestion('nonexistent')).toBeNull(); 202 202 + }); 203 203 + 204 204 + it('getSuggestions returns all tracked', () => { 205 205 + const mgr = new SuggestionManager(); 206 206 + const a = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 207 207 + const b = createSuggestionAttrs({ type: 'delete', author: 'Bob' }); 208 208 + mgr.addSuggestion(a); 209 209 + mgr.addSuggestion(b); 210 210 + expect(mgr.getSuggestions().length).toBe(2); 211 211 + }); 212 212 + 213 213 + it('getSuggestionsByAuthor filters correctly', () => { 214 214 + const mgr = new SuggestionManager(); 215 215 + mgr.addSuggestion(createSuggestionAttrs({ type: 'insert', author: 'Alice' })); 216 216 + mgr.addSuggestion(createSuggestionAttrs({ type: 'insert', author: 'Bob' })); 217 217 + mgr.addSuggestion(createSuggestionAttrs({ type: 'delete', author: 'Alice' })); 218 218 + expect(mgr.getSuggestionsByAuthor('Alice').length).toBe(2); 219 219 + expect(mgr.getSuggestionsByAuthor('Bob').length).toBe(1); 220 220 + }); 221 221 + 222 222 + describe('accept', () => { 223 223 + it('returns null for unknown ID', () => { 224 224 + const mgr = new SuggestionManager(); 225 225 + expect(mgr.accept('unknown')).toBeNull(); 226 226 + }); 227 227 + 228 228 + it('accept INSERT → remove-mark', () => { 229 229 + const mgr = new SuggestionManager(); 230 230 + const attrs = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 231 231 + mgr.addSuggestion(attrs); 232 232 + const action = mgr.accept(attrs.suggestionId); 233 233 + expect(action).toEqual({ action: 'remove-mark', type: 'insert', suggestionId: attrs.suggestionId }); 234 234 + }); 235 235 + 236 236 + it('accept DELETE → delete-text', () => { 237 237 + const mgr = new SuggestionManager(); 238 238 + const attrs = createSuggestionAttrs({ type: 'delete', author: 'Alice' }); 239 239 + mgr.addSuggestion(attrs); 240 240 + const action = mgr.accept(attrs.suggestionId); 241 241 + expect(action).toEqual({ action: 'delete-text', type: 'delete', suggestionId: attrs.suggestionId }); 242 242 + }); 243 243 + 244 244 + it('removes suggestion after accept', () => { 245 245 + const mgr = new SuggestionManager(); 246 246 + const attrs = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 247 247 + mgr.addSuggestion(attrs); 248 248 + mgr.accept(attrs.suggestionId); 249 249 + expect(mgr.getSuggestion(attrs.suggestionId)).toBeNull(); 250 250 + }); 251 251 + }); 252 252 + 253 253 + describe('reject', () => { 254 254 + it('returns null for unknown ID', () => { 255 255 + const mgr = new SuggestionManager(); 256 256 + expect(mgr.reject('unknown')).toBeNull(); 257 257 + }); 258 258 + 259 259 + it('reject INSERT → delete-text', () => { 260 260 + const mgr = new SuggestionManager(); 261 261 + const attrs = createSuggestionAttrs({ type: 'insert', author: 'Alice' }); 262 262 + mgr.addSuggestion(attrs); 263 263 + const action = mgr.reject(attrs.suggestionId); 264 264 + expect(action).toEqual({ action: 'delete-text', type: 'insert', suggestionId: attrs.suggestionId }); 265 265 + }); 266 266 + 267 267 + it('reject DELETE → remove-mark', () => { 268 268 + const mgr = new SuggestionManager(); 269 269 + const attrs = createSuggestionAttrs({ type: 'delete', author: 'Alice' }); 270 270 + mgr.addSuggestion(attrs); 271 271 + const action = mgr.reject(attrs.suggestionId); 272 272 + expect(action).toEqual({ action: 'remove-mark', type: 'delete', suggestionId: attrs.suggestionId }); 273 273 + }); 274 274 + 275 275 + it('removes suggestion after reject', () => { 276 276 + const mgr = new SuggestionManager(); 277 277 + const attrs = createSuggestionAttrs({ type: 'delete', author: 'A' }); 278 278 + mgr.addSuggestion(attrs); 279 279 + mgr.reject(attrs.suggestionId); 280 280 + expect(mgr.getSuggestion(attrs.suggestionId)).toBeNull(); 281 281 + }); 282 282 + }); 283 283 + 284 284 + describe('acceptAll / rejectAll', () => { 285 285 + it('acceptAll returns actions for all suggestions', () => { 286 286 + const mgr = new SuggestionManager(); 287 287 + mgr.addSuggestion(createSuggestionAttrs({ type: 'insert', author: 'A' })); 288 288 + mgr.addSuggestion(createSuggestionAttrs({ type: 'delete', author: 'B' })); 289 289 + const actions = mgr.acceptAll(); 290 290 + expect(actions.length).toBe(2); 291 291 + expect(mgr.getSuggestions().length).toBe(0); 292 292 + }); 293 293 + 294 294 + it('rejectAll returns actions for all suggestions', () => { 295 295 + const mgr = new SuggestionManager(); 296 296 + mgr.addSuggestion(createSuggestionAttrs({ type: 'insert', author: 'A' })); 297 297 + mgr.addSuggestion(createSuggestionAttrs({ type: 'delete', author: 'B' })); 298 298 + const actions = mgr.rejectAll(); 299 299 + expect(actions.length).toBe(2); 300 300 + expect(mgr.getSuggestions().length).toBe(0); 301 301 + }); 302 302 + 303 303 + it('acceptAll on empty returns empty array', () => { 304 304 + const mgr = new SuggestionManager(); 305 305 + expect(mgr.acceptAll()).toEqual([]); 306 306 + }); 307 307 + 308 308 + it('rejectAll on empty returns empty array', () => { 309 309 + const mgr = new SuggestionManager(); 310 310 + expect(mgr.rejectAll()).toEqual([]); 311 311 + }); 312 312 + }); 313 313 + 314 314 + describe('session integration', () => { 315 315 + it('getSessionAttrs returns attrs with session grouping', () => { 316 316 + let clock = 0; 317 317 + const mgr = new SuggestionManager({ now: () => clock }); 318 318 + const a = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 319 319 + mgr.updateSessionCursor(6); 320 320 + const b = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 6 }); 321 321 + expect(a.suggestionId).toBe(b.suggestionId); 322 322 + }); 323 323 + 324 324 + it('resetSession forces new session', () => { 325 325 + const mgr = new SuggestionManager(); 326 326 + const a = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 327 327 + mgr.resetSession(); 328 328 + const b = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 329 329 + expect(a.suggestionId).not.toBe(b.suggestionId); 330 330 + }); 331 331 + 332 332 + it('switching to editing mode resets session', () => { 333 333 + const mgr = new SuggestionManager(); 334 334 + mgr.setMode('suggesting'); 335 335 + const a = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 336 336 + mgr.setMode('editing'); 337 337 + mgr.setMode('suggesting'); 338 338 + const b = mgr.getSessionAttrs({ type: 'insert', author: 'Alice', cursorPos: 5 }); 339 339 + expect(a.suggestionId).not.toBe(b.suggestionId); 340 340 + }); 341 341 + }); 342 342 + });

+15

tests/tab-handler.test.ts

reviewed

··· 1 1 + /** 2 2 + * Tests for sheet tab key handler (src/sheets/tab-handler.ts). 3 3 + */ 4 4 + import { describe, it, expect } from 'vitest'; 5 5 + import { resolveSheetTabAction } from '../src/sheets/tab-handler.js'; 6 6 + 7 7 + describe('resolveSheetTabAction', () => { 8 8 + it('returns moveRight for Tab (no shift)', () => { 9 9 + expect(resolveSheetTabAction({ shiftKey: false })).toBe('moveRight'); 10 10 + }); 11 11 + 12 12 + it('returns moveLeft for Shift+Tab', () => { 13 13 + expect(resolveSheetTabAction({ shiftKey: true })).toBe('moveLeft'); 14 14 + }); 15 15 + });