Fix failed test scenario

Author: 任庆伟

src/shared/components/challenge-listing/index.jsx

@@ -16,14 +16,12 @@ import Listing from './Listing';
 import ChallengeCardPlaceholder from './placeholders/ChallengeCard';
 import SRMCard from './SRMCard';
 
-import { updateQuery } from 'utils/url';
 import './style.scss';
 
 // Number of challenge placeholder card to display
 const CHALLENGE_PLACEHOLDER_COUNT = 8;
 
 export default function ChallengeListing(props) {
-  updateQuery({ });
   let challenges = props.challenges;
 
   if (props.communityFilter) {

src/shared/reducers/challenge-listing/index.js

@@ -271,10 +271,12 @@ export function factory(req) {
 
   if (req) {
     state.filter = req.query.filter;
-    if (!!state.filter.startDate && moment(state.filter.startDate).isValid() === false) {
+    if (!!state.filter && !!state.filter.startDate
+        && moment(state.filter.startDate).isValid() === false) {
       delete state.filter.startDate;
     }
-    if (!!state.filter.endDate && moment(state.filter.endDate).isValid() === false) {
+    if (!!state.filter && !!state.filter.endDate
+        && moment(state.filter.endDate).isValid() === false) {
       delete state.filter.endDate;
     }
     state.selectedCommunityId = req.query.communityId;

src/shared/routes/Topcoder/ChallengeListing.jsx

@@ -8,6 +8,7 @@ import LoadingIndicator from 'components/LoadingIndicator';
 import qs from 'qs';
 import React from 'react';
 import { SplitRoute } from 'utils/router';
+import { updateQuery } from 'utils/url';
 
 export default function ChallengeListingRoute() {
   return (
@@ -19,6 +20,7 @@ export default function ChallengeListingRoute() {
           /* webpackChunkName: "challenge-listing" */
           'containers/challenge-listing/Listing',
         ).then(({ default: ChallengeListing }) => {
+          updateQuery({ });
           const query = renderProps.location.search ?
             qs.parse(renderProps.location.search.slice(1)) : null;
           const currencyFromUrl = _.get(query, 'currency');

src/shared/utils/url.js

@@ -29,9 +29,11 @@ export function updateQuery(update) {
     if (_.isUndefined(value)) delete query[key];
     else query[key] = value;
   });
-  if (!!query.filter.startDate && moment(query.filter.startDate).isValid() === false) {
+  if (!!query.filter && !!query.filter.startDate
+      && moment(query.filter.startDate).isValid() === false) {
     delete query.filter.startDate;
-  } else if (query.filter.endDate && moment(query.filter.endDate).isValid() === false) {
+  } else if (!!query.filter && !!query.filter.endDate
+      && moment(query.filter.endDate).isValid() === false) {
     delete query.filter.endDate;
   }