test fix

packages/core/package.json

@@ -5,6 +5,7 @@
   "main": "./dist/index.js",
   "scripts": {
     "test": "vitest run",
+    "test:watch": "vitest",
     "build": "tsup",
     "dev": "tsup --watch"
   },

packages/core/src/router/index.ts

@@ -9,7 +9,7 @@ export const internalRouter = async (req: ApiRequest<any, any>, options: LogLibO
     }
     const method = req.method.toUpperCase() as "POST" | "PUT" | "DELETE" | "GET"
     let path = ""
-    if (method === "POST") {
+    if (method === "POST" || method === "PUT" || method === "DELETE") {
         if (typeof req.body !== 'object' || Array.isArray(req.body)) {
             return { message: "Invalid request body. Expected an object.", code: 400 }
         }

packages/core/src/router/routes/event/post.ts

@@ -42,8 +42,8 @@ export const postEvent: ApiPostHandler<EventPostInput> = async (req, options) =>
                 code: 200,
                 data
             }
-        } catch {
-            throw new GenericError('Error creating event', { path: "/event" })
+        } catch (e) {
+            throw new GenericError('Error creating event', { path: "/event", e })
         }
     } else {
         throw new GenericError('Invalid request body', { path: "/event" })

packages/core/test/event.test.ts

@@ -1,15 +1,13 @@
 /* eslint-disable @typescript-eslint/no-empty-function */
 /* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { eventPost, EventPostInput } from '../src/router/routes/event/post';
+import { postEvent, EventPostInput } from '../src/router/routes/event/post';
 import { Adapter, Events, GenericError } from '../src';
 
 
 describe('eventPost', () => {
     const mockAdapter = {
-        tracker: {
-            createManyEvents: vi.fn(),
-        },
+        createManyEvents: vi.fn(),
     } as unknown as Adapter;
 
     beforeEach(() => {
@@ -49,7 +47,7 @@ describe('eventPost', () => {
             code: 200,
             data: expectedData,
         };
-        const response = await eventPost({ body: input, headers: {} }, { adapter: mockAdapter });
+        const response = await postEvent({ body: input, headers: {} }, { adapter: mockAdapter });
         expect(mockAdapter.createManyEvents).toHaveBeenCalledWith(expectedData);
         expect(response).toEqual(expectedResponse);
     });
@@ -66,7 +64,7 @@ describe('eventPost', () => {
                 },
             ],
         } as EventPostInput;
-        await expect(eventPost({ body: input, headers: {} }, { adapter: mockAdapter })).rejects.toThrow(
+        await expect(postEvent({ body: input, headers: {} }, { adapter: mockAdapter })).rejects.toThrow(
             new GenericError('Invalid request body', { path: '/event' })
         ).catch(() => { });
         expect(mockAdapter.createManyEvents).not.toHaveBeenCalled();

packages/core/test/pageview.test.ts

@@ -5,9 +5,7 @@ import { Adapter } from "../src";
 
 describe('pageViewPost', () => {
     const mockAdapter = {
-        tracker: {
-            createPageView: vi.fn(),
-        },
+        createPageView: vi.fn(),
     } as unknown as Adapter;
 
     beforeEach(() => {

packages/core/test/router.test.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
 import { describe, expect, it, vi } from "vitest";
 import { Adapter } from "../src/adapters";
 import { internalRouter } from "../src/router";
@@ -15,7 +16,7 @@ describe('internalRouter', () => {
         adapter: vi.fn() as unknown as Adapter
     };
 
-    type ValidRequest = ApiRequest<{ path: string }>
+    type ValidRequest = ApiRequest<{ path: string }, any>
     it('should return an error if the request body is not an object', async () => {
         const req = mockRequest('POST', '/test', 'invalid', {}) as unknown as ValidRequest
         req.body = JSON.stringify(req.body) as unknown as { path: '/' }

packages/core/test/session.test.ts

@@ -7,9 +7,7 @@ vi.mock("../src/router/routes/utils/detect/getIpAddress")
 
 describe('sessionPost', () => {
     const mockAdapter = {
-        tracker: {
-            createSession: vi.fn().mockResolvedValue({ message: 'Success', code: 200 }),
-        },
+        createSession: vi.fn().mockResolvedValue({ message: 'Success', code: 200 }),
     } as unknown as Adapter
     const mockOptions: LogLibOptions = {
         adapter: mockAdapter,
@@ -56,12 +54,18 @@ describe('sessionPost', () => {
         expect(res).toEqual({ message: 'bot', code: 200 });
     });
 
-    it('should return an error if the location is invalid', async () => {
+    it('should throw error if there is no way to get location', async () => {
         const req = mockRequest(validRequest);
         const mockGetLocation = vi.fn().mockResolvedValue(null);
-        const res = await sessionPost(req, { ...mockOptions, getLocation: mockGetLocation });
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
+        expect(sessionPost(req, { ...mockOptions, getLocation: mockGetLocation })).rejects.toThrowError();
+    });
+
+    it('should pass on custom location getter implementation', async () => {
+        const req = mockRequest(validRequest);
+        const mockGetLocation = vi.fn().mockResolvedValue({ city: 'Test City', country: 'Test Country' });
+        await sessionPost(req, { ...mockOptions, getLocation: mockGetLocation })
         expect(mockGetLocation).toHaveBeenCalled();
-        expect(res).toEqual({ message: 'Invalid location', code: 400 });
     });
 
     it('should return a localhost message if the IP address is localhost', async () => {

packages/core/test/user.test.ts

@@ -5,9 +5,7 @@ import { Adapter } from "../src";
 
 describe('userPost', () => {
     const mockAdapter = {
-        tracker: {
-            updateUser: vi.fn(),
-        },
+        updateUser: vi.fn(),
     } as unknown as Adapter;
 
     beforeEach(() => {