[SECURITY] Copy saved object flyout should not allow copying into the…

… active space (#116657) (#116818) * copy saved object flyout should not allow copying into the active space * stupid me Co-authored-by: Xavier Mouligneau <189600+XavierM@users.noreply.github.com>
elastic · Oct 29, 2021 · 15efa9d · 15efa9d
1 parent 41de838
commit 15efa9d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/...aces/public/copy_saved_objects_to_space/components/copy_to_space_flyout_internal.test.tsx b/...aces/public/copy_saved_objects_to_space/components/copy_to_space_flyout_internal.test.tsx
@@ -112,8 +112,7 @@ const setup = async (opts: SetupOpts = {}) => {
   return { wrapper, onClose, mockSpacesManager, mockToastNotifications, savedObjectToCopy };
 };
 
-// flaky https://github.com/elastic/kibana/issues/96708
-describe.skip('CopyToSpaceFlyout', () => {
+describe('CopyToSpaceFlyout', () => {
   it('waits for spaces to load', async () => {
     const { wrapper } = await setup({ returnBeforeSpacesLoad: true });
 

diff --git a/...ns/spaces/public/copy_saved_objects_to_space/components/copy_to_space_flyout_internal.tsx b/...ns/spaces/public/copy_saved_objects_to_space/components/copy_to_space_flyout_internal.tsx
@@ -75,7 +75,7 @@ export const CopyToSpaceFlyoutInternal = (props: CopyToSpaceFlyoutProps) => {
           isLoading: false,
           spaces: [...spacesMap.values()].filter(
             ({ isActiveSpace, isAuthorizedForPurpose }) =>
-              isActiveSpace || isAuthorizedForPurpose('copySavedObjectsIntoSpace')
+              !isActiveSpace && isAuthorizedForPurpose('copySavedObjectsIntoSpace')
           ),
         });
       })