From 414b3870a600d5172041d140aaa271f0143b068a Mon Sep 17 00:00:00 2001
From: Pavel Strunkin <pashidlos@gmail.com>
Date: Fri, 9 Jul 2021 15:17:33 +0300
Subject: [PATCH 1/2] Clean up old test variations

https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/275
---
 src/components/ProjectForm/ProjectForm.tsx | 20 ++++++++++++++++++++
 src/constants/project.ts                   |  1 +
 src/types/project.ts                       |  1 +
 3 files changed, 22 insertions(+)

diff --git a/src/components/ProjectForm/ProjectForm.tsx b/src/components/ProjectForm/ProjectForm.tsx
index 09c00dfa..c167ba58 100644
--- a/src/components/ProjectForm/ProjectForm.tsx
+++ b/src/components/ProjectForm/ProjectForm.tsx
@@ -94,6 +94,26 @@ export const ProjectForm: React.FunctionComponent = () => {
           });
         }}
       />
+      <TextValidator
+        name="maxBranchLifetime"
+        validators={["minNumber:1"]}
+        errorMessages={["Enter greater than 1"]}
+        InputProps={{ inputProps: { min: 1, step: 1 } }}
+        margin="dense"
+        id="maxBranchLifetime"
+        label="Max branch lifetime (days)"
+        type="number"
+        fullWidth
+        required
+        value={project.maxBranchLifetime}
+        onChange={(event) => {
+          const value = (event.target as HTMLInputElement).value;
+          setProjectEditState(projectDispatch, {
+            ...project,
+            maxBranchLifetime: parseInt(value),
+          });
+        }}
+      />
       <FormControlLabel
         label="Auto approve feature"
         control={
diff --git a/src/constants/project.ts b/src/constants/project.ts
index af9d6c8b..13fa1930 100644
--- a/src/constants/project.ts
+++ b/src/constants/project.ts
@@ -16,4 +16,5 @@ export const DEFAULT_PROJECT_EDIT_STATE: ProjectDto = {
   imageComparison: ImageComparison.pixelmatch,
   imageComparisonConfig: PIXELMATCH_DEFAULT_CONFIG,
   maxBuildAllowed: 100,
+  maxBranchLifetime: 30,
 };
diff --git a/src/types/project.ts b/src/types/project.ts
index 82685dee..e4d91e2d 100644
--- a/src/types/project.ts
+++ b/src/types/project.ts
@@ -12,6 +12,7 @@ export interface Project {
   imageComparison: ImageComparison;
   imageComparisonConfig: string;
   maxBuildAllowed: number;
+  maxBranchLifetime: number;
 }
 
 export type ProjectDto = Omit<Project, "updatedAt" | "createdAt" | "builds">;

From f2ea778bd8fe5219c684d3e91dc240e8e84bb4b0 Mon Sep 17 00:00:00 2001
From: Pavel Strunkin <pashidlos@gmail.com>
Date: Fri, 9 Jul 2021 17:44:17 +0300
Subject: [PATCH 2/2] Clean up old test variations

https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/275
---
 src/components/TestDetailsModal.tsx | 62 +++++++++++++++++------------
 1 file changed, 37 insertions(+), 25 deletions(-)

diff --git a/src/components/TestDetailsModal.tsx b/src/components/TestDetailsModal.tsx
index 3cc2a2ec..e90d7871 100644
--- a/src/components/TestDetailsModal.tsx
+++ b/src/components/TestDetailsModal.tsx
@@ -17,10 +17,7 @@ import {
 import { ToggleButton } from "@material-ui/lab";
 import { useHotkeys } from "react-hotkeys-hook";
 import { TestRun } from "../types";
-import {
-  testRunService,
-  staticService,
-} from "../services";
+import { testRunService, staticService } from "../services";
 import { TestStatus } from "../types/testStatus";
 import { useHistory, Prompt } from "react-router-dom";
 import { IgnoreArea, UpdateIgnoreAreaDto } from "../types/ignoreArea";
@@ -173,9 +170,9 @@ const TestDetailsModal: React.FunctionComponent<{
   const fitStageToScreen = () => {
     const scale = image
       ? Math.min(
-        stageWidth < image.width ? stageWidth / image.width : 1,
-        stageHeigth < image.height ? stageHeigth / image.height : 1
-      )
+          stageWidth < image.width ? stageWidth / image.width : 1,
+          stageHeigth < image.height ? stageHeigth / image.height : 1
+        )
       : 1;
     setStageScale(scale);
     resetPositioin();
@@ -190,23 +187,36 @@ const TestDetailsModal: React.FunctionComponent<{
     let newIgnoreArea = ignoreAreas.find((area) => selectedRectId! === area.id);
     if (newIgnoreArea) {
       setProcessing(true);
-      testRunService.getList(testRun.buildId).then(
-        (testRuns: TestRun[]) => {
+      testRunService
+        .getList(testRun.buildId)
+        .then((testRuns: TestRun[]) => {
           let allIds = testRuns.map((item) => item.id);
-          let data: UpdateIgnoreAreaDto = { ids: allIds, ignoreAreas: [newIgnoreArea!] };
+          let data: UpdateIgnoreAreaDto = {
+            ids: allIds,
+            ignoreAreas: [newIgnoreArea!],
+          };
           testRunService.addIgnoreAreas(data).then(() => {
             setProcessing(false);
             setSelectedRectId(undefined);
-            enqueueSnackbar("Ignore areas are updated in all images in this build.", {
-              variant: "success",
-            });
+            enqueueSnackbar(
+              "Ignore areas are updated in all images in this build.",
+              {
+                variant: "success",
+              }
+            );
+          });
+        })
+        .catch((error) => {
+          enqueueSnackbar("There was an error : " + error, {
+            variant: "error",
           });
-        }).catch((error) => {
-          enqueueSnackbar("There was an error : " + error, { variant: "error" });
           setProcessing(false);
         });
     } else {
-      enqueueSnackbar("There was an error determining which ignore area to apply.", { variant: "error" });
+      enqueueSnackbar(
+        "There was an error determining which ignore area to apply.",
+        { variant: "error" }
+      );
     }
   };
 
@@ -265,10 +275,10 @@ const TestDetailsModal: React.FunctionComponent<{
             )}
             {(testRun.status === TestStatus.unresolved ||
               testRun.status === TestStatus.new) && (
-                <Grid item>
-                  <ApproveRejectButtons testRun={testRun} />
-                </Grid>
-              )}
+              <Grid item>
+                <ApproveRejectButtons testRun={testRun} />
+              </Grid>
+            )}
             <Grid item>
               <IconButton color="inherit" onClick={handleClose}>
                 <Close />
@@ -277,7 +287,7 @@ const TestDetailsModal: React.FunctionComponent<{
           </Grid>
         </Toolbar>
       </AppBar>
-      {(processing) && <LinearProgress />}
+      {processing && <LinearProgress />}
       <Box m={1}>
         <Grid container alignItems="center">
           <Grid item xs={12}>
@@ -349,13 +359,14 @@ const TestDetailsModal: React.FunctionComponent<{
                   </IconButton>
                 </Grid>
               </Tooltip>
-              <Tooltip title="Apply selected ignore area to all images in this build." aria-label="apply ignore area">
+              <Tooltip
+                title="Apply selected ignore area to all images in this build."
+                aria-label="apply ignore area"
+              >
                 <Grid item>
                   <IconButton
                     disabled={!selectedRectId || ignoreAreas.length === 0}
-                    onClick={() =>
-                      applyIgnoreArea()
-                    }
+                    onClick={() => applyIgnoreArea()}
                   >
                     <Collections />
                   </IconButton>
@@ -374,6 +385,7 @@ const TestDetailsModal: React.FunctionComponent<{
           <Grid item>
             <Button
               color="primary"
+              disabled={!testRun.testVariationId}
               onClick={() => {
                 history.push(
                   `${routes.VARIATION_DETAILS_PAGE}/${testRun.testVariationId}`