feat(COMPASS-4530): Increase default max time ms to 1 minute (#179)

* feat(COMPASS-4530): Increase default max time ms to 1 minute and add spaces to the field label

* fix: Update assertion for the max time ms

* chore(deps): Update query parser dev dependency

Author: gribnoysup

package-lock.json

@@ -119,7 +119,7 @@
     "mongodb-connection-model": "^14.5.1",
     "mongodb-data-service": "^16.5.2",
     "mongodb-js-precommit": "^2.2.0",
-    "mongodb-query-parser": "^2.1.1",
+    "mongodb-query-parser": "^2.2.0",
     "mongodb-reflux-store": "^0.0.1",
     "mongodb-schema": "^8.2.5",
     "ms": "^2.1.2",
@@ -159,4 +159,4 @@
     "lodash": "^4.17.15",
     "mongodb-query-util": "^0.2.1"
   }
-}
+}

package.json

@@ -18,6 +18,9 @@ import QUERY_PROPERTIES from 'constants/query-properties';
 
 import styles from './query-bar.less';
 
+/**
+ * @type {Record<string, { type: 'document' | 'numeric' | 'boolean', placeholder: string | null, link: string, label?: string }>}
+ */
 const OPTION_DEFINITION = {
   filter: {
     type: 'document',
@@ -51,8 +54,9 @@ const OPTION_DEFINITION = {
     link: 'https://docs.mongodb.com/manual/reference/method/cursor.limit/'
   },
   maxTimeMS: {
+    label: 'Max Time MS',
     type: 'numeric',
-    placeholder: '5000',
+    placeholder: '60000',
     link: 'https://docs.mongodb.com/manual/reference/method/cursor.maxTimeMS/'
   },
   sample: {
@@ -210,9 +214,11 @@ class QueryBar extends Component {
     const value = OPTION_DEFINITION[option].type === 'boolean' ?
       this.props[option] : this.props[`${option}String`];
 
+    const label = OPTION_DEFINITION[option].label || option;
+
     return (
       <QueryOption
-        label={option}
+        label={label}
         autoPopulated={autoPopulated}
         serverVersion={this.props.serverVersion}
         hasToggle={hasToggle}

src/components/query-bar/query-bar.jsx

@@ -24,7 +24,7 @@
 }
 
 .is-numeric-type {
-  flex-basis: 170px;
+  flex-basis: 185px;
 }
 
 .is-boolean-type {

src/components/query-option/query-option.less

@@ -13,7 +13,7 @@ const DEFAULT_SKIP = 0;
 const DEFAULT_LIMIT = 0;
 const DEFAULT_SAMPLE = false;
 
-const DEFAULT_MAX_TIME_MS = ms('5 seconds');
+const DEFAULT_MAX_TIME_MS = ms('1m');
 const DEFAULT_SAMPLE_SIZE = 1000;
 const DEFAULT_STATE = RESET_STATE;
 

src/constants/query-bar-store.js

@@ -75,7 +75,7 @@ describe('QueryChangedStore [Store]', function() {
       expect(state.project).to.be.deep.equal(null);
       expect(state.collation).to.be.deep.equal(null);
       expect(state.sample).to.be.false;
-      expect(state.maxTimeMS).to.be.equal(5000);
+      expect(state.maxTimeMS).to.be.equal(60000);
       done();
     });