From 1680f5248016b2a29192c2a5a0eaa0a83b742e42 Mon Sep 17 00:00:00 2001
From: pedrobonamin <pedrobonamin@gmail.com>
Date: Mon, 28 Oct 2024 15:53:42 +0100
Subject: [PATCH] fix(core): remove releases menu

---
 packages/sanity/src/core/index.ts             |   1 -
 .../core/releases/components/ReleasesMenu.tsx | 148 ----------
 .../__tests__/ReleasesMenu.test.tsx           | 263 ------------------
 .../src/core/releases/components/index.ts     |   1 -
 .../detail/__tests__/ReleaseReview.test.tsx   |   4 -
 .../detail/__tests__/ReleaseSummary.test.tsx  |   4 -
 6 files changed, 421 deletions(-)
 delete mode 100644 packages/sanity/src/core/releases/components/ReleasesMenu.tsx
 delete mode 100644 packages/sanity/src/core/releases/components/__tests__/ReleasesMenu.test.tsx

diff --git a/packages/sanity/src/core/index.ts b/packages/sanity/src/core/index.ts
index 78e87ca8c41..24882c7d1d6 100644
--- a/packages/sanity/src/core/index.ts
+++ b/packages/sanity/src/core/index.ts
@@ -37,7 +37,6 @@ export {
   ReleaseAvatar,
   ReleaseBadge,
   ReleaseForm,
-  ReleasesMenu,
   useDocumentVersions,
   usePerspective,
   useVersionOperations,
diff --git a/packages/sanity/src/core/releases/components/ReleasesMenu.tsx b/packages/sanity/src/core/releases/components/ReleasesMenu.tsx
deleted file mode 100644
index e9afcda7046..00000000000
--- a/packages/sanity/src/core/releases/components/ReleasesMenu.tsx
+++ /dev/null
@@ -1,148 +0,0 @@
-import {CheckmarkIcon} from '@sanity/icons'
-// eslint-disable-next-line no-restricted-imports -- MenuItem requires props, only supported by @sanity/ui
-import {Box, Flex, Menu, MenuDivider, MenuItem, Spinner, Text} from '@sanity/ui'
-import {memo, type ReactElement, useCallback, useMemo} from 'react'
-import {styled} from 'styled-components'
-
-import {MenuButton, Tooltip} from '../../../ui-components'
-import {useTranslation} from '../../i18n'
-import {type ReleaseDocument} from '../../store/release/types'
-import {usePerspective} from '../hooks'
-import {LATEST} from '../util/const'
-import {ReleaseBadge} from './ReleaseBadge'
-
-const StyledMenu = styled(Menu)`
-  min-width: 200px;
-`
-
-const StyledBox = styled(Box)`
-  overflow: auto;
-  max-height: 200px;
-`
-
-interface BundleListProps {
-  button: ReactElement
-  releases: ReleaseDocument[] | null
-  loading: boolean
-  actions?: ReactElement
-}
-
-/**
- * @internal
- */
-export const ReleasesMenu = memo(function ReleasesMenu(props: BundleListProps): ReactElement {
-  const {releases, loading, actions, button} = props
-  const {currentGlobalBundle, setPerspectiveFromRelease} = usePerspective()
-  const {t} = useTranslation()
-
-  const sortedBundlesToDisplay = useMemo(() => {
-    if (!releases) return []
-
-    return releases.filter(({state}) => state !== 'archived')
-  }, [releases])
-  const hasBundles = sortedBundlesToDisplay.length > 0
-
-  const handleReleaseChange = useCallback(
-    (releaseId: string) => () => {
-      setPerspectiveFromRelease(releaseId)
-    },
-    [setPerspectiveFromRelease],
-  )
-
-  return (
-    <>
-      <MenuButton
-        button={button}
-        id="release-menu"
-        menu={
-          <StyledMenu data-testid="release-menu">
-            {loading ? (
-              <Flex padding={4} justify="center" data-testid="spinner">
-                <Spinner muted />
-              </Flex>
-            ) : (
-              <>
-                <MenuItem
-                  iconRight={
-                    currentGlobalBundle._id === LATEST._id ? (
-                      <CheckmarkIcon data-testid="latest-checkmark-icon" />
-                    ) : undefined
-                  }
-                  onClick={handleReleaseChange('drafts')}
-                  pressed={false}
-                  text={LATEST.metadata.title}
-                  data-testid="latest-menu-item"
-                />
-                {hasBundles && (
-                  <>
-                    <MenuDivider />
-                    <StyledBox data-testid="releases-list">
-                      {sortedBundlesToDisplay.map((release) => (
-                        <MenuItem
-                          key={release._id}
-                          onClick={handleReleaseChange(release._id)}
-                          padding={1}
-                          pressed={false}
-                          data-testid={`release-${release._id}`}
-                        >
-                          <Tooltip content={t('release.deleted-tooltip')} placement="bottom-start">
-                            <Flex>
-                              <ReleaseBadge
-                                hue={release.metadata.hue}
-                                icon={release.metadata.icon}
-                                padding={2}
-                              />
-
-                              <Box flex={1} padding={2} style={{minWidth: 100}}>
-                                <Text size={1} weight="medium">
-                                  {release.metadata.title}
-                                </Text>
-                              </Box>
-
-                              {/*<Box padding={2}>
-                                <Text muted size={1}>
-                                  {release.publishAt ? (
-                                    <RelativeTime time={release.publishAt as Date} useTemporalPhrase />
-                                  ) : (
-                                    'No target date'
-                                  )}
-                                </Text>
-                              </Box>*/}
-
-                              <Box padding={2}>
-                                <Text size={1}>
-                                  <CheckmarkIcon
-                                    style={{
-                                      opacity: currentGlobalBundle._id === release._id ? 1 : 0,
-                                    }}
-                                    data-testid={`${release._id}-checkmark-icon`}
-                                  />
-                                </Text>
-                              </Box>
-                            </Flex>
-                          </Tooltip>
-                        </MenuItem>
-                      ))}
-                    </StyledBox>
-                  </>
-                )}
-
-                {actions && (
-                  <>
-                    <MenuDivider />
-                    {actions}
-                  </>
-                )}
-              </>
-            )}
-          </StyledMenu>
-        }
-        popover={{
-          placement: 'bottom-start',
-          portal: true,
-          zOffset: 3000,
-        }}
-      />
-    </>
-  )
-})
diff --git a/packages/sanity/src/core/releases/components/__tests__/ReleasesMenu.test.tsx b/packages/sanity/src/core/releases/components/__tests__/ReleasesMenu.test.tsx
deleted file mode 100644
index 7dd1bac1ac0..00000000000
--- a/packages/sanity/src/core/releases/components/__tests__/ReleasesMenu.test.tsx
+++ /dev/null
@@ -1,263 +0,0 @@
-import {fireEvent, render, screen, within} from '@testing-library/react'
-import userEvent from '@testing-library/user-event'
-import {act} from 'react'
-import {beforeEach, describe, expect, it, type Mock, vi} from 'vitest'
-
-import {createTestProvider} from '../../../../../test/testUtils/TestProvider'
-import {Button} from '../../../../ui-components'
-import {type ReleaseDocument, useReleases} from '../../../store'
-import {RELEASE_DOCUMENT_TYPE} from '../../../store/release/constants'
-import {usePerspective} from '../../hooks/usePerspective'
-import {LATEST} from '../../util/const'
-import {ReleasesMenu} from '../ReleasesMenu'
-
-vi.mock('../../hooks/usePerspective', () => ({
-  usePerspective: vi.fn().mockReturnValue({
-    currentGlobalBundle: {},
-    setPerspectiveFromRelease: vi.fn(),
-  }),
-}))
-
-vi.mock('../../util/util', () => ({
-  isDraftOrPublished: vi.fn(),
-}))
-
-vi.mock('../../../store/release/useReleases', () => ({
-  useReleases: vi.fn().mockReturnValue({}),
-}))
-
-const mockUseReleases = useReleases as Mock<typeof useReleases>
-
-const TEST_RELEASE: ReleaseDocument = {
-  _id: 'some-id',
-  _type: RELEASE_DOCUMENT_TYPE,
-  // yesterday
-  _createdAt: '2024-10-22T13:18:59.207Z',
-  _updatedAt: '2024-10-22T13:18:59.207Z',
-  name: 'Release Foo',
-  state: 'active',
-  finalDocumentStates: [],
-  createdBy: 'User 1',
-  metadata: {
-    icon: 'drop',
-    description: '',
-    title: '',
-    releaseType: 'asap',
-    hue: 'magenta',
-  },
-}
-
-describe('ReleasesMenu', () => {
-  const mockUsePerspective = usePerspective as Mock
-  const ButtonTest = <Button text="Button Test" />
-  const mockReleases: ReleaseDocument[] = [
-    {
-      ...TEST_RELEASE,
-      _id: 'spring-drop',
-      metadata: {
-        ...TEST_RELEASE.metadata,
-        description: 'What a spring drop, allergies galore 🌸',
-        title: 'Spring Drop',
-      },
-    },
-    {
-      ...TEST_RELEASE,
-      _id: 'autumn-drop',
-      metadata: {
-        ...TEST_RELEASE.metadata,
-        icon: 'drop',
-        title: 'Autumn Drop',
-        hue: 'yellow',
-        releaseType: 'asap',
-      },
-    },
-    {
-      ...TEST_RELEASE,
-      _id: 'f6b2c2cc-1732-4465-bfb3-dd205b5d78e9',
-      metadata: {
-        ...TEST_RELEASE.metadata,
-        title: 'Summer Drop',
-        hue: 'red',
-        createdBy: '',
-        releaseType: 'asap',
-      },
-    },
-  ]
-
-  beforeEach(() => {
-    mockUsePerspective.mockClear()
-  })
-
-  it('should render loading spinner when loading is true', async () => {
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={[]} loading />, {
-      wrapper,
-    })
-
-    expect(screen.getByRole('button', {name: 'Button Test'})).toBeInTheDocument()
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByTestId('release-menu')).toBeInTheDocument()
-      expect(screen.getByTestId('spinner')).toBeInTheDocument()
-    })
-  })
-
-  it('should render latest release menu item when releases are null', async () => {
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={null} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByTestId('latest-menu-item')).toBeInTheDocument()
-      expect(screen.queryByTestId('releases-list')).not.toBeInTheDocument()
-    })
-  })
-
-  it('should render latest release menu item when releases are archived', async () => {
-    const wrapper = await createTestProvider()
-    const archivedBundles = mockReleases.map(
-      (release): ReleaseDocument => ({
-        ...release,
-        state: 'archived',
-      }),
-    )
-    render(<ReleasesMenu button={ButtonTest} releases={archivedBundles} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByTestId('latest-menu-item')).toBeInTheDocument()
-      expect(screen.queryByTestId('releases-list')).not.toBeInTheDocument()
-    })
-  })
-
-  it('should render latest release menu item as selected when currentGlobalBundle is LATEST', async () => {
-    mockUsePerspective.mockReturnValue({
-      currentGlobalBundle: LATEST,
-      setPerspectiveFromRelease: vi.fn(),
-    })
-
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={[]} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByTestId('latest-menu-item')).toBeInTheDocument()
-      expect(screen.getByTestId('latest-checkmark-icon')).toBeInTheDocument()
-    })
-  })
-
-  it('should render release as selected when currentGlobalBundle is that release', async () => {
-    mockUsePerspective.mockReturnValue({
-      currentGlobalBundle: mockReleases[0],
-      setPerspectiveFromRelease: vi.fn(),
-    })
-
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={mockReleases} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByText(mockReleases[0].metadata.title)).toBeInTheDocument()
-      expect(screen.getByTestId(`${mockReleases[0]._id}-checkmark-icon`)).toBeInTheDocument()
-    })
-  })
-
-  it('should render release menu items when releases are provided', async () => {
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={mockReleases} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByText('Spring Drop')).toBeInTheDocument()
-      expect(screen.getByText('Autumn Drop')).toBeInTheDocument()
-      expect(screen.getByText('Summer Drop')).toBeInTheDocument()
-    })
-  })
-
-  it('should call setPerspective when a release menu item is clicked', async () => {
-    const setPerspectiveFromRelease = vi.fn()
-    mockUsePerspective.mockReturnValue({
-      currentGlobalBundle: LATEST,
-      setPerspectiveFromRelease,
-    })
-
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={mockReleases} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      userEvent.click(screen.getByTestId('release-spring-drop'))
-      expect(setPerspectiveFromRelease).toHaveBeenCalledWith('spring-drop')
-    })
-  })
-
-  it('should render actions when actions prop is provided', async () => {
-    const actions = <Button text="Actions" />
-
-    const wrapper = await createTestProvider()
-    render(
-      <ReleasesMenu
-        button={ButtonTest}
-        releases={mockReleases}
-        loading={false}
-        actions={actions}
-      />,
-      {
-        wrapper,
-      },
-    )
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    act(() => {
-      expect(screen.getByRole('button', {name: 'Actions'})).toBeInTheDocument()
-    })
-  })
-
-  it('should not show archived releases when not included in the list', async () => {
-    mockUseReleases.mockReturnValue({
-      dispatch: vi.fn(),
-      loading: false,
-      data: [
-        {
-          _id: 'mock-deleted-release',
-          _type: 'system-tmp.release',
-          state: 'archived',
-          metadata: {
-            title: 'Mock Archived Bundle',
-          },
-        } as ReleaseDocument,
-      ],
-    })
-    const wrapper = await createTestProvider()
-    render(<ReleasesMenu button={ButtonTest} releases={mockReleases} loading={false} />, {
-      wrapper,
-    })
-
-    fireEvent.click(screen.getByRole('button', {name: 'Button Test'}))
-
-    expect(
-      within(screen.getByTestId('releases-list')).queryByText('Mock Archived Bundle'),
-    ).not.toBeInTheDocument()
-  })
-})
diff --git a/packages/sanity/src/core/releases/components/index.ts b/packages/sanity/src/core/releases/components/index.ts
index 66e95047a1f..e6d1860734c 100644
--- a/packages/sanity/src/core/releases/components/index.ts
+++ b/packages/sanity/src/core/releases/components/index.ts
@@ -2,4 +2,3 @@ export * from './dialog/DiscardVersionDialog'
 export * from './dialog/ReleaseForm'
 export * from './ReleaseAvatar'
 export * from './ReleaseBadge'
-export * from './ReleasesMenu'
diff --git a/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseReview.test.tsx b/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseReview.test.tsx
index 35f76ac3b37..19b26ff4f09 100644
--- a/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseReview.test.tsx
+++ b/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseReview.test.tsx
@@ -132,10 +132,6 @@ vi.mock('sanity/router', async (importOriginal) => ({
   }),
 }))
 
-vi.mock('../../../components/ReleasesMenu', () => ({
-  ReleasesMenu: () => <div>ReleasesMenu</div>,
-}))
-
 vi.mock('../../../../preview/useObserveDocument', () => {
   return {
     useObserveDocument: vi.fn(),
diff --git a/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseSummary.test.tsx b/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseSummary.test.tsx
index 8d6147ec7d4..6705fd871bf 100644
--- a/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseSummary.test.tsx
+++ b/packages/sanity/src/core/releases/tool/detail/__tests__/ReleaseSummary.test.tsx
@@ -24,10 +24,6 @@ vi.mock('../../../../user-color', () => ({
   useUserColor: vi.fn().mockReturnValue('red'),
 }))
 
-vi.mock('../../../components/ReleasesMenu', () => ({
-  ReleasesMenu: () => <div>ReleasesMenu</div>,
-}))
-
 const timeNow = new Date()
 
 const releaseDocuments: DocumentInRelease[] = [