From 87fed754f36ec58e8c12dbc6121b1ddfda016f5c Mon Sep 17 00:00:00 2001
From: bendtherules <abhasbhattacharya2@gmail.com>
Date: Sat, 20 Oct 2018 15:46:35 +0530
Subject: [PATCH] Add aggregated status

---
 src/app/app.tsx            |  3 +++
 src/core/store/model.ts    |  5 +++++
 src/core/store/reducers.ts | 34 ++++++++++++++++++++++++++++++----
 3 files changed, 38 insertions(+), 4 deletions(-)

diff --git a/src/app/app.tsx b/src/app/app.tsx
index 96b28df..aae2e90 100644
--- a/src/app/app.tsx
+++ b/src/app/app.tsx
@@ -57,6 +57,9 @@ export class App extends React.Component<AppPropTypes, { username: string | unde
       }, {
         Header: FilterSteps.noCommits,
         accessor: `filterState.${FilterSteps.noCommits}.status`
+      }, {
+        Header: FilterSteps.aggregated,
+        accessor: `filterState.${FilterSteps.aggregated}.status`
       },
     ];
 
diff --git a/src/core/store/model.ts b/src/core/store/model.ts
index 701779a..8a9f761 100644
--- a/src/core/store/model.ts
+++ b/src/core/store/model.ts
@@ -23,6 +23,7 @@ export enum FilterSteps {
   notForkContributor = 'notForkContributor',
   notParentContributor = 'notParentContributor',
   noCommits = 'noCommits',
+  aggregated = 'aggregated',
 }
 
 export enum FilterStatus {
@@ -52,6 +53,10 @@ export interface RepoFilterState {
     status: FilterStatus;
     // failReason?: FailReasons.NoCommits;
   };
+  [FilterSteps.aggregated]: {
+    status: FilterStatus;
+    // failReason?: FailReasons.NoCommits;
+  };
 }
 
 export type RepoFilterStateMap = Map<
diff --git a/src/core/store/reducers.ts b/src/core/store/reducers.ts
index 6de83c7..e9b4072 100644
--- a/src/core/store/reducers.ts
+++ b/src/core/store/reducers.ts
@@ -1,7 +1,7 @@
 import { Reducer, createStore, Store } from 'redux';
 import { devToolsEnhancer } from 'redux-devtools-extension';
 
-import { ApplicationState, FilterStatus, RepoFilterState } from './model';
+import { ApplicationState, FilterStatus, RepoFilterState, FilterSteps } from './model';
 import {
   ActionNames,
   InitUserNameArgs,
@@ -10,6 +10,26 @@ import {
   ActionsType,
 } from './actions';
 
+
+function calculateAggregateStatus(repoFilterState: RepoFilterState) {
+  let aggregatedStatus = FilterStatus.pass;
+
+  const filterStepValues = Object.values(FilterSteps) as FilterSteps[];
+
+
+  for (const stepName of filterStepValues) {
+    const stepStatus = repoFilterState[stepName].status;
+    if (stepStatus === FilterStatus.fail) {
+      aggregatedStatus = FilterStatus.fail;
+      break;
+    } else if (stepStatus === FilterStatus.waiting) {
+      aggregatedStatus = FilterStatus.waiting;
+    }
+  }
+
+  return aggregatedStatus;
+}
+
 export const initialState: ApplicationState = {
   username: '',
   repoStateMap: new Map(),
@@ -23,7 +43,7 @@ export const reducer: Reducer<ApplicationState, ActionsType> = (
   // all the cases handled.
   switch (action.type) {
     case ActionNames.InitUserName:
-      return { ...state, username: action.payload.userName };
+      return { ...initialState, username: action.payload.userName };
 
     case ActionNames.InitRepo: {
       const clonedAppState = new Map(state.repoStateMap);
@@ -44,6 +64,9 @@ export const reducer: Reducer<ApplicationState, ActionsType> = (
           noCommits: {
             status: FilterStatus.waiting,
           },
+          aggregated: {
+            status: FilterStatus.waiting,
+          },
         },
       });
       return { ...state, repoStateMap: clonedAppState };
@@ -54,14 +77,17 @@ export const reducer: Reducer<ApplicationState, ActionsType> = (
         JSON.stringify(clonedAppState.get(action.payload.repoName))
       ) as
         | {
-            filterState: RepoFilterState;
-          }
+          filterState: RepoFilterState;
+        }
         | undefined;
 
       if (clonedRepoState !== undefined) {
         clonedRepoState.filterState[action.payload.filterStep] =
           action.payload.filterState;
 
+        clonedRepoState.filterState[FilterSteps.aggregated].status =
+          calculateAggregateStatus(clonedRepoState.filterState);
+
         clonedAppState.set(action.payload.repoName, clonedRepoState);
         return { ...state, repoStateMap: clonedAppState };
       } else {