fixup: use label for selecting button, better ns check

Author: Anemy

packages/compass-data-modeling/src/components/drawer/collection-drawer-content.tsx

@@ -93,8 +93,8 @@ export function getIsCollectionNameValid(
 
   const isDuplicate = namespacesWithoutCurrent.some(
     (ns) =>
-      ns === `${toNS(namespace).database}.${collectionName}` ||
-      ns === `${toNS(namespace).database}.${collectionName.trim()}`
+      ns.trim() ===
+      `${toNS(namespace).database}.${collectionName.trim()}`.trim()
   );
 
   return {

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

@@ -280,9 +280,7 @@ describe('DiagramEditorSidePanel', function () {
       expect(screen.getByLabelText('Name')).to.have.value('countries');
     });
 
-    userEvent.click(
-      screen.getByTestId('data-modeling-drawer-actions-delete-action')
-    );
+    userEvent.click(screen.getByLabelText(/delete collection/i));
 
     await waitFor(() => {
       expect(screen.queryByText('countries')).not.to.exist;

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

@@ -57,7 +57,11 @@ function DiagramEditorSidePanel({
           ></CollectionDrawerContent>
         ),
         actions: [
-          { action: 'delete', label: 'Delete', icon: 'Trash' as const },
+          {
+            action: 'delete',
+            label: 'Delete Collection',
+            icon: 'Trash' as const,
+          },
         ],
         handleAction: (actionName: string) => {
           if (actionName === 'delete') {