@onebun/core 0.2.7 → 0.2.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onebun/core",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "description": "Core package for OneBun framework - decorators, DI, modules, controllers",
   "license": "LGPL-3.0",
   "author": "RemRyahirev",

package/src/application/application.test.ts

@@ -38,6 +38,7 @@ import {
 } from '../decorators/decorators';
 import { Controller as BaseController } from '../module/controller';
 import { BaseMiddleware } from '../module/middleware';
+import { clearGlobalServicesRegistry } from '../module/module';
 import { Service } from '../module/service';
 import { QueueService, QUEUE_NOT_ENABLED_ERROR_MESSAGE } from '../queue';
 import { Subscribe } from '../queue/decorators';
@@ -157,6 +158,7 @@ describe('OneBunApplication', () => {
 
   afterEach(() => {
     register.clear();
+    clearGlobalServicesRegistry();
   });
 
   describe('Constructor and basic initialization', () => {
@@ -1842,7 +1844,7 @@ describe('OneBunApplication', () => {
       expect(body.result.hasOptional).toBe(false);
     });
 
-    test('should return 500 when required query parameter is missing', async () => {
+    test('should return 400 when required query parameter is missing', async () => {
       @Controller('/api')
       class ApiController extends BaseController {
         @Get('/required-query')
@@ -1869,7 +1871,7 @@ describe('OneBunApplication', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const response = await (mockServer as any).fetchHandler(request);
 
-      expect(response.status).toBe(500);
+      expect(response.status).toBe(400);
     });
 
     test('should pass validation with required query parameter present', async () => {
@@ -1966,7 +1968,7 @@ describe('OneBunApplication', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const response = await (mockServer as any).fetchHandler(request);
 
-      expect(response.status).toBe(500);
+      expect(response.status).toBe(400);
     });
 
     test('should validate body with arktype schema', async () => {
@@ -2045,7 +2047,7 @@ describe('OneBunApplication', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const response = await (mockServer as any).fetchHandler(request);
 
-      expect(response.status).toBe(500);
+      expect(response.status).toBe(400);
     });
 
     test('should handle metrics endpoint', async () => {
@@ -3449,7 +3451,7 @@ describe('OneBunApplication', () => {
       expect(body.result.isUndefined).toBe(true);
     });
 
-    test('should return 500 when required cookie is missing', async () => {
+    test('should return 400 when required cookie is missing', async () => {
       @Controller('/api')
       class ApiController extends BaseController {
         @Get('/auth')
@@ -3474,7 +3476,7 @@ describe('OneBunApplication', () => {
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const response = await (mockServer as any).fetchHandler(request);
 
-      expect(response.status).toBe(500);
+      expect(response.status).toBe(400);
     });
 
     test('should extract multiple cookies with @Cookie decorator', async () => {
@@ -4036,5 +4038,49 @@ describe('OneBunApplication', () => {
 
       await app.stop();
     });
+
+    test('QueueService is injectable in controllers of child modules', async () => {
+      @Controller('/child-queue')
+      class ChildQueueController extends BaseController {
+        constructor(private queueService: QueueService) {
+          super();
+        }
+
+        publishTest(pattern: string, data: unknown) {
+          return this.queueService.publish(pattern, data);
+        }
+      }
+
+      @Controller('/child-sub')
+      class ChildSubscribeController extends BaseController {
+        @Subscribe('child.test')
+        async handle(): Promise<void> {}
+      }
+
+      @Module({
+        controllers: [ChildQueueController, ChildSubscribeController],
+      })
+      class ChildQueueModule {}
+
+      @Module({
+        imports: [ChildQueueModule],
+      })
+      class ParentQueueModule {}
+
+      const app = createTestApp(ParentQueueModule, {
+        port: 0,
+        queue: { enabled: true, adapter: 'memory' },
+      });
+      await app.start();
+
+      const rootModule = (app as unknown as { rootModule: ModuleInstance }).rootModule;
+      const controller = rootModule.getControllerInstance?.(ChildQueueController) as
+        | ChildQueueController
+        | undefined;
+      expect(controller).toBeDefined();
+      await expect(controller!.publishTest('child.test', { ok: true })).resolves.toBeDefined();
+
+      await app.stop();
+    });
   });
 });