remove create capped options

Author: mabaasit

packages/compass-e2e-tests/helpers/commands/add-collection.ts

@@ -2,9 +2,6 @@ import type { CompassBrowser } from '../compass-browser';
 import * as Selectors from '../selectors';
 
 export type AddCollectionOptions = {
-  capped?: {
-    size: number;
-  };
   customCollation?: {
     locale: string;
     strength: number;
@@ -51,15 +48,6 @@ export async function addCollection(
     );
   }
 
-  if (collectionOptions && collectionOptions.capped) {
-    await browser.clickVisible(Selectors.CreateCollectionCappedCheckboxLabel);
-
-    await browser.setValueVisible(
-      Selectors.CreateCollectionCappedSizeInput,
-      collectionOptions.capped.size.toString()
-    );
-  }
-
   if (collectionOptions && collectionOptions.customCollation) {
     await browser.clickVisible(
       Selectors.CreateCollectionCustomCollationCheckboxLabel

packages/compass-e2e-tests/helpers/selectors.ts

@@ -458,10 +458,6 @@ export const CreateCollectionCreateButton =
   '[data-testid="create-collection-modal"] [data-testid="submit-button"]';
 export const CreateCollectionCancelButton =
   '[data-testid="create-collection-modal"] [data-testid="cancel-button"]';
-export const CreateCollectionCappedCheckboxLabel =
-  '[data-testid="capped-collection-fields"] [data-testid="capped-collection-fields-label"]';
-export const CreateCollectionCappedSizeInput =
-  '[data-testid="capped-collection-fields"] [data-testid="capped-size"]';
 export const CreateCollectionCollectionOptionsAccordion =
   '[data-testid="create-collection-modal"] [data-testid="additional-collection-preferences"]';
 export const CreateCollectionCustomCollationCheckboxLabel =

packages/compass-e2e-tests/tests/database-collections-tab.test.ts

@@ -174,39 +174,6 @@ describe('Database collections tab', function () {
     await browser.waitUntilActiveDatabaseTab(DEFAULT_CONNECTION_NAME_1, 'test');
   });
 
-  it('can create a capped collection', async function () {
-    const collectionName = 'my-capped-collection';
-
-    // open the create collection modal from the button at the top
-    await browser.clickVisible(Selectors.DatabaseCreateCollectionButton);
-
-    await browser.addCollection(
-      collectionName,
-      {
-        capped: {
-          size: 1000,
-        },
-      },
-      'add-collection-modal-capped.png'
-    );
-
-    await browser.navigateToDatabaseCollectionsTab(
-      DEFAULT_CONNECTION_NAME_1,
-      'test'
-    );
-
-    const selector = Selectors.collectionCard('test', collectionName);
-    await browser.scrollToVirtualItem(
-      Selectors.CollectionsGrid,
-      selector,
-      'grid'
-    );
-    const collectionCard = browser.$(selector);
-    await collectionCard.waitForDisplayed();
-
-    // TODO: how do we make sure this is really a capped collection?
-  });
-
   it('can create a collection with custom collation', async function () {
     const collectionName = 'my-custom-collation-collection';
 

packages/compass-telemetry/src/telemetry-events.ts

@@ -2253,11 +2253,6 @@ type SwitchViewTypeEvent = ConnectionScopedEvent<{
 type CollectionCreatedEvent = ConnectionScopedEvent<{
   name: 'Collection Created';
   payload: {
-    /**
-     * Indicates whether the collection is capped.
-     */
-    is_capped: boolean;
-
     /**
      * Indicates whether the collection has a custom collation.
      */
@@ -2293,11 +2288,6 @@ type CollectionCreatedEvent = ConnectionScopedEvent<{
 type DatabaseCreatedEvent = ConnectionScopedEvent<{
   name: 'Database Created';
   payload: {
-    /**
-     * Indicates whether the first collection in the database is capped.
-     */
-    is_capped: boolean;
-
     /**
      * Indicates whether the first collection in the database has a custom collation.
      */

packages/databases-collections/src/components/collection-fields/capped-collection-fields.jsx

@@ -15,7 +15,6 @@ const EXPIRE_AFTER_SECONDS_DESCRIPTION =
   'The _id field must be a date or an array that contains date values.';
 
 function ClusteredCollectionFields({
-  isCapped,
   isTimeSeries,
   isClustered,
   clusteredIndex,
@@ -34,7 +33,7 @@ function ClusteredCollectionFields({
   return (
     <CollapsibleFieldSet
       toggled={isClustered}
-      disabled={isTimeSeries || isCapped}
+      disabled={isTimeSeries}
       onToggle={(checked) => onChangeIsClustered(checked)}
       label="Clustered Collection"
       data-testid="clustered-collection-fields"
@@ -72,7 +71,6 @@ function ClusteredCollectionFields({
 }
 
 ClusteredCollectionFields.propTypes = {
-  isCapped: PropTypes.bool.isRequired,
   isTimeSeries: PropTypes.bool.isRequired,
   isClustered: PropTypes.bool.isRequired,
   clusteredIndex: PropTypes.object.isRequired,

packages/databases-collections/src/components/collection-fields/capped-collection-fields.spec.jsx

@@ -14,7 +14,6 @@ describe('ClusteredCollectionFields [Component]', function () {
       component = mount(
         <ClusteredCollectionFields
           isTimeSeries={false}
-          isCapped={false}
           isClustered
           clusteredIndex={{}}
           onChangeIsClustered={() => {}}
@@ -40,7 +39,6 @@ describe('ClusteredCollectionFields [Component]', function () {
       component = mount(
         <ClusteredCollectionFields
           isTimeSeries={false}
-          isCapped={false}
           isClustered={false}
           clusteredIndex={{}}
           onChangeIsClustered={() => {}}
@@ -72,7 +70,6 @@ describe('ClusteredCollectionFields [Component]', function () {
       component = mount(
         <ClusteredCollectionFields
           isTimeSeries={false}
-          isCapped={false}
           isClustered={false}
           clusteredIndex={{}}
           onChangeIsClustered={onChangeSpy}
@@ -97,30 +94,4 @@ describe('ClusteredCollectionFields [Component]', function () {
       expect(onChangeSpy.firstCall.args[0]).to.deep.equal(true);
     });
   });
-
-  describe('when the isCapped prop is true', function () {
-    let component;
-
-    beforeEach(function () {
-      component = mount(
-        <ClusteredCollectionFields
-          isTimeSeries={false}
-          isCapped
-          isClustered={false}
-          clusteredIndex={{}}
-          onChangeIsClustered={() => {}}
-          onChangeField={() => {}}
-          expireAfterSeconds=""
-        />
-      );
-    });
-
-    afterEach(function () {
-      component = null;
-    });
-
-    it('has the clustered checkbox disabled', function () {
-      expect(component.find('Checkbox').props().disabled).to.equal(true);
-    });
-  });
 });