COMPASS-1190: Backport COMPASS 1144 default sort should be null (#1009)

* COMPASS-1190: Backport COMPASS 1144 Allow modifying or disabling the default sort for explains (#1008)

* set default sort to null

* retrieve sidebar database count on data service connect

* expect index usage to be at least 0

This seems to be a repeat error here where it index usage could be 0.

* Fix ChartStoreIntegrationTest

Note: It was removed from master.

* Resolve 'renders the index usages' test on MongoDB 3.4+

This was changed to 'at least' in master, but a quick fix for 1.7 should be enough.

Author: pzrq

src/internal-packages/query/lib/store/query-store.js

@@ -20,7 +20,7 @@ const RESET_STATE = 'reset';
 const APPLY_STATE = 'apply';
 
 const DEFAULT_FILTER = {};
-const DEFAULT_SORT = { _id: 1 };
+const DEFAULT_SORT = null;
 const DEFAULT_LIMIT = 0;
 const DEFAULT_SKIP = 0;
 const DEFAULT_PROJECT = null;

test/functional/compass-functional.test.js

@@ -699,7 +699,7 @@ describe('Compass Main Functional Test Suite #spectron', function() {
             .getIndexUsages()
             .should
             .eventually
-            .equal(isIndexUsageEnabled(serverVersion) ? '8' : '0');
+            .equal(isIndexUsageEnabled(serverVersion) ? '4' : '0');
         });
 
         it('renders the index properties', function() {

test/renderer/chart.store.integration.test.js

@@ -48,7 +48,7 @@ describe('ChartStoreIntegration', function() {
       const QUERY = {filter: {foo: 1}};
       const EXPECTED = Object.assign({}, {
         project: null,
-        sort: {_id: 1},
+        sort: null,
         skip: 0,
         limit: 0,
         ns: '',

test/unit/query-changed-store.test.js

@@ -57,7 +57,7 @@ describe('QueryChangedStore', () => {
   it('contains all the other query options', (done) => {
     unsubscribe = QueryChangedStore.listen((state) => {
       expect(state.filter).to.be.deep.equal({foo: 1});
-      expect(state.sort).to.be.deep.equal({_id: 1});
+      expect(state.sort).to.be.deep.equal(null);
       expect(state.skip).to.be.equal(0);
       expect(state.limit).to.be.equal(0);
       expect(state.project).to.be.deep.equal(null);

test/unit/query-store.test.js

@@ -452,7 +452,7 @@ describe('QueryStore', () => {
           unsubscribe = QueryStore.listen((state) => {
             expect(state.sortString).to.be.equal('{"sort": null}');
             expect(state.sortValid).to.be.false;
-            expect(state.sort).to.deep.equal({_id: 1});
+            expect(state.sort).to.deep.equal(null);
             done();
           });
           QueryStore.setQueryString('sort', '{"sort": null}');