fix(search): fix tests

Author: ycdzj

src/resources/resources.service.ts

@@ -24,7 +24,7 @@ export class ResourcesService {
       return [];
     }
     const resources = await this.getParentResources(namespaceId, resourceId);
-    if (!resources) {
+    if (resources.length === 0) {
       throw new NotFoundException('Resource not found');
     }
     return resources;
@@ -34,11 +34,8 @@ export class ResourcesService {
     namespaceId: string,
     resourceId: string | null,
   ): Promise<ResourceMetaDto[]> {
-    if (!resourceId) {
-      return [];
-    }
     const resources: Resource[] = [];
-    while (true) {
+    while (resourceId) {
       const resource = await this.resourceRepository.findOne({
         select: [
           'id',
@@ -55,9 +52,6 @@ export class ResourcesService {
         return [];
       }
       resources.push(resource);
-      if (!resource.parentId) {
-        break;
-      }
       resourceId = resource.parentId;
     }
     return resources.map((r) => ResourceMetaDto.fromEntity(r));

src/search/search.e2e-spec.ts

@@ -17,6 +17,7 @@ import { ConfigService } from '@nestjs/config';
 import { getRepositoryToken } from '@nestjs/typeorm';
 import { WizardTaskService } from 'omniboxd/tasks/wizard-task.service';
 import { Task } from 'omniboxd/tasks/tasks.entity';
+import { ResourcesService } from 'omniboxd/resources/resources.service';
 
 // Set environment variable to avoid the error in SearchService constructor
 process.env.OBB_WIZARD_BASE_URL = 'http://localhost:8000';
@@ -111,6 +112,12 @@ describe('SearchController (e2e)', () => {
             listAllResources: jest.fn().mockResolvedValue([]),
           },
         },
+        {
+          provide: ResourcesService,
+          useValue: {
+            getParentResources: jest.fn().mockResolvedValue([]),
+          },
+        },
         {
           provide: MessagesService,
           useValue: {

src/search/search.module.ts

@@ -11,6 +11,7 @@ import { ConversationsModule } from 'omniboxd/conversations/conversations.module
 import { TypeOrmModule } from '@nestjs/typeorm';
 import { Task } from 'omniboxd/tasks/tasks.entity';
 import { TasksModule } from 'omniboxd/tasks/tasks.module';
+import { ResourcesModule } from 'omniboxd/resources/resources.module';
 
 @Module({
   exports: [SearchService],
@@ -19,6 +20,7 @@ import { TasksModule } from 'omniboxd/tasks/tasks.module';
   imports: [
     PermissionsModule,
     NamespaceResourcesModule,
+    ResourcesModule,
     MessagesModule,
     ConversationsModule,
     TasksModule,