build stats added

Author: pashidlos

src/components/TestDetailsModal.tsx

@@ -37,6 +37,7 @@ import { TestRunDetails } from "./TestRunDetails";
 import useImage from "use-image";
 import { routes } from "../constants";
 import { NoImagePlaceholder } from "./NoImageAvailable";
+import { useBuildDispatch, updateBuild } from "../contexts/build.context";
 
 const useStyles = makeStyles((theme) => ({
   imageContainer: {
@@ -65,6 +66,7 @@ const TestDetailsModal: React.FunctionComponent<{
 }> = ({ testRun, updateTestRun }) => {
   const classes = useStyles();
   const history = useHistory();
+  const buildDispatch = useBuildDispatch();
 
   const stageWidth = (window.innerWidth / 2) * 0.9;
   const stageHeigth = window.innerHeight;
@@ -172,6 +174,7 @@ const TestDetailsModal: React.FunctionComponent<{
                   onClick={() =>
                     testRunService.approve(testRun.id).then((testRun) => {
                       updateTestRun(testRun);
+                      updateBuild(buildDispatch, testRun);
                     })
                   }
                 >
@@ -180,9 +183,10 @@ const TestDetailsModal: React.FunctionComponent<{
                 <Button
                   color="secondary"
                   onClick={() =>
-                    testRunService
-                      .reject(testRun.id)
-                      .then((testRun) => updateTestRun(testRun))
+                    testRunService.reject(testRun.id).then((testRun) => {
+                      updateTestRun(testRun);
+                      updateBuild(buildDispatch, testRun);
+                    })
                   }
                 >
                   Reject

src/contexts/build.context.tsx

@@ -1,6 +1,7 @@
-import * as React from "react";
-import { Build } from "../types";
+import React from "react";
+import { Build, TestRun } from "../types";
 import { buildsService } from "../services";
+import { TestStatus } from "../types/testStatus";
 
 interface IRequestAction {
   type: "request";
@@ -22,7 +23,23 @@ interface IDeleteAction {
   payload: string;
 }
 
-type IAction = IRequestAction | IGetAction | IDeleteAction | ISelectAction;
+interface IUpdateAction {
+  type: "update";
+  payload: TestRun;
+}
+
+interface IStatsAction {
+  type: "stats";
+  payload: string;
+}
+
+type IAction =
+  | IRequestAction
+  | IGetAction
+  | IDeleteAction
+  | ISelectAction
+  | IStatsAction
+  | IUpdateAction;
 
 type Dispatch = (action: IAction) => void;
 type State = {
@@ -49,6 +66,57 @@ function buildReducer(state: State, action: IAction): State {
         ...state,
         selectedBuildId: action.payload,
       };
+    case "stats":
+      return {
+        ...state,
+        buildList: state.buildList.map((build) => {
+          if (build.id === action.payload) {
+            // reset stats
+            build.passedCount = 0;
+            build.unresolvedCount = 0;
+            build.failedCount = 0;
+
+            // calculate stats
+            build.testRuns.forEach((testRun) => {
+              switch (testRun.status) {
+                case TestStatus.approved:
+                case TestStatus.ok: {
+                  build.passedCount += 1;
+                  break;
+                }
+                case TestStatus.unresolved:
+                case TestStatus.new: {
+                  build.unresolvedCount += 1;
+                  break;
+                }
+                case TestStatus.failed: {
+                  build.failedCount += 1;
+                  break;
+                }
+              }
+            });
+          }
+          return build;
+        }),
+      };
+    case "update":
+      return {
+        ...state,
+        buildList: state.buildList.map((build) => {
+          if (build.id === action.payload.buildId) {
+            return {
+              ...build,
+              testRuns: build.testRuns.map((testRun) => {
+                if (testRun.id === action.payload.id) {
+                  return action.payload;
+                }
+                return testRun;
+              }),
+            };
+          }
+          return build;
+        }),
+      };
     case "get":
       return {
         ...state,
@@ -99,6 +167,7 @@ async function getBuildList(dispatch: Dispatch, id: string) {
     .getList(id)
     .then((items) => {
       dispatch({ type: "get", payload: items });
+      items.forEach((build) => dispatch({ type: "stats", payload: build.id }));
     })
     .catch((error) => {
       console.log(error.toString());
@@ -124,11 +193,17 @@ async function selectBuild(dispatch: Dispatch, id: string) {
   dispatch({ type: "select", payload: id });
 }
 
+async function updateBuild(dispatch: Dispatch, testRun: TestRun) {
+  dispatch({ type: "update", payload: testRun });
+  dispatch({ type: "stats", payload: testRun.buildId });
+}
+
 export {
   BuildProvider,
   useBuildState,
   useBuildDispatch,
   getBuildList,
   deleteBuild,
   selectBuild,
+  updateBuild,
 };

src/types/build.ts

@@ -1,11 +1,16 @@
 import { TestRun } from "./testRun";
 
 export interface Build {
-    id: string,
-    projectName: string,
-    branchName: string,
-    status: string,
-    createdAt: Date,
-    createdBy: string,
-    testRuns: TestRun[]
-}
+  id: string;
+  projectName: string;
+  branchName: string;
+  status: string;
+  createdAt: Date;
+  createdBy: string;
+  testRuns: TestRun[];
+
+  // stats
+  unresolvedCount: number;
+  passedCount: number;
+  failedCount: number;
+}

src/types/testRun.ts

@@ -2,7 +2,7 @@ import { TestStatus } from "./testStatus";
 
 export interface TestRun {
   id: string;
-  buildId: number;
+  buildId: string;
   imageName: string;
   diffName: string;
   diffPercent: number;