test: update existing tests for improved accuracy

- Remove afterEach cleanup logic (unnecessary with Vitest browser mode auto-reset)
- Remove redundant test cases and improve test names in blokr.test.ts
- Change event-blocking.test.ts from mousedown to click event tests
- Reflect core improvements from previous commits

Author: knowledgecode

tests/blokr.test.ts

@@ -1,15 +1,7 @@
-import { describe, it, expect, afterEach, vi } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import blokr from '../src/blokr.ts';
 
 describe('Blokr Factory Function', () => {
-  afterEach(() => {
-    // Clean up any locks after each test
-    const globalInstance = blokr();
-    if (globalInstance.isLocked()) {
-      globalInstance.unlock();
-    }
-  });
-
   describe('Factory Behavior', () => {
     it('should return a Blokr instance', () => {
       const instance = blokr();
@@ -196,18 +188,6 @@ describe('Blokr Factory Function', () => {
       vi.useRealTimers();
     });
 
-    it('should not set timeout when timeout is 0', () => {
-      const instance = blokr();
-      const setTimeoutSpy = vi.spyOn(globalThis, 'setTimeout');
-
-      instance.lock({ timeout: 0 });
-
-      expect(setTimeoutSpy).not.toHaveBeenCalled();
-
-      setTimeoutSpy.mockRestore();
-      instance.unlock();
-    });
-
     it('should clear timeout on manual unlock', () => {
       vi.useFakeTimers();
 
@@ -282,11 +262,12 @@ describe('Blokr Factory Function', () => {
       instance.unlock();
     });
 
-    it('should use default scope "inside" for global instance', () => {
+    it('should block all events for global instance regardless of scope', () => {
       const instance = blokr();
-      // Global instance with no target should still work (no filtering)
-      const result = instance.lock();
+      // Global instance blocks all events, scope parameter is ignored
+      const result = instance.lock({ scope: 'outside' });
       expect(result).toBe(true);
+      expect(instance.isLocked()).toBe(true);
       instance.unlock();
     });
   });

tests/event-blocking.test.ts

@@ -1,26 +1,18 @@
-import { describe, it, expect, afterEach, vi } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import blokr from '../src/blokr.ts';
 
 describe('Event Blocking Integration', () => {
-  afterEach(() => {
-    // Clean up any locks after each test
-    const globalInstance = blokr();
-    if (globalInstance.isLocked()) {
-      globalInstance.unlock();
-    }
-  });
-
   describe('Global Event Blocking', () => {
-    it('should block mousedown events when locked globally', () => {
+    it('should block click events when locked globally', () => {
       const handler = vi.fn();
       const element = document.createElement('button');
-      element.addEventListener('mousedown', handler);
+      element.addEventListener('click', handler);
       document.body.appendChild(element);
 
       const instance = blokr();
       instance.lock();
 
-      const event = new MouseEvent('mousedown', {
+      const event = new MouseEvent('click', {
         bubbles: true,
         cancelable: true
       });