cleanup and update selector

Author: paula-stacho

packages/compass-components/src/components/drawer-portal.spec.tsx

@@ -215,14 +215,14 @@ describe('DrawerSection', function () {
     expect(screen.getByTestId('drawer-state')).to.have.text('closed');
 
     // Open the drawer
-    userEvent.click(screen.getByTestId('toggle-drawer'));
+    userEvent.click(screen.getByRole('button', { name: 'Open drawer' }));
     await waitFor(() => {
       expect(screen.getByTestId('drawer-state')).to.have.text('open');
       expect(screen.getByText('This is the controlled section')).to.be.visible;
     });
 
     // Close the drawer
-    userEvent.click(screen.getByTestId('toggle-drawer'));
+    userEvent.click(screen.getByRole('button', { name: 'Close drawer' }));
     await waitFor(() => {
       expect(screen.getByTestId('drawer-state')).to.have.text('closed');
       expect(screen.queryByText('This is the controlled section')).not.to.exist;

packages/compass-data-modeling/src/components/drawer/diagram-editor-side-panel.tsx

@@ -144,12 +144,11 @@ export default connect(
     const model = selectCurrentModelFromState(state);
 
     if (selected.type === 'collection') {
-      const isNewCollection = !selected.id;
       const doesCollectionExist = model.collections.find((collection) => {
         return collection.ns === selected.id;
       });
 
-      if (!isNewCollection && !doesCollectionExist) {
+      if (!doesCollectionExist) {
         // TODO(COMPASS-9680): When the selected collection doesn't exist then we
         // don't show any selection. We can get into this state with undo/redo.
         return {
@@ -160,7 +159,7 @@ export default connect(
       return {
         selectedItems: {
           ...selected,
-          label: selected.id || 'New Collection',
+          label: selected.id,
         },
       };
     }