feat(compass-crud): Show document actions for time series collections COMPASS-4781

packages/compass-aggregations/src/modules/create-view/index.js

@@ -171,7 +171,7 @@ export const createView = () => {
               isReadonly: true,
               sourceName: viewSource,
               editViewName: null,
-              isSourceReadonly: state.isReadonly,
+              sourceReadonly: state.isReadonly,
               sourceViewOn: state.sourceName,
               sourcePipeline: viewPipeline
             }

packages/compass-aggregations/src/modules/update-view.js

@@ -87,7 +87,7 @@ export const updateView = () => {
           isReadonly: true,
           sourceName: state.namespace,
           editViewName: null,
-          isSourceReadonly: state.isReadonly,
+          sourceReadonly: state.isReadonly,
           sourceViewOn: state.sourceName,
           sourcePipeline: viewPipeline
         };

packages/compass-collection/electron/renderer/index.js

@@ -102,7 +102,7 @@ dataService.connect((error, ds) => {
       isReadonly: false,
       sourceName: null,
       editViewName: null,
-      isSourceReadonly: false,
+      sourceReadonly: false,
       sourceViewOn: null
     }
   );
@@ -113,7 +113,7 @@ dataService.connect((error, ds) => {
       isReadonly: true,
       sourceName: 'citibike.trips',
       editViewName: null,
-      isSourceReadonly: false,
+      sourceReadonly: false,
       sourceViewOn: null,
       sourcePipeline: [{ '$match': { name: 'testing' }}]
     }
@@ -125,7 +125,7 @@ dataService.connect((error, ds) => {
       isReadonly: true,
       sourceName: 'citibike.tripsOfShortDuration',
       editViewName: null,
-      isSourceReadonly: true,
+      sourceReadonly: true,
       sourceViewOn: 'citibike.trips',
       sourcePipeline: [{ '$match': { gender: 0 }}]
     }

packages/compass-collection/src/components/collection-header/collection-header.jsx

@@ -13,6 +13,7 @@ class CollectionHeader extends Component {
     globalAppRegistry: PropTypes.func.isRequired,
     namespace: PropTypes.string.isRequired,
     isReadonly: PropTypes.bool.isRequired,
+    isTimeSeries: PropTypes.bool.isRequired,
     statsPlugin: PropTypes.func.isRequired,
     selectOrCreateTab: PropTypes.func.isRequired,
     statsStore: PropTypes.object.isRequired,
@@ -24,27 +25,29 @@ class CollectionHeader extends Component {
   };
 
   modifySource = () => {
-    this.props.selectOrCreateTab(
-      this.props.sourceName,
-      this.props.sourceReadonly,
-      this.props.sourceViewOn,
-      this.props.namespace,
-      false,
-      null,
-      this.props.pipeline
-    );
+    this.props.selectOrCreateTab({
+      namespace: this.props.sourceName,
+      isReadonly: this.props.sourceReadonly,
+      isTimeSeries: this.props.isTimeSeries,
+      sourceName: this.props.sourceViewOn,
+      editViewName: this.props.namespace,
+      sourceReadonly: false,
+      sourceViewOn: null,
+      sourcePipeline: this.props.pipeline
+    });
   }
 
   returnToView = () => {
-    this.props.selectOrCreateTab(
-      this.props.editViewName,
-      true,
-      this.props.namespace,
-      null,
-      this.props.isReadonly,
-      this.props.sourceName,
-      this.props.pipeline
-    );
+    this.props.selectOrCreateTab({
+      namespace: this.props.editViewName,
+      isReadonly: true,
+      isTimeSeries: this.props.isTimeSeries,
+      sourceName: this.props.namespace,
+      editViewName: null,
+      sourceReadonly: this.props.isReadonly,
+      sourceViewOn: this.props.sourceName,
+      sourcePipeline: this.props.pipeline
+    });
   }
 
   handleDBClick = (db) => {

packages/compass-collection/src/components/collection/collection.jsx

@@ -11,6 +11,7 @@ class Collection extends Component {
 
   static propTypes = {
     namespace: PropTypes.string.isRequired,
+    isTimeSeries: PropTypes.bool,
     isReadonly: PropTypes.bool.isRequired,
     tabs: PropTypes.array.isRequired,
     views: PropTypes.array.isRequired,
@@ -69,6 +70,7 @@ class Collection extends Component {
             globalAppRegistry={this.props.globalAppRegistry}
             namespace={this.props.namespace}
             isReadonly={this.props.isReadonly}
+            isTimeSeries={this.props.isTimeSeries}
             statsPlugin={this.props.statsPlugin}
             statsStore={this.props.statsStore}
             editViewName={this.props.editViewName}

packages/compass-collection/src/components/collection/collection.spec.js

@@ -19,6 +19,7 @@ describe('Collection [Component]', () => {
     component = mount(
       <Collection
         isReadonly={false}
+        isTimeSeries={false}
         tabs={[]}
         views={[]}
         queryHistoryIndexes={[]}

packages/compass-collection/src/components/create-tab/create-tab.jsx

@@ -7,31 +7,20 @@ class CreateTab extends PureComponent {
   static displayName = 'CreateTabComponent';
 
   static propTypes = {
-    createNewTab: PropTypes.func.isRequired,
-    activeNamespace: PropTypes.string.isRequired,
-    activeIsReadonly: PropTypes.bool.isRequired,
-    activeSourceName: PropTypes.string
+    createNewTab: PropTypes.func.isRequired
   };
 
-  /**
-   * Create a new tab with the same namespace as the last one.
-   */
-  createTab = () => {
-    this.props.createNewTab(
-      this.props.activeNamespace,
-      this.props.activeIsReadonly,
-      this.props.activeSourceName
-    );
-  }
-
   /**
    * Render the Create Tab component.
    *
    * @returns {Component} The rendered component.
    */
   render() {
     return (
-      <div className={styles['create-tab']} onClick={this.createTab}>
+      <div
+        className={styles['create-tab']}
+        onClick={() => this.props.createNewTab()}
+      >
         +
       </div>
     );

packages/compass-collection/src/components/create-tab/create-tab.spec.js

@@ -13,10 +13,8 @@ describe('CreateTab [Component]', () => {
 
     component = mount(
       <CreateTab
-        activeNamespace="db.coll"
-        activeIsReadonly={false}
-        activeSourceName=""
-        createNewTab={createNewTabSpy} />
+        createNewTab={createNewTabSpy}
+      />
     );
   });
 
@@ -32,7 +30,7 @@ describe('CreateTab [Component]', () => {
   context('when clicking the create button', () => {
     it('calls the action', () => {
       component.find(`.${styles['create-tab']}`).simulate('click');
-      expect(createNewTabSpy.calledWith('db.coll')).to.equal(true);
+      expect(createNewTabSpy.called).to.equal(true);
     });
   });
 });

packages/compass-collection/src/components/workspace/workspace.jsx

@@ -41,6 +41,12 @@ const KEY_CLOSE_BRKT = 221;
  */
 const KEY_OPEN_BRKT = 219;
 
+const DEFAULT_NEW_TAB = {
+  namespace: '',
+  isReadonly: false,
+  sourceName: ''
+};
+
 /**
  * The collection workspace contains tabs of multiple collections.
  */
@@ -88,14 +94,31 @@ class Workspace extends PureComponent {
     this.props.moveTab(oldIndex, newIndex);
   }
 
+  onCreateNewTab = () => {
+    const activeTab = this.activeTab();
+    const newTabProps = activeTab
+      ? {
+        namespace: activeTab.namespace,
+        isReadonly: activeTab.isReadonly,
+        isTimeSeries: activeTab.isTimeSeries,
+        sourceName: activeTab.sourceName,
+        editViewName: activeTab.editViewName,
+        sourceReadonly: activeTab.sourceReadonly,
+        sourceViewOn: activeTab.sourceViewOn,
+        sourcePipeline: activeTab.pipeline
+      }
+      : DEFAULT_NEW_TAB;
+    this.props.createNewTab(newTabProps);
+  }
+
   /**
    * Handle key press. This listens for CTRL/CMD+T and CTRL/CMD+W to control
    * natural opening and closing of collection tabs. CTRL/CMD+SHIFT+] and
    * CTRL/CMD+SHIFT+[ to go forward and backwards through the tabs.
    *
    * @param {Event} evt - The event.
    */
-  handleKeypress(evt) {
+  handleKeypress = (evt) => {
     if (evt.ctrlKey || evt.metaKey) {
       if (evt.shiftKey) {
         if (evt.keyCode === KEY_CLOSE_BRKT) {
@@ -109,7 +132,7 @@ class Workspace extends PureComponent {
           evt.preventDefault();
         }
       } else if (evt.keyCode === KEY_T) {
-        this.props.createNewTab(this.activeNamespace());
+        this.onCreateNewTab();
       }
     }
   }
@@ -118,26 +141,6 @@ class Workspace extends PureComponent {
     return this.props.tabs.find(tab => tab.isActive);
   }
 
-  /**
-   * Get the active namespace in the list.
-   *
-   * @returns {String} The active namespace in the list.
-   */
-  activeNamespace() {
-    const activeTab = this.activeTab();
-    return activeTab ? activeTab.namespace : '';
-  }
-
-  activeIsReadonly() {
-    const activeTab = this.activeTab();
-    return activeTab ? activeTab.isReadonly : false;
-  }
-
-  activeSourceName() {
-    const activeTab = this.activeTab();
-    return activeTab ? activeTab.sourceName : undefined;
-  }
-
   renderTab = (tab, i) => {
     return (
       <CollectionTab
@@ -201,6 +204,7 @@ class Workspace extends PureComponent {
           id={tab.id}
           namespace={tab.namespace}
           isReadonly={tab.isReadonly}
+          isTimeSeries={tab.isTimeSeries}
           sourceName={tab.sourceName}
           editViewName={tab.editViewName}
           sourceReadonly={tab.sourceReadonly}
@@ -236,10 +240,8 @@ class Workspace extends PureComponent {
           <div className={styles['workspace-tabs-container']}>
             {this.renderTabs()}
             <CreateTab
-              createNewTab={this.props.createNewTab}
-              activeNamespace={this.activeNamespace()}
-              activeIsReadonly={this.activeIsReadonly()}
-              activeSourceName={this.activeSourceName()} />
+              createNewTab={this.onCreateNewTab}
+            />
           </div>
           <div className={styles['workspace-tabs-right']}>
             <div onClick={this.props.nextTab} className={styles['workspace-tabs-next']}>