test module removed

Author: pashidlos

src/app.module.ts

@@ -5,7 +5,6 @@ import { BuildsModule } from './builds/builds.module';
 import { ProjectsModule } from './projects/projects.module';
 import { TestRunsModule } from './test-runs/test-runs.module';
 import { TestVariationsModule } from './test-variations/test-variations.module';
-import { TestModule } from './test/test.module';
 import { PrismaService } from './prisma/prisma.service';
 import { ConfigModule } from '@nestjs/config';
 
@@ -18,7 +17,6 @@ import { ConfigModule } from '@nestjs/config';
     ProjectsModule,
     TestRunsModule,
     TestVariationsModule,
-    TestModule,
   ],
   providers: [PrismaService],
 })

src/test/dto/create-test-request.dto.ts renamed to src/test-runs/dto/create-test-request.dto.ts

@@ -1,14 +1,20 @@
 import { Test, TestingModule } from '@nestjs/testing';
 import { TestRunsController } from './test-runs.controller';
 import { TestRunsService } from './test-runs.service';
+import { TestVariationsService } from '../test-variations/test-variations.service';
+import { PrismaService } from '../prisma/prisma.service';
 
 describe('TestRuns Controller', () => {
   let controller: TestRunsController;
 
   beforeEach(async () => {
     const module: TestingModule = await Test.createTestingModule({
       controllers: [TestRunsController],
-      providers: [{ provide: TestRunsService, useValue: {} }]
+      providers: [
+        { provide: TestRunsService, useValue: {} },
+        { provide: TestVariationsService, useValue: {} },
+        { provide: PrismaService, useValue: {} },
+      ],
     }).compile();
 
     controller = module.get<TestRunsController>(TestRunsController);

src/test/dto/ignore-area.dto.ts renamed to src/test-runs/dto/ignore-area.dto.ts

@@ -1,10 +1,13 @@
-import { Controller, Delete, UseGuards, Param, ParseUUIDPipe, Put, Body, Get, Query } from '@nestjs/common';
-import { ApiTags, ApiParam, ApiBearerAuth, ApiQuery } from '@nestjs/swagger';
+import { Controller, Delete, UseGuards, Param, ParseUUIDPipe, Put, Body, Get, Query, Post } from '@nestjs/common';
+import { ApiTags, ApiParam, ApiBearerAuth, ApiQuery, ApiSecurity, ApiOkResponse } from '@nestjs/swagger';
 import { JwtAuthGuard } from '../auth/guards/auth.guard';
 import { TestRun } from '@prisma/client';
 import { TestRunsService } from './test-runs.service';
-import { IgnoreAreaDto } from '../test/dto/ignore-area.dto';
+import { IgnoreAreaDto } from './dto/ignore-area.dto';
 import { CommentDto } from '../shared/dto/comment.dto';
+import { TestRunResultDto } from './dto/testRunResult.dto';
+import { ApiGuard } from '../auth/guards/api.guard';
+import { CreateTestRequestDto } from './dto/create-test-request.dto';
 
 @ApiTags('test-runs')
 @Controller('test-runs')
@@ -69,4 +72,12 @@ export class TestRunsController {
   updateComment(@Param('testRunId', new ParseUUIDPipe()) id: string, @Body() body: CommentDto): Promise<TestRun> {
     return this.testRunsService.updateComment(id, body);
   }
+
+  @Post()
+  @ApiSecurity('api_key')
+  @ApiOkResponse({ type: TestRunResultDto })
+  @UseGuards(ApiGuard)
+  postTestRun(@Body() createTestRequestDto: CreateTestRequestDto): Promise<TestRunResultDto> {
+    return this.testRunsService.postTestRun(createTestRequestDto);
+  }
 }

src/test/dto/testRunResult.dto.ts renamed to src/test-runs/dto/testRunResult.dto.ts

@@ -4,9 +4,10 @@ import { SharedModule } from '../shared/shared.module';
 import { PrismaService } from '../prisma/prisma.service';
 import { TestRunsController } from './test-runs.controller';
 import { EventsGateway } from '../events/events.gateway';
+import { TestVariationsModule } from '../test-variations/test-variations.module';
 
 @Module({
-  imports: [SharedModule],
+  imports: [SharedModule, TestVariationsModule],
   providers: [TestRunsService, PrismaService, EventsGateway],
   exports: [TestRunsService],
   controllers: [TestRunsController]

src/test-runs/test-runs.controller.spec.ts

@@ -6,12 +6,13 @@ import { StaticService } from '../shared/static/static.service';
 import { PNG } from 'pngjs';
 import { TestStatus, Build, TestRun } from '@prisma/client';
 import Pixelmatch from 'pixelmatch';
-import { CreateTestRequestDto } from '../test/dto/create-test-request.dto';
+import { CreateTestRequestDto } from './dto/create-test-request.dto';
 import { DiffResult } from './diffResult';
-import { IgnoreAreaDto } from '../test/dto/ignore-area.dto';
+import { IgnoreAreaDto } from './dto/ignore-area.dto';
 import { EventsGateway } from '../events/events.gateway';
 import { CommentDto } from '../shared/dto/comment.dto';
 import { BuildDto } from '../builds/dto/build.dto';
+import { TestVariationsService } from '../test-variations/test-variations.service';
 
 jest.mock('pixelmatch');
 
@@ -43,7 +44,7 @@ const initService = async ({
           },
           build: {
             findOne: buildFindOneMock,
-          }
+          },
         },
       },
       {
@@ -58,9 +59,13 @@ const initService = async ({
         provide: EventsGateway,
         useValue: {
           newTestRun: eventNewTestRunMock,
-          buildUpdated: eventBuildUpdatedMock
+          buildUpdated: eventBuildUpdatedMock,
         },
       },
+      {
+        provide: TestVariationsService,
+        useValue: {},
+      },
     ],
   }).compile();
 
@@ -113,7 +118,7 @@ describe('TestRunsService', () => {
         status: TestStatus.failed,
       },
     });
-    expect(service.emitUpdateBuildEvent).toBeCalledWith(testRun.buildId)
+    expect(service.emitUpdateBuildEvent).toBeCalledWith(testRun.buildId);
   });
 
   describe('approve', () => {
@@ -174,7 +179,7 @@ describe('TestRunsService', () => {
           },
         },
       });
-      expect(service.emitUpdateBuildEvent).toBeCalledWith(testRun.buildId)
+      expect(service.emitUpdateBuildEvent).toBeCalledWith(testRun.buildId);
     });
   });
 
@@ -615,23 +620,23 @@ describe('TestRunsService', () => {
         },
       ],
     };
-    const buildFindOneMock = jest.fn().mockResolvedValueOnce(build)
-    const eventBuildUpdatedMock = jest.fn()
+    const buildFindOneMock = jest.fn().mockResolvedValueOnce(build);
+    const eventBuildUpdatedMock = jest.fn();
     service = await initService({
       buildFindOneMock,
       eventBuildUpdatedMock,
     });
 
-    await service.emitUpdateBuildEvent(build.id)
+    await service.emitUpdateBuildEvent(build.id);
 
     expect(buildFindOneMock).toHaveBeenCalledWith({
       where: {
-        id: build.id
+        id: build.id,
       },
       include: {
-        testRuns: true
-      }
-    })
-    expect(eventBuildUpdatedMock).toHaveBeenCalledWith(new BuildDto(build))
-  })
+        testRuns: true,
+      },
+    });
+    expect(eventBuildUpdatedMock).toHaveBeenCalledWith(new BuildDto(build));
+  });
 });

src/test-runs/test-runs.controller.ts

@@ -1,19 +1,22 @@
 import { Injectable } from '@nestjs/common';
 import { PNG } from 'pngjs';
 import Pixelmatch from 'pixelmatch';
-import { CreateTestRequestDto } from '../test/dto/create-test-request.dto';
-import { IgnoreAreaDto } from '../test/dto/ignore-area.dto';
+import { CreateTestRequestDto } from './dto/create-test-request.dto';
+import { IgnoreAreaDto } from './dto/ignore-area.dto';
 import { StaticService } from '../shared/static/static.service';
 import { PrismaService } from '../prisma/prisma.service';
 import { TestRun, TestStatus, TestVariation } from '@prisma/client';
 import { DiffResult } from './diffResult';
 import { EventsGateway } from '../events/events.gateway';
 import { CommentDto } from '../shared/dto/comment.dto';
 import { BuildDto } from '../builds/dto/build.dto';
+import { TestRunResultDto } from '../test-runs/dto/testRunResult.dto';
+import { TestVariationsService } from '../test-variations/test-variations.service';
 
 @Injectable()
 export class TestRunsService {
   constructor(
+    private testVariationService: TestVariationsService,
     private prismaService: PrismaService,
     private staticService: StaticService,
     private eventsGateway: EventsGateway,
@@ -40,6 +43,14 @@ export class TestRunsService {
     });
   }
 
+  async postTestRun(createTestRequestDto: CreateTestRequestDto): Promise<TestRunResultDto> {
+    const testVariation = await this.testVariationService.findOrCreate(createTestRequestDto);
+
+    const testRun = await this.create(testVariation, createTestRequestDto);
+
+    return new TestRunResultDto(testRun, testVariation);
+  }
+
   async emitUpdateBuildEvent(buildId: string) {
     const build = await this.prismaService.build.findOne({
       where: {

src/test-runs/test-runs.module.ts

@@ -4,7 +4,7 @@ import { TestVariationsService } from './test-variations.service';
 import { TestVariation, Baseline } from '@prisma/client';
 import { JwtAuthGuard } from '../auth/guards/auth.guard';
 import { PrismaService } from '../prisma/prisma.service';
-import { IgnoreAreaDto } from '../test/dto/ignore-area.dto';
+import { IgnoreAreaDto } from '../test-runs/dto/ignore-area.dto';
 import { CommentDto } from '../shared/dto/comment.dto';
 
 @ApiTags('test-variations')