@@ -139,30 +139,24 @@ describe("BrowserOrchestrator", () => {
139139
140140 describe ( "Search Query" , ( ) => {
141141 it ( "should set search query" , async ( ) => {
142-
143-
144142 await orchestrator . setSearchQuery ( "test query" )
145143
146144 expect ( mockCommands . browserSetSearchQuery ) . toHaveBeenCalledWith ( "test query" , null )
147145 } )
148146
149147 it ( "should set search query for specific tab" , async ( ) => {
150-
151-
152148 await orchestrator . setSearchQuery ( "effects query" , "effects" )
153149
154150 expect ( mockCommands . browserSetSearchQuery ) . toHaveBeenCalledWith ( "effects query" , "effects" )
155151 } )
156152
157153 it ( "should handle search query error" , async ( ) => {
158-
159154 mockCommands . browserSetSearchQuery = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Search failed" } )
160155
161156 await expect ( orchestrator . setSearchQuery ( "test" ) ) . rejects . toThrow ( "Search failed" )
162157 } )
163158
164159 it ( "should clear error after successful search" , async ( ) => {
165-
166160 // Reset mock to return success
167161 mockCommands . browserSetSearchQuery = vi . fn ( ) . mockResolvedValue ( { status : "ok" } )
168162
@@ -174,23 +168,18 @@ describe("BrowserOrchestrator", () => {
174168
175169 describe ( "Favorites Toggle" , ( ) => {
176170 it ( "should toggle favorites" , async ( ) => {
177-
178-
179171 await orchestrator . toggleFavorites ( )
180172
181173 expect ( mockCommands . browserToggleFavorites ) . toHaveBeenCalledWith ( null )
182174 } )
183175
184176 it ( "should toggle favorites for specific tab" , async ( ) => {
185-
186-
187177 await orchestrator . toggleFavorites ( "filters" )
188178
189179 expect ( mockCommands . browserToggleFavorites ) . toHaveBeenCalledWith ( "filters" )
190180 } )
191181
192182 it ( "should handle toggle favorites error" , async ( ) => {
193-
194183 mockCommands . browserToggleFavorites = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Toggle failed" } )
195184
196185 await expect ( orchestrator . toggleFavorites ( ) ) . rejects . toThrow ( "Toggle failed" )
@@ -199,23 +188,18 @@ describe("BrowserOrchestrator", () => {
199188
200189 describe ( "Sort Settings" , ( ) => {
201190 it ( "should set sort" , async ( ) => {
202-
203-
204191 await orchestrator . setSort ( "date" , "desc" )
205192
206193 expect ( mockCommands . browserSetSort ) . toHaveBeenCalledWith ( "date" , "desc" , null )
207194 } )
208195
209196 it ( "should set sort for specific tab" , async ( ) => {
210-
211-
212197 await orchestrator . setSort ( "name" , "asc" , "effects" )
213198
214199 expect ( mockCommands . browserSetSort ) . toHaveBeenCalledWith ( "name" , "asc" , "effects" )
215200 } )
216201
217202 it ( "should handle sort error" , async ( ) => {
218-
219203 mockCommands . browserSetSort = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Sort failed" } )
220204
221205 await expect ( orchestrator . setSort ( "date" , "asc" ) ) . rejects . toThrow ( "Sort failed" )
@@ -224,16 +208,12 @@ describe("BrowserOrchestrator", () => {
224208
225209 describe ( "Group By Settings" , ( ) => {
226210 it ( "should set group by" , async ( ) => {
227-
228-
229211 await orchestrator . setGroupBy ( "type" )
230212
231213 expect ( mockCommands . browserSetGroupBy ) . toHaveBeenCalledWith ( "type" , null )
232214 } )
233215
234216 it ( "should set group by for specific tab" , async ( ) => {
235-
236-
237217 await orchestrator . setGroupBy ( "category" , "templates" )
238218
239219 expect ( mockCommands . browserSetGroupBy ) . toHaveBeenCalledWith ( "category" , "templates" )
@@ -242,16 +222,12 @@ describe("BrowserOrchestrator", () => {
242222
243223 describe ( "Filter Settings" , ( ) => {
244224 it ( "should set filter" , async ( ) => {
245-
246-
247225 await orchestrator . setFilter ( "video" )
248226
249227 expect ( mockCommands . browserSetFilter ) . toHaveBeenCalledWith ( "video" , null )
250228 } )
251229
252230 it ( "should set filter for specific tab" , async ( ) => {
253-
254-
255231 await orchestrator . setFilter ( "image" , "media" )
256232
257233 expect ( mockCommands . browserSetFilter ) . toHaveBeenCalledWith ( "image" , "media" )
@@ -260,23 +236,18 @@ describe("BrowserOrchestrator", () => {
260236
261237 describe ( "View Mode Settings" , ( ) => {
262238 it ( "should set view mode" , async ( ) => {
263-
264-
265239 await orchestrator . setViewMode ( "grid" )
266240
267241 expect ( mockCommands . browserSetViewMode ) . toHaveBeenCalledWith ( "grid" , null )
268242 } )
269243
270244 it ( "should set view mode for specific tab" , async ( ) => {
271-
272-
273245 await orchestrator . setViewMode ( "list" , "effects" )
274246
275247 expect ( mockCommands . browserSetViewMode ) . toHaveBeenCalledWith ( "list" , "effects" )
276248 } )
277249
278250 it ( "should handle all view modes" , async ( ) => {
279-
280251 const viewModes : Array < "thumbnails" | "list" | "grid" > = [ "thumbnails" , "list" , "grid" ]
281252
282253 for ( const viewMode of viewModes ) {
@@ -288,23 +259,18 @@ describe("BrowserOrchestrator", () => {
288259
289260 describe ( "Preview Size Settings" , ( ) => {
290261 it ( "should set preview size" , async ( ) => {
291-
292-
293262 await orchestrator . setPreviewSize ( 3 )
294263
295264 expect ( mockCommands . browserSetPreviewSize ) . toHaveBeenCalledWith ( 3 , null )
296265 } )
297266
298267 it ( "should set preview size for specific tab" , async ( ) => {
299-
300-
301268 await orchestrator . setPreviewSize ( 4 , "filters" )
302269
303270 expect ( mockCommands . browserSetPreviewSize ) . toHaveBeenCalledWith ( 4 , "filters" )
304271 } )
305272
306273 it ( "should handle all size indices" , async ( ) => {
307-
308274 const sizeIndices = [ 0 , 1 , 2 , 3 , 4 ]
309275
310276 for ( const sizeIndex of sizeIndices ) {
@@ -316,15 +282,12 @@ describe("BrowserOrchestrator", () => {
316282
317283 describe ( "Reset Tab Settings" , ( ) => {
318284 it ( "should reset tab settings" , async ( ) => {
319-
320-
321285 await orchestrator . resetTabSettings ( "media" )
322286
323287 expect ( mockCommands . browserResetTabSettings ) . toHaveBeenCalledWith ( "media" )
324288 } )
325289
326290 it ( "should handle reset error" , async ( ) => {
327-
328291 mockCommands . browserResetTabSettings = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Reset failed" } )
329292
330293 await expect ( orchestrator . resetTabSettings ( "media" ) ) . rejects . toThrow ( "Reset failed" )
@@ -333,39 +296,30 @@ describe("BrowserOrchestrator", () => {
333296
334297 describe ( "File Selection" , ( ) => {
335298 it ( "should select file" , async ( ) => {
336-
337-
338299 await orchestrator . selectFile ( "file-1" )
339300
340301 expect ( mockCommands . browserSelectFile ) . toHaveBeenCalledWith ( "file-1" , null )
341302 } )
342303
343304 it ( "should select file for specific tab" , async ( ) => {
344-
345-
346305 await orchestrator . selectFile ( "effect-1" , "effects" )
347306
348307 expect ( mockCommands . browserSelectFile ) . toHaveBeenCalledWith ( "effect-1" , "effects" )
349308 } )
350309
351310 it ( "should deselect file" , async ( ) => {
352-
353-
354311 await orchestrator . deselectFile ( "file-1" )
355312
356313 expect ( mockCommands . browserDeselectFile ) . toHaveBeenCalledWith ( "file-1" , null )
357314 } )
358315
359316 it ( "should deselect file for specific tab" , async ( ) => {
360-
361-
362317 await orchestrator . deselectFile ( "effect-1" , "effects" )
363318
364319 expect ( mockCommands . browserDeselectFile ) . toHaveBeenCalledWith ( "effect-1" , "effects" )
365320 } )
366321
367322 it ( "should handle file selection error" , async ( ) => {
368-
369323 mockCommands . browserSelectFile = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Select failed" } )
370324
371325 await expect ( orchestrator . selectFile ( "file-1" ) ) . rejects . toThrow ( "Select failed" )
@@ -374,7 +328,6 @@ describe("BrowserOrchestrator", () => {
374328
375329 describe ( "Toggle File Selection" , ( ) => {
376330 it ( "should toggle file selection - select when not selected" , async ( ) => {
377-
378331 // Reset mocks to return success
379332 mockCommands . browserSelectFile = vi . fn ( ) . mockResolvedValue ( { status : "ok" } )
380333
@@ -385,7 +338,6 @@ describe("BrowserOrchestrator", () => {
385338 } )
386339
387340 it ( "should toggle file selection - deselect when selected" , async ( ) => {
388-
389341 // Reset mocks to return success
390342 mockCommands . browserSelectFile = vi . fn ( ) . mockResolvedValue ( { status : "ok" } )
391343 mockCommands . browserDeselectFile = vi . fn ( ) . mockResolvedValue ( { status : "ok" } )
@@ -412,7 +364,6 @@ describe("BrowserOrchestrator", () => {
412364
413365 describe ( "Select All Files" , ( ) => {
414366 it ( "should select all files" , async ( ) => {
415-
416367 const fileIds = [ "file-1" , "file-2" , "file-3" ]
417368
418369 await orchestrator . selectAllFiles ( fileIds )
@@ -421,7 +372,6 @@ describe("BrowserOrchestrator", () => {
421372 } )
422373
423374 it ( "should select all files for specific tab" , async ( ) => {
424-
425375 const fileIds = [ "effect-1" , "effect-2" ]
426376
427377 await orchestrator . selectAllFiles ( fileIds , "effects" )
@@ -432,16 +382,12 @@ describe("BrowserOrchestrator", () => {
432382
433383 describe ( "Deselect All Files" , ( ) => {
434384 it ( "should deselect all files" , async ( ) => {
435-
436-
437385 await orchestrator . deselectAllFiles ( )
438386
439387 expect ( mockCommands . browserDeselectAllFiles ) . toHaveBeenCalledWith ( null )
440388 } )
441389
442390 it ( "should deselect all files for specific tab" , async ( ) => {
443-
444-
445391 await orchestrator . deselectAllFiles ( "effects" )
446392
447393 expect ( mockCommands . browserDeselectAllFiles ) . toHaveBeenCalledWith ( "effects" )
@@ -500,7 +446,6 @@ describe("BrowserOrchestrator", () => {
500446 } )
501447
502448 it ( "should notify subscribers on state change" , async ( ) => {
503-
504449 mockCommands . browserSwitchTab = vi . fn ( ) . mockResolvedValue ( { status : "ok" } )
505450
506451 const callback = vi . fn ( )
@@ -554,15 +499,12 @@ describe("BrowserOrchestrator", () => {
554499
555500 describe ( "Error Handling" , ( ) => {
556501 it ( "should set error state on command failure" , async ( ) => {
557-
558502 mockCommands . browserSwitchTab = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Test error" } )
559503
560504 await expect ( orchestrator . switchTab ( "effects" ) ) . rejects . toThrow ( "Test error" )
561505 } )
562506
563507 it ( "should clear error after successful operation" , async ( ) => {
564-
565-
566508 // Cause an error
567509 mockCommands . browserSetSearchQuery = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Test error" } )
568510 try {
@@ -581,8 +523,6 @@ describe("BrowserOrchestrator", () => {
581523
582524 describe ( "Loading State" , ( ) => {
583525 it ( "should set loading state during operations" , async ( ) => {
584-
585-
586526 // Mock slow command
587527 mockCommands . browserSetSearchQuery = vi
588528 . fn ( )
@@ -600,7 +540,6 @@ describe("BrowserOrchestrator", () => {
600540 } )
601541
602542 it ( "should clear loading state on error" , async ( ) => {
603-
604543 mockCommands . browserSetSearchQuery = vi . fn ( ) . mockResolvedValue ( { status : "error" , error : "Test error" } )
605544
606545 try {
0 commit comments