chore(static): remove button "show skipped" and component "SkippedList"

Author: oldskytree

lib/static/components/controls/common-controls.js

@@ -51,11 +51,6 @@ class ControlButtons extends Component {
                     />
                 </div>
                 <GroupTestsSelect />
-                <ControlButton
-                    label="Show skipped"
-                    isActive={view.showSkipped}
-                    handler={actions.toggleSkipped}
-                />
                 <ControlSelect
                     label="Diff mode"
                     value={view.diffMode}

lib/static/components/gui.js

@@ -7,7 +7,6 @@ import {bindActionCreators} from 'redux';
 import {ROOT} from '../../constants/extension-points';
 import * as actions from '../modules/actions';
 import ControlButtons from './controls/gui-controls';
-import SkippedList from './skipped-list';
 import Loading from './loading';
 import ModalContainer from '../containers/modal';
 import MainTree from './main-tree';
@@ -84,7 +83,6 @@ class Gui extends Component {
                     <FaviconChanger />
                     <ControlButtons />
                     <main className="container">
-                        <SkippedList />
                         <MainTree />
                     </main>
                     <Loading active={loading.active} content={loading.content} />

lib/static/components/report.js

@@ -9,7 +9,6 @@ import * as actions from '../modules/actions';
 import Loading from './loading';
 import Header from './header';
 import ControlButtons from './controls/report-controls';
-import SkippedList from './skipped-list';
 import MainTree from './main-tree';
 import CustomScripts from './custom-scripts';
 import FaviconChanger from './favicon-changer';
@@ -55,7 +54,6 @@ class Report extends Component {
                     <Header/>
                     <ControlButtons/>
                     <main className="container">
-                        <SkippedList />
                         <MainTree />
                     </main>
                     <BottomProgressBar />

lib/static/components/skipped-list.js

@@ -27,7 +27,6 @@ export default {
     VIEW_EXPAND_RETRIES: 'VIEW_EXPAND_RETRIES',
     VIEW_COLLAPSE_ALL: 'VIEW_COLLAPSE_ALL',
     CHANGE_VIEW_MODE: 'CHANGE_VIEW_MODE',
-    VIEW_TOGGLE_SKIPPED: 'VIEW_TOGGLE_SKIPPED',
     VIEW_UPDATE_BASE_HOST: 'VIEW_UPDATE_BASE_HOST',
     VIEW_UPDATE_FILTER_BY_NAME: 'VIEW_UPDATE_FILTER_BY_NAME',
     VIEW_SET_STRICT_MATCH_FILTER: 'VIEW_SET_STRICT_MATCH_FILTER',

lib/static/modules/action-names.js

@@ -187,7 +187,6 @@ export const expandAll = () => ({type: actionNames.VIEW_EXPAND_ALL});
 export const expandErrors = () => ({type: actionNames.VIEW_EXPAND_ERRORS});
 export const expandRetries = () => ({type: actionNames.VIEW_EXPAND_RETRIES});
 export const collapseAll = () => ({type: actionNames.VIEW_COLLAPSE_ALL});
-export const toggleSkipped = () => ({type: actionNames.VIEW_TOGGLE_SKIPPED});
 export const processBegin = () => ({type: actionNames.PROCESS_BEGIN});
 export const processEnd = () => ({type: actionNames.PROCESS_END});
 export const updateBaseHost = (host) => ({type: actionNames.VIEW_UPDATE_BASE_HOST, host});

lib/static/modules/actions.js

@@ -69,7 +69,6 @@ export default Object.assign(defaults, {
         viewMode: viewModes.ALL,
         diffMode: diffModes.THREE_UP.id,
         expand: EXPAND_ERRORS,
-        showSkipped: false,
         baseHost: '',
         testNameFilter: '',
         strictMatchFilter: false,

lib/static/modules/default-state.js

@@ -13,7 +13,6 @@ export default store => next => action => {
             expand: view.expand,
             viewMode: view.viewMode,
             diffMode: view.diffMode,
-            showSkipped: view.showSkipped,
             strictMatchFilter: view.strictMatchFilter
         });
     }

lib/static/modules/middlewares/local-storage.js

@@ -66,7 +66,6 @@ export default metrikaClass => store => next => action => {
         case actionNames.VIEW_COLLAPSE_ALL:
         case actionNames.VIEW_EXPAND_ERRORS:
         case actionNames.VIEW_EXPAND_RETRIES:
-        case actionNames.VIEW_TOGGLE_SKIPPED:
         case actionNames.VIEW_UPDATE_BASE_HOST:
         case actionNames.VIEW_THREE_UP_DIFF:
         case actionNames.VIEW_THREE_UP_SCALED_DIFF:

lib/static/modules/middlewares/metrika.js

@@ -57,7 +57,6 @@ describe('lib/static/modules/middlewares/local-storage', () => {
 
                 assert.calledOnceWith(localStorageWrapper.setItem, 'view', {
                     expand: EXPAND_ERRORS,
-                    showSkipped: false,
                     strictMatchFilter: false,
                     viewMode: viewModes.ALL,
                     diffMode: diffModes.THREE_UP.id