test(spinner): enable withSpinner state restoration tests

Removed .todo() markers from 10 integration tests that verify
withSpinner and withSpinnerSync correctly restore color and
shimmer state. Tests now pass after setShimmer bug fix.

Tests enabled:
- withSpinner: 6 tests (color restore, named colors, shimmer,
  disable shimmer, no options, error handling)
- withSpinnerSync: 4 tests (color restore, shimmer, no options,
  error handling)

All 11 tests now passing. Closes #14

Author: jdalton

test/spinner.test.ts

@@ -17,10 +17,7 @@ describe('spinner', () => {
   })
 
   describe('withSpinner', () => {
-    // Note: These tests require full logger context which isn't available in unit test env
-    // They are marked as todo until proper integration test setup is implemented
-    // Tracked in: https://github.com/SocketDev/socket-lib/issues/14
-    it.todo('should restore color after operation', async () => {
+    it('should restore color after operation', async () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color
 
@@ -40,30 +37,27 @@ describe('spinner', () => {
       expect(spinner.color).toEqual(originalColor)
     })
 
-    it.todo(
-      'should restore color after operation with named color',
-      async () => {
-        const spinner = Spinner({ color: 'cyan' })
-        const originalColor = spinner.color
+    it('should restore color after operation with named color', async () => {
+      const spinner = Spinner({ color: 'cyan' })
+      const originalColor = spinner.color
 
-        await withSpinner({
-          message: 'Testing...',
-          operation: async () => {
-            // Just verify operation runs
-            expect(true).toBe(true)
-          },
-          spinner,
-          withOptions: {
-            color: 'red',
-          },
-        })
+      await withSpinner({
+        message: 'Testing...',
+        operation: async () => {
+          // Just verify operation runs
+          expect(true).toBe(true)
+        },
+        spinner,
+        withOptions: {
+          color: 'red',
+        },
+      })
 
-        // After operation, color should be restored
-        expect(spinner.color).toEqual(originalColor)
-      },
-    )
+      // After operation, color should be restored
+      expect(spinner.color).toEqual(originalColor)
+    })
 
-    it.todo('should restore shimmer state after operation', async () => {
+    it('should restore shimmer state after operation', async () => {
       const spinner = Spinner({ shimmer: { dir: 'ltr', speed: 0.5 } })
       const originalShimmer = spinner.shimmerState
 
@@ -84,29 +78,26 @@ describe('spinner', () => {
       expect(spinner.shimmerState?.speed).toBe(originalShimmer?.speed)
     })
 
-    it.todo(
-      'should disable shimmer after operation if it was disabled before',
-      async () => {
-        const spinner = Spinner() // No shimmer
+    it('should disable shimmer after operation if it was disabled before', async () => {
+      const spinner = Spinner() // No shimmer
 
-        await withSpinner({
-          message: 'Testing...',
-          operation: async () => {
-            // During operation, shimmer should be enabled
-            expect(spinner.shimmerState).toBeDefined()
-          },
-          spinner,
-          withOptions: {
-            shimmer: { dir: 'ltr' },
-          },
-        })
+      await withSpinner({
+        message: 'Testing...',
+        operation: async () => {
+          // During operation, shimmer should be enabled
+          expect(spinner.shimmerState).toBeDefined()
+        },
+        spinner,
+        withOptions: {
+          shimmer: { dir: 'ltr' },
+        },
+      })
 
-        // After operation, shimmer should be disabled again
-        expect(spinner.shimmerState).toBeUndefined()
-      },
-    )
+      // After operation, shimmer should be disabled again
+      expect(spinner.shimmerState).toBeUndefined()
+    })
 
-    it.todo('should work without withOptions', async () => {
+    it('should work without withOptions', async () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color
 
@@ -131,7 +122,7 @@ describe('spinner', () => {
       expect(result).toBe(42)
     })
 
-    it.todo('should restore state even if operation throws', async () => {
+    it('should restore state even if operation throws', async () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color
 
@@ -154,7 +145,7 @@ describe('spinner', () => {
   })
 
   describe('withSpinnerSync', () => {
-    it.todo('should restore color after operation', () => {
+    it('should restore color after operation', () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color
 
@@ -174,7 +165,7 @@ describe('spinner', () => {
       expect(spinner.color).toEqual(originalColor)
     })
 
-    it.todo('should restore shimmer state after operation', () => {
+    it('should restore shimmer state after operation', () => {
       const spinner = Spinner({ shimmer: { dir: 'ltr', speed: 0.5 } })
       const originalShimmer = spinner.shimmerState
 
@@ -195,7 +186,7 @@ describe('spinner', () => {
       expect(spinner.shimmerState?.speed).toBe(originalShimmer?.speed)
     })
 
-    it.todo('should work without withOptions', () => {
+    it('should work without withOptions', () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color
 
@@ -211,7 +202,7 @@ describe('spinner', () => {
       expect(spinner.color).toEqual(originalColor)
     })
 
-    it.todo('should restore state even if operation throws', () => {
+    it('should restore state even if operation throws', () => {
       const spinner = Spinner({ color: [140, 82, 255] })
       const originalColor = spinner.color