Update test-runs.service.spec.ts

Author: pashidlos

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

@@ -159,6 +159,7 @@ describe('TestRunsService', () => {
         branchName: 'master',
         baselineBranchName: 'master',
         comment: 'some comment',
+        merge: false
       };
       const testRunUpdateMock = jest.fn();
       const testRunFindOneMock = jest.fn().mockResolvedValueOnce(testRun);
@@ -178,7 +179,79 @@ describe('TestRunsService', () => {
       service.findOne = testRunFindOneMock;
       service.emitUpdateBuildEvent = jest.fn();
 
-      await service.approve(testRun.id);
+      await service.approve(testRun.id, false);
+
+      expect(testRunFindOneMock).toHaveBeenCalledWith(testRun.id);
+      expect(getImageMock).toHaveBeenCalledWith(testRun.imageName);
+      expect(saveImageMock).toHaveBeenCalledTimes(1);
+      expect(testRunUpdateMock).toHaveBeenCalledWith({
+        where: { id: testRun.id },
+        data: {
+          status: TestStatus.approved,
+          testVariation: {
+            update: {
+              baselineName,
+              baselines: {
+                create: {
+                  baselineName,
+                  testRun: {
+                    connect: {
+                      id: testRun.id,
+                    },
+                  },
+                },
+              },
+            },
+          },
+        },
+      });
+      expect(service.emitUpdateBuildEvent).toBeCalledWith(testRun.buildId);
+    });
+
+    it('should approve merge', async () => {
+      const testRun: TestRun = {
+        id: 'id',
+        imageName: 'imageName',
+        diffName: 'diffName',
+        baselineName: 'baselineName',
+        diffPercent: 1,
+        pixelMisMatchCount: 10,
+        diffTollerancePercent: 12,
+        status: TestStatus.new,
+        buildId: 'buildId',
+        testVariationId: 'testVariationId',
+        updatedAt: new Date(),
+        createdAt: new Date(),
+        name: 'test run name',
+        ignoreAreas: '[]',
+        browser: 'browser',
+        device: 'device',
+        os: 'os',
+        viewport: 'viewport',
+        branchName: 'develop',
+        baselineBranchName: 'master',
+        comment: 'some comment',
+        merge: false
+      };
+      const testRunUpdateMock = jest.fn();
+      const testRunFindOneMock = jest.fn().mockResolvedValueOnce(testRun);
+      const baselineName = 'some baseline name';
+      const saveImageMock = jest.fn().mockReturnValueOnce(baselineName);
+      const getImageMock = jest.fn().mockReturnValueOnce(
+        new PNG({
+          width: 10,
+          height: 10,
+        })
+      );
+      service = await initService({
+        testRunUpdateMock,
+        saveImageMock,
+        getImageMock,
+      });
+      service.findOne = testRunFindOneMock;
+      service.emitUpdateBuildEvent = jest.fn();
+
+      await service.approve(testRun.id, true);
 
       expect(testRunFindOneMock).toHaveBeenCalledWith(testRun.id);
       expect(getImageMock).toHaveBeenCalledWith(testRun.imageName);
@@ -232,6 +305,7 @@ describe('TestRunsService', () => {
         branchName: 'develop',
         baselineBranchName: 'master',
         comment: 'some comment',
+        merge: false,
         testVariation: {
           id: '123',
           projectId: 'project Id',
@@ -285,7 +359,7 @@ describe('TestRunsService', () => {
       service.findOne = testRunFindOneMock;
       service.emitUpdateBuildEvent = jest.fn();
 
-      await service.approve(testRun.id);
+      await service.approve(testRun.id, false);
 
       expect(testRunFindOneMock).toHaveBeenCalledWith(testRun.id);
       expect(getImageMock).toHaveBeenCalledWith(testRun.imageName);
@@ -342,6 +416,7 @@ describe('TestRunsService', () => {
       device: 'device',
       diffTollerancePercent: undefined,
       branchName: 'develop',
+      merge: true,
     };
     const testRunWithResult = {
       id: 'id',
@@ -435,6 +510,7 @@ describe('TestRunsService', () => {
         baselineBranchName: testVariation.branchName,
         branchName: createTestRequestDto.branchName,
         diffTollerancePercent: createTestRequestDto.diffTollerancePercent,
+        merge: createTestRequestDto.merge,
         status: TestStatus.new,
       },
     });
@@ -773,6 +849,7 @@ describe('TestRunsService', () => {
           comment: 'some comment',
           baselineBranchName: 'master',
           branchName: 'develop',
+          merge: false,
         },
       ],
     };
@@ -845,6 +922,7 @@ describe('TestRunsService', () => {
       comment: 'some comment',
       baselineBranchName: 'master',
       branchName: 'develop',
+      merge: false,
     };
     const testVariationFindOrCreateMock = jest.fn().mockResolvedValueOnce(testVariation);
     const testRunFindManyMock = jest.fn().mockResolvedValueOnce([testRun]);