Merge pull request #5 from bendtherules/add-ui

Add aggregated status

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`
       },
     ];
 

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<

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 {