(PC-33349)[PRO] fix: The app preview should not display the venue det…

…ails.
pass-culture · Jan 13, 2025 · acbfc0d · acbfc0d
1 parent 2bd9a89
commit acbfc0d
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 11 deletions.
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx
@@ -235,7 +235,7 @@ export const OfferSection = ({
               descriptions={[
                 {
                   title: 'Intitulé',
-                  text: offerData.address?.label,
+                  text: offerData.address?.label || '-',
                 },
                 {
                   title: 'Adresse',

diff --git a/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx b/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx
@@ -663,6 +663,38 @@ describe('Summary', () => {
       expect(screen.getAllByText('mon adresse')).toHaveLength(2)
       expect(screen.getAllByText('ma street 1 ma ville')).toHaveLength(2)
     })
+
+    it('should render component with new sections and empty address data', async () => {
+      vi.spyOn(api, 'getOfferer').mockResolvedValue(
+        defaultGetOffererResponseModel
+      )
+      customContext.offer = getIndividualOfferFactory({
+        isEvent: true,
+        address: null,
+      })
+
+      renderSummary(
+        customContext,
+        generatePath(
+          getIndividualOfferPath({
+            step: OFFER_WIZARD_STEP_IDS.SUMMARY,
+            mode: OFFER_WIZARD_MODE.CREATION,
+          }),
+          { offerId: 'AA' }
+        ),
+        { features: ['WIP_ENABLE_OFFER_ADDRESS'] }
+      )
+
+      expect(await screen.findByText(/Structure/)).toBeInTheDocument()
+      expect(
+        await screen.findByText('Localisation de l’offre')
+      ).toBeInTheDocument()
+
+      expect(screen.getByText('Intitulé : -')).toBeInTheDocument()
+      expect(screen.getByText('Adresse : -')).toBeInTheDocument()
+
+      expect(screen.getAllByText('-')).toHaveLength(9)
+    })
   })
 
   describe('banners', () => {

diff --git a/pro/src/components/OfferAppPreview/VenueDetails/VenueDetails.tsx b/pro/src/components/OfferAppPreview/VenueDetails/VenueDetails.tsx
@@ -2,6 +2,7 @@ import {
   AddressResponseIsLinkedToVenueModel,
   GetOfferVenueResponseModel,
 } from 'apiClient/v1'
+import { useActiveFeature } from 'commons/hooks/useActiveFeature'
 import { computeAddressDisplayName } from 'repository/venuesService'
 
 import style from './VenueDetails.module.scss'
@@ -17,18 +18,35 @@ export const VenueDetails = ({
   address,
   withdrawalDetails,
 }: VenueDetailsProps): JSX.Element => {
-  const { street, postalCode, city } = address || venue
+  const isOfferAddressEnabled = useActiveFeature('WIP_ENABLE_OFFER_ADDRESS')
 
-  const label = address ? address.label || '' : venue.publicName || venue.name
+  function computeAddress() {
+    let venueAddressString = '-'
+    let label = '-'
 
-  const venueAddressString = computeAddressDisplayName(
-    {
-      street,
-      postalCode: postalCode || '',
-      city: city || '',
-    },
-    false
-  )
+    if (!isOfferAddressEnabled) {
+      label = venue.publicName || venue.name
+      venueAddressString = [label, venue.street, venue.postalCode, venue.city]
+        .filter((str) => Boolean(str))
+        .join(' - ')
+    } else if (address) {
+      const { street, postalCode, city } = address
+
+      label = address.label || '-'
+
+      venueAddressString = computeAddressDisplayName(
+        {
+          street,
+          postalCode: postalCode || '',
+          city: city || '',
+        },
+        false
+      )
+    }
+    return { label, venueAddressString }
+  }
+
+  const { label, venueAddressString } = computeAddress()
 
   return (
     <div className={style['venue-details']}>