zero-query 1.1.1 → 1.2.0

package/tests/http.test.js

@@ -1,648 +1,648 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { http } from '../src/http.js';
-
-
-// ---------------------------------------------------------------------------
-// Mock fetch
-// ---------------------------------------------------------------------------
-let fetchSpy;
-
-function mockFetch(response = {}, ok = true, status = 200) {
-  const body = typeof response === 'string' ? response : JSON.stringify(response);
-  const contentType = typeof response === 'string' ? 'text/plain' : 'application/json';
-
-  fetchSpy = vi.fn(() =>
-    Promise.resolve({
-      ok,
-      status,
-      statusText: ok ? 'OK' : 'Error',
-      headers: {
-        get: (h) => h.toLowerCase() === 'content-type' ? contentType : null,
-        entries: () => [[`content-type`, contentType]],
-      },
-      json: () => Promise.resolve(typeof response === 'object' ? response : JSON.parse(response)),
-      text: () => Promise.resolve(body),
-      blob: () => Promise.resolve(new Blob([body])),
-    })
-  );
-  globalThis.fetch = fetchSpy;
-}
-
-beforeEach(() => {
-  http.configure({ baseURL: '', headers: { 'Content-Type': 'application/json' }, timeout: 30000 });
-});
-
-afterEach(() => {
-  vi.restoreAllMocks();
-});
-
-
-// ---------------------------------------------------------------------------
-// GET requests
-// ---------------------------------------------------------------------------
-
-describe('http.get', () => {
-  it('makes a GET request', async () => {
-    mockFetch({ users: [] });
-    const result = await http.get('https://api.test.com/users');
-    expect(fetchSpy).toHaveBeenCalledOnce();
-    expect(result.data).toEqual({ users: [] });
-    expect(result.ok).toBe(true);
-    expect(result.status).toBe(200);
-  });
-
-  it('appends query params for GET', async () => {
-    mockFetch({});
-    await http.get('https://api.test.com/search', { q: 'hello', page: '1' });
-    const url = fetchSpy.mock.calls[0][0];
-    expect(url).toContain('q=hello');
-    expect(url).toContain('page=1');
-  });
-
-  it('uses baseURL', async () => {
-    http.configure({ baseURL: 'https://api.test.com' });
-    mockFetch({});
-    await http.get('/users');
-    expect(fetchSpy.mock.calls[0][0]).toBe('https://api.test.com/users');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// POST requests
-// ---------------------------------------------------------------------------
-
-describe('http.post', () => {
-  it('sends JSON body', async () => {
-    mockFetch({ id: 1 });
-    const result = await http.post('https://api.test.com/users', { name: 'Tony' });
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.method).toBe('POST');
-    expect(opts.body).toBe(JSON.stringify({ name: 'Tony' }));
-    expect(result.data).toEqual({ id: 1 });
-  });
-
-  it('handles FormData body', async () => {
-    mockFetch({});
-    const form = new FormData();
-    form.append('file', 'data');
-    await http.post('https://api.test.com/upload', form);
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.body).toBe(form);
-    // Content-Type should be removed for FormData
-    expect(opts.headers['Content-Type']).toBeUndefined();
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Error handling
-// ---------------------------------------------------------------------------
-
-describe('http - error handling', () => {
-  it('throws on non-ok response', async () => {
-    mockFetch({ error: 'Not Found' }, false, 404);
-    await expect(http.get('https://api.test.com/missing')).rejects.toThrow('HTTP 404');
-  });
-
-  it('error includes response data', async () => {
-    mockFetch({ message: 'Unauthorized' }, false, 401);
-    try {
-      await http.get('https://api.test.com/protected');
-    } catch (err) {
-      expect(err.response).toBeDefined();
-      expect(err.response.status).toBe(401);
-    }
-  });
-
-  it('throws on invalid URL', async () => {
-    await expect(http.get(null)).rejects.toThrow('requires a URL string');
-    await expect(http.get(undefined)).rejects.toThrow('requires a URL string');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// PUT / PATCH / DELETE
-// ---------------------------------------------------------------------------
-
-describe('http - other methods', () => {
-  it('PUT sends correct method', async () => {
-    mockFetch({});
-    await http.put('https://api.test.com/users/1', { name: 'Updated' });
-    expect(fetchSpy.mock.calls[0][1].method).toBe('PUT');
-  });
-
-  it('PATCH sends correct method', async () => {
-    mockFetch({});
-    await http.patch('https://api.test.com/users/1', { name: 'Patched' });
-    expect(fetchSpy.mock.calls[0][1].method).toBe('PATCH');
-  });
-
-  it('DELETE sends correct method', async () => {
-    mockFetch({});
-    await http.delete('https://api.test.com/users/1');
-    expect(fetchSpy.mock.calls[0][1].method).toBe('DELETE');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// configure
-// ---------------------------------------------------------------------------
-
-describe('http.configure', () => {
-  it('sets baseURL', async () => {
-    http.configure({ baseURL: 'https://example.com/api' });
-    mockFetch({});
-    await http.get('/data');
-    expect(fetchSpy.mock.calls[0][0]).toBe('https://example.com/api/data');
-  });
-
-  it('merges headers', async () => {
-    http.configure({ headers: { Authorization: 'Bearer token' } });
-    mockFetch({});
-    await http.get('https://api.test.com/me');
-    expect(fetchSpy.mock.calls[0][1].headers.Authorization).toBe('Bearer token');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Text response
-// ---------------------------------------------------------------------------
-
-describe('http - text response', () => {
-  it('parses text response', async () => {
-    mockFetch('Hello World');
-    const result = await http.get('https://api.test.com/text');
-    expect(result.data).toBe('Hello World');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Interceptors
-// ---------------------------------------------------------------------------
-
-describe('http - interceptors', () => {
-  it('request interceptor via onRequest', async () => {
-    http.configure({ baseURL: '' });
-    http.onRequest((fetchOpts, url) => {
-      fetchOpts.headers['X-Custom'] = 'test';
-    });
-    mockFetch({});
-    await http.get('https://api.test.com/data');
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.headers['X-Custom']).toBe('test');
-  });
-
-  it('response interceptor via onResponse', async () => {
-    http.onResponse((result) => {
-      result.intercepted = true;
-    });
-    mockFetch({ x: 1 });
-    const result = await http.get('https://api.test.com/data');
-    expect(result.intercepted).toBe(true);
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Timeout / abort
-// ---------------------------------------------------------------------------
-
-describe('http - abort signal', () => {
-  it('passes signal through options', async () => {
-    const controller = new AbortController();
-    mockFetch({});
-    await http.get('https://api.test.com/data', null, { signal: controller.signal });
-    const opts = fetchSpy.mock.calls[0][1];
-    // Signal may be combined via AbortSignal.any - verify it responds to user abort
-    expect(opts.signal).toBeDefined();
-    expect(opts.signal.aborted).toBe(false);
-    controller.abort();
-    expect(opts.signal.aborted).toBe(true);
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Blob response
-// ---------------------------------------------------------------------------
-
-describe('http - blob response', () => {
-  it('can request blob responses', async () => {
-    mockFetch('binary data');
-    const result = await http.get('https://api.test.com/file', null, { responseType: 'blob' });
-    // Should have a data field (blob or fallback text)
-    expect(result.data).toBeDefined();
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// HEAD and OPTIONS methods
-// ---------------------------------------------------------------------------
-
-describe('http - raw fetch pass-through', () => {
-  it('raw() delegates to native fetch', async () => {
-    mockFetch({ ok: true });
-    await http.raw('https://api.test.com/ping', { method: 'HEAD' });
-    expect(fetchSpy).toHaveBeenCalledWith('https://api.test.com/ping', { method: 'HEAD' });
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Request with custom headers
-// ---------------------------------------------------------------------------
-
-describe('http - custom per-request headers', () => {
-  it('merges per-request headers', async () => {
-    mockFetch({});
-    await http.get('https://api.test.com/data', null, {
-      headers: { 'X-Request-Id': '123' },
-    });
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.headers['X-Request-Id']).toBe('123');
-  });
-});
-
-
-// ---------------------------------------------------------------------------
-// Response metadata
-// ---------------------------------------------------------------------------
-
-describe('http - response metadata', () => {
-  it('includes status and ok in result', async () => {
-    mockFetch({ data: 'yes' }, true, 200);
-    const result = await http.get('https://api.test.com/data');
-    expect(result.ok).toBe(true);
-    expect(result.status).toBe(200);
-  });
-
-  it('includes statusText in error', async () => {
-    mockFetch({ error: 'bad' }, false, 500);
-    try {
-      await http.get('https://api.test.com/fail');
-    } catch (err) {
-      expect(err.message).toContain('500');
-    }
-  });
-});
-
-
-// ===========================================================================
-// configure()
-// ===========================================================================
-
-describe('http.configure', () => {
-  it('sets baseURL', async () => {
-    http.configure({ baseURL: 'https://api.myapp.com' });
-    mockFetch({ ok: true });
-    await http.get('/users');
-    expect(fetchSpy.mock.calls[0][0]).toBe('https://api.myapp.com/users');
-  });
-
-  it('merges headers', async () => {
-    http.configure({ headers: { Authorization: 'Bearer token123' } });
-    mockFetch({ ok: true });
-    await http.get('https://api.test.com/me');
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.headers['Authorization']).toBe('Bearer token123');
-    expect(opts.headers['Content-Type']).toBe('application/json');
-  });
-
-  it('sets timeout', () => {
-    http.configure({ timeout: 5000 });
-    // Just verifying it doesn't throw
-    expect(true).toBe(true);
-  });
-});
-
-
-// ===========================================================================
-// interceptors
-// ===========================================================================
-
-describe('http.onRequest - extra', () => {
-  it('interceptor modifying URL', async () => {
-    // Note: interceptors from earlier tests may still be registered; this one
-    // only adds a query param so it doesn't break others
-    http.onRequest((opts, url) => ({ url: url + '?token=abc', options: opts }));
-    mockFetch({ ok: true });
-    await http.get('https://api.test.com/data');
-    expect(fetchSpy.mock.calls[0][0]).toContain('?token=abc');
-  });
-});
-
-// Test interceptor blocking in isolation - it permanently modifies internal state,
-// so we verify behavior without adding a blocking interceptor that breaks later tests
-describe('http - interceptor blocking concept', () => {
-  it('onRequest returning false would throw blocked error', () => {
-    // Just verify the error message format expected by the source code
-    expect(() => { throw new Error('Request blocked by interceptor'); }).toThrow('blocked by interceptor');
-  });
-});
-
-describe('http.onResponse - extra', () => {
-  it('response interceptors are additive', async () => {
-    // onResponse was already called in earlier tests; verify the callback
-    const spy = vi.fn();
-    http.onResponse(spy);
-    mockFetch({ ok: true });
-    await http.get('https://api.test.com/data123');
-    expect(spy).toHaveBeenCalledTimes(1);
-  });
-});
-
-
-// ===========================================================================
-// DELETE with body
-// ===========================================================================
-
-describe('http.delete', () => {
-  it('sends a DELETE request', async () => {
-    mockFetch({ deleted: true });
-    const result = await http.delete('https://api.test.com/item/1');
-    expect(fetchSpy.mock.calls[0][1].method).toBe('DELETE');
-    expect(result.data).toEqual({ deleted: true });
-  });
-
-  it('DELETE with body', async () => {
-    mockFetch({ deleted: true });
-    await http.delete('https://api.test.com/items', { ids: [1, 2] });
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.method).toBe('DELETE');
-    expect(JSON.parse(opts.body)).toEqual({ ids: [1, 2] });
-  });
-});
-
-
-// ===========================================================================
-// GET with existing query string
-// ===========================================================================
-
-describe('http.get - query params', () => {
-  it('appends params with & when URL already has ?', async () => {
-    mockFetch({ ok: true });
-    await http.get('https://api.test.com/search?q=hello', { page: 2 });
-    expect(fetchSpy.mock.calls[0][0]).toContain('?q=hello&page=2');
-  });
-});
-
-
-// ===========================================================================
-// string body
-// ===========================================================================
-
-describe('http.post - string body', () => {
-  it('sends string body as-is', async () => {
-    mockFetch({ ok: true });
-    await http.post('https://api.test.com/raw', 'plain text body');
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.body).toBe('plain text body');
-  });
-});
-
-
-// ===========================================================================
-// URL validation
-// ===========================================================================
-
-describe('http - URL validation', () => {
-  it('throws on missing URL', async () => {
-    await expect(http.get(undefined)).rejects.toThrow('URL string');
-  });
-
-  it('throws on non-string URL', async () => {
-    await expect(http.get(123)).rejects.toThrow('URL string');
-  });
-});
-
-
-// ===========================================================================
-// createAbort
-// ===========================================================================
-
-describe('http.createAbort', () => {
-  it('returns an AbortController', () => {
-    const controller = http.createAbort();
-    expect(controller).toBeInstanceOf(AbortController);
-  });
-});
-
-
-// ===========================================================================
-// timeout / abort
-// ===========================================================================
-
-describe('http - AbortController integration', () => {
-  it('createAbort signal can be passed as option', async () => {
-    const controller = http.createAbort();
-    mockFetch({ ok: true });
-    await http.get('https://api.test.com/data', undefined, { signal: controller.signal });
-    expect(fetchSpy).toHaveBeenCalled();
-  });
-});
-
-
-// ===========================================================================
-// HEAD requests
-// ===========================================================================
-
-describe('http.head', () => {
-  it('sends a HEAD request', async () => {
-    mockFetch({});
-    const result = await http.head('https://api.test.com/resource');
-    expect(fetchSpy.mock.calls[0][1].method).toBe('HEAD');
-    expect(result.ok).toBe(true);
-    expect(result.status).toBe(200);
-  });
-
-  it('does not send a body', async () => {
-    mockFetch({});
-    await http.head('https://api.test.com/resource');
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.body).toBeUndefined();
-  });
-
-  it('accepts per-request options', async () => {
-    mockFetch({});
-    await http.head('https://api.test.com/resource', {
-      headers: { 'X-Check': 'exists' },
-    });
-    const opts = fetchSpy.mock.calls[0][1];
-    expect(opts.headers['X-Check']).toBe('exists');
-  });
-
-  it('returns response headers', async () => {
-    mockFetch({});
-    const result = await http.head('https://api.test.com/resource');
-    expect(result.headers).toBeDefined();
-    expect(typeof result.headers).toBe('object');
-  });
-});
-
-
-// ===========================================================================
-// Interceptor unsubscribe
-// ===========================================================================
-
-describe('http - interceptor unsubscribe', () => {
-  it('onRequest returns an unsubscribe function', async () => {
-    http.clearInterceptors();
-    const spy = vi.fn();
-    const unsub = http.onRequest(spy);
-    expect(typeof unsub).toBe('function');
-
-    mockFetch({});
-    await http.get('https://api.test.com/a');
-    expect(spy).toHaveBeenCalledTimes(1);
-
-    unsub();
-    await http.get('https://api.test.com/b');
-    expect(spy).toHaveBeenCalledTimes(1); // not called again
-  });
-
-  it('onResponse returns an unsubscribe function', async () => {
-    http.clearInterceptors();
-    const spy = vi.fn();
-    const unsub = http.onResponse(spy);
-    expect(typeof unsub).toBe('function');
-
-    mockFetch({});
-    await http.get('https://api.test.com/a');
-    expect(spy).toHaveBeenCalledTimes(1);
-
-    unsub();
-    await http.get('https://api.test.com/b');
-    expect(spy).toHaveBeenCalledTimes(1);
-  });
-
-  it('double-unsubscribe is safe', async () => {
-    http.clearInterceptors();
-    const spy = vi.fn();
-    const unsub = http.onRequest(spy);
-    unsub();
-    unsub(); // should not throw
-    mockFetch({});
-    await http.get('https://api.test.com/data');
-    expect(spy).not.toHaveBeenCalled();
-  });
-});
-
-
-// ===========================================================================
-// clearInterceptors
-// ===========================================================================
-
-describe('http.clearInterceptors', () => {
-  it('clears all interceptors when called with no args', async () => {
-    http.clearInterceptors();
-    const reqSpy = vi.fn();
-    const resSpy = vi.fn();
-    http.onRequest(reqSpy);
-    http.onResponse(resSpy);
-
-    http.clearInterceptors();
-    mockFetch({});
-    await http.get('https://api.test.com/data');
-    expect(reqSpy).not.toHaveBeenCalled();
-    expect(resSpy).not.toHaveBeenCalled();
-  });
-
-  it('clears only request interceptors with "request"', async () => {
-    http.clearInterceptors();
-    const reqSpy = vi.fn();
-    const resSpy = vi.fn();
-    http.onRequest(reqSpy);
-    http.onResponse(resSpy);
-
-    http.clearInterceptors('request');
-    mockFetch({});
-    await http.get('https://api.test.com/data');
-    expect(reqSpy).not.toHaveBeenCalled();
-    expect(resSpy).toHaveBeenCalledTimes(1);
-  });
-
-  it('clears only response interceptors with "response"', async () => {
-    http.clearInterceptors();
-    const reqSpy = vi.fn();
-    const resSpy = vi.fn();
-    http.onRequest(reqSpy);
-    http.onResponse(resSpy);
-
-    http.clearInterceptors('response');
-    mockFetch({});
-    await http.get('https://api.test.com/data');
-    expect(reqSpy).toHaveBeenCalledTimes(1);
-    expect(resSpy).not.toHaveBeenCalled();
-  });
-});
-
-
-// ===========================================================================
-// http.all - parallel requests
-// ===========================================================================
-
-describe('http.all', () => {
-  it('resolves all parallel requests', async () => {
-    mockFetch({ ok: true });
-    const results = await http.all([
-      http.get('https://api.test.com/a'),
-      http.get('https://api.test.com/b'),
-      http.get('https://api.test.com/c'),
-    ]);
-    expect(results).toHaveLength(3);
-    expect(results.every(r => r.ok)).toBe(true);
-  });
-
-  it('rejects if any request fails', async () => {
-    http.clearInterceptors();
-    mockFetch({ error: 'fail' }, false, 500);
-    await expect(
-      http.all([
-        http.get('https://api.test.com/a'),
-        http.get('https://api.test.com/b'),
-      ])
-    ).rejects.toThrow();
-  });
-
-  it('handles empty array', async () => {
-    const results = await http.all([]);
-    expect(results).toEqual([]);
-  });
-});
-
-
-// ===========================================================================
-// http.getConfig
-// ===========================================================================
-
-describe('http.getConfig', () => {
-  it('returns current config', () => {
-    http.configure({ baseURL: 'https://myapi.com', timeout: 5000 });
-    const config = http.getConfig();
-    expect(config.baseURL).toBe('https://myapi.com');
-    expect(config.timeout).toBe(5000);
-    expect(config.headers).toBeDefined();
-  });
-
-  it('returns a copy (not the internal reference)', () => {
-    const config = http.getConfig();
-    config.baseURL = 'https://mutated.com';
-    config.headers['X-Evil'] = 'injected';
-    const fresh = http.getConfig();
-    expect(fresh.baseURL).not.toBe('https://mutated.com');
-    expect(fresh.headers['X-Evil']).toBeUndefined();
-  });
-
-  it('reflects updates after configure', () => {
-    http.configure({ baseURL: '' });
-    expect(http.getConfig().baseURL).toBe('');
-    http.configure({ baseURL: 'https://updated.com' });
-    expect(http.getConfig().baseURL).toBe('https://updated.com');
-  });
-});
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { http } from '../src/http.js';
+
+
+// ---------------------------------------------------------------------------
+// Mock fetch
+// ---------------------------------------------------------------------------
+let fetchSpy;
+
+function mockFetch(response = {}, ok = true, status = 200) {
+  const body = typeof response === 'string' ? response : JSON.stringify(response);
+  const contentType = typeof response === 'string' ? 'text/plain' : 'application/json';
+
+  fetchSpy = vi.fn(() =>
+    Promise.resolve({
+      ok,
+      status,
+      statusText: ok ? 'OK' : 'Error',
+      headers: {
+        get: (h) => h.toLowerCase() === 'content-type' ? contentType : null,
+        entries: () => [[`content-type`, contentType]],
+      },
+      json: () => Promise.resolve(typeof response === 'object' ? response : JSON.parse(response)),
+      text: () => Promise.resolve(body),
+      blob: () => Promise.resolve(new Blob([body])),
+    })
+  );
+  globalThis.fetch = fetchSpy;
+}
+
+beforeEach(() => {
+  http.configure({ baseURL: '', headers: { 'Content-Type': 'application/json' }, timeout: 30000 });
+});
+
+afterEach(() => {
+  vi.restoreAllMocks();
+});
+
+
+// ---------------------------------------------------------------------------
+// GET requests
+// ---------------------------------------------------------------------------
+
+describe('http.get', () => {
+  it('makes a GET request', async () => {
+    mockFetch({ users: [] });
+    const result = await http.get('https://api.test.com/users');
+    expect(fetchSpy).toHaveBeenCalledOnce();
+    expect(result.data).toEqual({ users: [] });
+    expect(result.ok).toBe(true);
+    expect(result.status).toBe(200);
+  });
+
+  it('appends query params for GET', async () => {
+    mockFetch({});
+    await http.get('https://api.test.com/search', { q: 'hello', page: '1' });
+    const url = fetchSpy.mock.calls[0][0];
+    expect(url).toContain('q=hello');
+    expect(url).toContain('page=1');
+  });
+
+  it('uses baseURL', async () => {
+    http.configure({ baseURL: 'https://api.test.com' });
+    mockFetch({});
+    await http.get('/users');
+    expect(fetchSpy.mock.calls[0][0]).toBe('https://api.test.com/users');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// POST requests
+// ---------------------------------------------------------------------------
+
+describe('http.post', () => {
+  it('sends JSON body', async () => {
+    mockFetch({ id: 1 });
+    const result = await http.post('https://api.test.com/users', { name: 'Tony' });
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.method).toBe('POST');
+    expect(opts.body).toBe(JSON.stringify({ name: 'Tony' }));
+    expect(result.data).toEqual({ id: 1 });
+  });
+
+  it('handles FormData body', async () => {
+    mockFetch({});
+    const form = new FormData();
+    form.append('file', 'data');
+    await http.post('https://api.test.com/upload', form);
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.body).toBe(form);
+    // Content-Type should be removed for FormData
+    expect(opts.headers['Content-Type']).toBeUndefined();
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Error handling
+// ---------------------------------------------------------------------------
+
+describe('http - error handling', () => {
+  it('throws on non-ok response', async () => {
+    mockFetch({ error: 'Not Found' }, false, 404);
+    await expect(http.get('https://api.test.com/missing')).rejects.toThrow('HTTP 404');
+  });
+
+  it('error includes response data', async () => {
+    mockFetch({ message: 'Unauthorized' }, false, 401);
+    try {
+      await http.get('https://api.test.com/protected');
+    } catch (err) {
+      expect(err.response).toBeDefined();
+      expect(err.response.status).toBe(401);
+    }
+  });
+
+  it('throws on invalid URL', async () => {
+    await expect(http.get(null)).rejects.toThrow('requires a URL string');
+    await expect(http.get(undefined)).rejects.toThrow('requires a URL string');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// PUT / PATCH / DELETE
+// ---------------------------------------------------------------------------
+
+describe('http - other methods', () => {
+  it('PUT sends correct method', async () => {
+    mockFetch({});
+    await http.put('https://api.test.com/users/1', { name: 'Updated' });
+    expect(fetchSpy.mock.calls[0][1].method).toBe('PUT');
+  });
+
+  it('PATCH sends correct method', async () => {
+    mockFetch({});
+    await http.patch('https://api.test.com/users/1', { name: 'Patched' });
+    expect(fetchSpy.mock.calls[0][1].method).toBe('PATCH');
+  });
+
+  it('DELETE sends correct method', async () => {
+    mockFetch({});
+    await http.delete('https://api.test.com/users/1');
+    expect(fetchSpy.mock.calls[0][1].method).toBe('DELETE');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// configure
+// ---------------------------------------------------------------------------
+
+describe('http.configure', () => {
+  it('sets baseURL', async () => {
+    http.configure({ baseURL: 'https://example.com/api' });
+    mockFetch({});
+    await http.get('/data');
+    expect(fetchSpy.mock.calls[0][0]).toBe('https://example.com/api/data');
+  });
+
+  it('merges headers', async () => {
+    http.configure({ headers: { Authorization: 'Bearer token' } });
+    mockFetch({});
+    await http.get('https://api.test.com/me');
+    expect(fetchSpy.mock.calls[0][1].headers.Authorization).toBe('Bearer token');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Text response
+// ---------------------------------------------------------------------------
+
+describe('http - text response', () => {
+  it('parses text response', async () => {
+    mockFetch('Hello World');
+    const result = await http.get('https://api.test.com/text');
+    expect(result.data).toBe('Hello World');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Interceptors
+// ---------------------------------------------------------------------------
+
+describe('http - interceptors', () => {
+  it('request interceptor via onRequest', async () => {
+    http.configure({ baseURL: '' });
+    http.onRequest((fetchOpts, url) => {
+      fetchOpts.headers['X-Custom'] = 'test';
+    });
+    mockFetch({});
+    await http.get('https://api.test.com/data');
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.headers['X-Custom']).toBe('test');
+  });
+
+  it('response interceptor via onResponse', async () => {
+    http.onResponse((result) => {
+      result.intercepted = true;
+    });
+    mockFetch({ x: 1 });
+    const result = await http.get('https://api.test.com/data');
+    expect(result.intercepted).toBe(true);
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Timeout / abort
+// ---------------------------------------------------------------------------
+
+describe('http - abort signal', () => {
+  it('passes signal through options', async () => {
+    const controller = new AbortController();
+    mockFetch({});
+    await http.get('https://api.test.com/data', null, { signal: controller.signal });
+    const opts = fetchSpy.mock.calls[0][1];
+    // Signal may be combined via AbortSignal.any - verify it responds to user abort
+    expect(opts.signal).toBeDefined();
+    expect(opts.signal.aborted).toBe(false);
+    controller.abort();
+    expect(opts.signal.aborted).toBe(true);
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Blob response
+// ---------------------------------------------------------------------------
+
+describe('http - blob response', () => {
+  it('can request blob responses', async () => {
+    mockFetch('binary data');
+    const result = await http.get('https://api.test.com/file', null, { responseType: 'blob' });
+    // Should have a data field (blob or fallback text)
+    expect(result.data).toBeDefined();
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// HEAD and OPTIONS methods
+// ---------------------------------------------------------------------------
+
+describe('http - raw fetch pass-through', () => {
+  it('raw() delegates to native fetch', async () => {
+    mockFetch({ ok: true });
+    await http.raw('https://api.test.com/ping', { method: 'HEAD' });
+    expect(fetchSpy).toHaveBeenCalledWith('https://api.test.com/ping', { method: 'HEAD' });
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Request with custom headers
+// ---------------------------------------------------------------------------
+
+describe('http - custom per-request headers', () => {
+  it('merges per-request headers', async () => {
+    mockFetch({});
+    await http.get('https://api.test.com/data', null, {
+      headers: { 'X-Request-Id': '123' },
+    });
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.headers['X-Request-Id']).toBe('123');
+  });
+});
+
+
+// ---------------------------------------------------------------------------
+// Response metadata
+// ---------------------------------------------------------------------------
+
+describe('http - response metadata', () => {
+  it('includes status and ok in result', async () => {
+    mockFetch({ data: 'yes' }, true, 200);
+    const result = await http.get('https://api.test.com/data');
+    expect(result.ok).toBe(true);
+    expect(result.status).toBe(200);
+  });
+
+  it('includes statusText in error', async () => {
+    mockFetch({ error: 'bad' }, false, 500);
+    try {
+      await http.get('https://api.test.com/fail');
+    } catch (err) {
+      expect(err.message).toContain('500');
+    }
+  });
+});
+
+
+// ===========================================================================
+// configure()
+// ===========================================================================
+
+describe('http.configure', () => {
+  it('sets baseURL', async () => {
+    http.configure({ baseURL: 'https://api.myapp.com' });
+    mockFetch({ ok: true });
+    await http.get('/users');
+    expect(fetchSpy.mock.calls[0][0]).toBe('https://api.myapp.com/users');
+  });
+
+  it('merges headers', async () => {
+    http.configure({ headers: { Authorization: 'Bearer token123' } });
+    mockFetch({ ok: true });
+    await http.get('https://api.test.com/me');
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.headers['Authorization']).toBe('Bearer token123');
+    expect(opts.headers['Content-Type']).toBe('application/json');
+  });
+
+  it('sets timeout', () => {
+    http.configure({ timeout: 5000 });
+    // Just verifying it doesn't throw
+    expect(true).toBe(true);
+  });
+});
+
+
+// ===========================================================================
+// interceptors
+// ===========================================================================
+
+describe('http.onRequest - extra', () => {
+  it('interceptor modifying URL', async () => {
+    // Note: interceptors from earlier tests may still be registered; this one
+    // only adds a query param so it doesn't break others
+    http.onRequest((opts, url) => ({ url: url + '?token=abc', options: opts }));
+    mockFetch({ ok: true });
+    await http.get('https://api.test.com/data');
+    expect(fetchSpy.mock.calls[0][0]).toContain('?token=abc');
+  });
+});
+
+// Test interceptor blocking in isolation - it permanently modifies internal state,
+// so we verify behavior without adding a blocking interceptor that breaks later tests
+describe('http - interceptor blocking concept', () => {
+  it('onRequest returning false would throw blocked error', () => {
+    // Just verify the error message format expected by the source code
+    expect(() => { throw new Error('Request blocked by interceptor'); }).toThrow('blocked by interceptor');
+  });
+});
+
+describe('http.onResponse - extra', () => {
+  it('response interceptors are additive', async () => {
+    // onResponse was already called in earlier tests; verify the callback
+    const spy = vi.fn();
+    http.onResponse(spy);
+    mockFetch({ ok: true });
+    await http.get('https://api.test.com/data123');
+    expect(spy).toHaveBeenCalledTimes(1);
+  });
+});
+
+
+// ===========================================================================
+// DELETE with body
+// ===========================================================================
+
+describe('http.delete', () => {
+  it('sends a DELETE request', async () => {
+    mockFetch({ deleted: true });
+    const result = await http.delete('https://api.test.com/item/1');
+    expect(fetchSpy.mock.calls[0][1].method).toBe('DELETE');
+    expect(result.data).toEqual({ deleted: true });
+  });
+
+  it('DELETE with body', async () => {
+    mockFetch({ deleted: true });
+    await http.delete('https://api.test.com/items', { ids: [1, 2] });
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.method).toBe('DELETE');
+    expect(JSON.parse(opts.body)).toEqual({ ids: [1, 2] });
+  });
+});
+
+
+// ===========================================================================
+// GET with existing query string
+// ===========================================================================
+
+describe('http.get - query params', () => {
+  it('appends params with & when URL already has ?', async () => {
+    mockFetch({ ok: true });
+    await http.get('https://api.test.com/search?q=hello', { page: 2 });
+    expect(fetchSpy.mock.calls[0][0]).toContain('?q=hello&page=2');
+  });
+});
+
+
+// ===========================================================================
+// string body
+// ===========================================================================
+
+describe('http.post - string body', () => {
+  it('sends string body as-is', async () => {
+    mockFetch({ ok: true });
+    await http.post('https://api.test.com/raw', 'plain text body');
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.body).toBe('plain text body');
+  });
+});
+
+
+// ===========================================================================
+// URL validation
+// ===========================================================================
+
+describe('http - URL validation', () => {
+  it('throws on missing URL', async () => {
+    await expect(http.get(undefined)).rejects.toThrow('URL string');
+  });
+
+  it('throws on non-string URL', async () => {
+    await expect(http.get(123)).rejects.toThrow('URL string');
+  });
+});
+
+
+// ===========================================================================
+// createAbort
+// ===========================================================================
+
+describe('http.createAbort', () => {
+  it('returns an AbortController', () => {
+    const controller = http.createAbort();
+    expect(controller).toBeInstanceOf(AbortController);
+  });
+});
+
+
+// ===========================================================================
+// timeout / abort
+// ===========================================================================
+
+describe('http - AbortController integration', () => {
+  it('createAbort signal can be passed as option', async () => {
+    const controller = http.createAbort();
+    mockFetch({ ok: true });
+    await http.get('https://api.test.com/data', undefined, { signal: controller.signal });
+    expect(fetchSpy).toHaveBeenCalled();
+  });
+});
+
+
+// ===========================================================================
+// HEAD requests
+// ===========================================================================
+
+describe('http.head', () => {
+  it('sends a HEAD request', async () => {
+    mockFetch({});
+    const result = await http.head('https://api.test.com/resource');
+    expect(fetchSpy.mock.calls[0][1].method).toBe('HEAD');
+    expect(result.ok).toBe(true);
+    expect(result.status).toBe(200);
+  });
+
+  it('does not send a body', async () => {
+    mockFetch({});
+    await http.head('https://api.test.com/resource');
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.body).toBeUndefined();
+  });
+
+  it('accepts per-request options', async () => {
+    mockFetch({});
+    await http.head('https://api.test.com/resource', {
+      headers: { 'X-Check': 'exists' },
+    });
+    const opts = fetchSpy.mock.calls[0][1];
+    expect(opts.headers['X-Check']).toBe('exists');
+  });
+
+  it('returns response headers', async () => {
+    mockFetch({});
+    const result = await http.head('https://api.test.com/resource');
+    expect(result.headers).toBeDefined();
+    expect(typeof result.headers).toBe('object');
+  });
+});
+
+
+// ===========================================================================
+// Interceptor unsubscribe
+// ===========================================================================
+
+describe('http - interceptor unsubscribe', () => {
+  it('onRequest returns an unsubscribe function', async () => {
+    http.clearInterceptors();
+    const spy = vi.fn();
+    const unsub = http.onRequest(spy);
+    expect(typeof unsub).toBe('function');
+
+    mockFetch({});
+    await http.get('https://api.test.com/a');
+    expect(spy).toHaveBeenCalledTimes(1);
+
+    unsub();
+    await http.get('https://api.test.com/b');
+    expect(spy).toHaveBeenCalledTimes(1); // not called again
+  });
+
+  it('onResponse returns an unsubscribe function', async () => {
+    http.clearInterceptors();
+    const spy = vi.fn();
+    const unsub = http.onResponse(spy);
+    expect(typeof unsub).toBe('function');
+
+    mockFetch({});
+    await http.get('https://api.test.com/a');
+    expect(spy).toHaveBeenCalledTimes(1);
+
+    unsub();
+    await http.get('https://api.test.com/b');
+    expect(spy).toHaveBeenCalledTimes(1);
+  });
+
+  it('double-unsubscribe is safe', async () => {
+    http.clearInterceptors();
+    const spy = vi.fn();
+    const unsub = http.onRequest(spy);
+    unsub();
+    unsub(); // should not throw
+    mockFetch({});
+    await http.get('https://api.test.com/data');
+    expect(spy).not.toHaveBeenCalled();
+  });
+});
+
+
+// ===========================================================================
+// clearInterceptors
+// ===========================================================================
+
+describe('http.clearInterceptors', () => {
+  it('clears all interceptors when called with no args', async () => {
+    http.clearInterceptors();
+    const reqSpy = vi.fn();
+    const resSpy = vi.fn();
+    http.onRequest(reqSpy);
+    http.onResponse(resSpy);
+
+    http.clearInterceptors();
+    mockFetch({});
+    await http.get('https://api.test.com/data');
+    expect(reqSpy).not.toHaveBeenCalled();
+    expect(resSpy).not.toHaveBeenCalled();
+  });
+
+  it('clears only request interceptors with "request"', async () => {
+    http.clearInterceptors();
+    const reqSpy = vi.fn();
+    const resSpy = vi.fn();
+    http.onRequest(reqSpy);
+    http.onResponse(resSpy);
+
+    http.clearInterceptors('request');
+    mockFetch({});
+    await http.get('https://api.test.com/data');
+    expect(reqSpy).not.toHaveBeenCalled();
+    expect(resSpy).toHaveBeenCalledTimes(1);
+  });
+
+  it('clears only response interceptors with "response"', async () => {
+    http.clearInterceptors();
+    const reqSpy = vi.fn();
+    const resSpy = vi.fn();
+    http.onRequest(reqSpy);
+    http.onResponse(resSpy);
+
+    http.clearInterceptors('response');
+    mockFetch({});
+    await http.get('https://api.test.com/data');
+    expect(reqSpy).toHaveBeenCalledTimes(1);
+    expect(resSpy).not.toHaveBeenCalled();
+  });
+});
+
+
+// ===========================================================================
+// http.all - parallel requests
+// ===========================================================================
+
+describe('http.all', () => {
+  it('resolves all parallel requests', async () => {
+    mockFetch({ ok: true });
+    const results = await http.all([
+      http.get('https://api.test.com/a'),
+      http.get('https://api.test.com/b'),
+      http.get('https://api.test.com/c'),
+    ]);
+    expect(results).toHaveLength(3);
+    expect(results.every(r => r.ok)).toBe(true);
+  });
+
+  it('rejects if any request fails', async () => {
+    http.clearInterceptors();
+    mockFetch({ error: 'fail' }, false, 500);
+    await expect(
+      http.all([
+        http.get('https://api.test.com/a'),
+        http.get('https://api.test.com/b'),
+      ])
+    ).rejects.toThrow();
+  });
+
+  it('handles empty array', async () => {
+    const results = await http.all([]);
+    expect(results).toEqual([]);
+  });
+});
+
+
+// ===========================================================================
+// http.getConfig
+// ===========================================================================
+
+describe('http.getConfig', () => {
+  it('returns current config', () => {
+    http.configure({ baseURL: 'https://myapi.com', timeout: 5000 });
+    const config = http.getConfig();
+    expect(config.baseURL).toBe('https://myapi.com');
+    expect(config.timeout).toBe(5000);
+    expect(config.headers).toBeDefined();
+  });
+
+  it('returns a copy (not the internal reference)', () => {
+    const config = http.getConfig();
+    config.baseURL = 'https://mutated.com';
+    config.headers['X-Evil'] = 'injected';
+    const fresh = http.getConfig();
+    expect(fresh.baseURL).not.toBe('https://mutated.com');
+    expect(fresh.headers['X-Evil']).toBeUndefined();
+  });
+
+  it('reflects updates after configure', () => {
+    http.configure({ baseURL: '' });
+    expect(http.getConfig().baseURL).toBe('');
+    http.configure({ baseURL: 'https://updated.com' });
+    expect(http.getConfig().baseURL).toBe('https://updated.com');
+  });
+});