@faststore/core 4.10.1-dev.5 → 4.10.1-dev.7

package/test/components/account/OrganizationDrawer.browser.test.tsx

@@ -108,7 +108,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -124,7 +124,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -141,7 +141,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -152,7 +152,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen={false}
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -160,7 +160,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -181,7 +181,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -200,7 +200,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -215,7 +215,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={vi.fn()}
-        isRepresentative
+        isB2BBuyer
         initialView="switch"
       />
     )
@@ -258,7 +258,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -301,7 +301,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -313,7 +313,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen={false}
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -323,7 +323,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 
@@ -340,7 +340,7 @@ describe('OrganizationDrawer', () => {
       <OrganizationDrawer
         isOpen
         closeDrawer={mockCloseDrawer}
-        isRepresentative={false}
+        isB2BBuyer={false}
       />
     )
 

package/test/components/account/OrganizationDrawerBody.test.tsx

@@ -27,7 +27,7 @@ describe('OrganizationDrawerBody', () => {
   it('hides the Quotes route when the user is not an org member', () => {
     mockUseAccountNavigationLabels.mockReturnValue(undefined)
 
-    render(<OrganizationDrawerBody isRepresentative isOrgMember={false} />)
+    render(<OrganizationDrawerBody isB2BBuyer isOrgMember={false} />)
 
     expect(screen.queryByText('Quotes')).toBeNull()
     expect(screen.getByText('Profile')).toBeTruthy()
@@ -36,7 +36,7 @@ describe('OrganizationDrawerBody', () => {
   it('shows the Quotes route when the user belongs to an org unit', () => {
     mockUseAccountNavigationLabels.mockReturnValue(undefined)
 
-    render(<OrganizationDrawerBody isRepresentative isOrgMember />)
+    render(<OrganizationDrawerBody isB2BBuyer isOrgMember />)
 
     expect(screen.getByText('Quotes')).toBeTruthy()
   })
@@ -44,7 +44,7 @@ describe('OrganizationDrawerBody', () => {
   it('hides User Details when not a representative, regardless of org membership', () => {
     mockUseAccountNavigationLabels.mockReturnValue(undefined)
 
-    render(<OrganizationDrawerBody isRepresentative={false} isOrgMember />)
+    render(<OrganizationDrawerBody isB2BBuyer={false} isOrgMember />)
 
     expect(screen.queryByText('User Details')).toBeNull()
     expect(screen.getByText('Quotes')).toBeTruthy()

package/test/components/account/OrganizationSignInButton.browser.test.tsx

@@ -2,10 +2,13 @@
  * @vitest-environment jsdom
  */
 import '@testing-library/jest-dom/vitest'
-import { render, screen } from '@testing-library/react'
+import { fireEvent, render, screen } from '@testing-library/react'
 import { describe, expect, it, vi } from 'vitest'
 
 const mockUseSession = vi.hoisted(() => vi.fn())
+const mockOrganizationDrawer = vi.hoisted(() =>
+  vi.fn((_props: Record<string, unknown>) => null)
+)
 vi.mock('src/sdk/session', () => ({ useSession: mockUseSession }))
 vi.mock('src/sdk/account/accountPageContext', () => ({
   useAccountNavigationLabels: () => ({
@@ -16,7 +19,7 @@ vi.mock('src/sdk/account/accountPageContext', () => ({
 vi.mock(
   'src/components/account/Drawer/OrganizationDrawer/OrganizationDrawer',
   () => ({
-    OrganizationDrawer: () => null,
+    OrganizationDrawer: mockOrganizationDrawer,
   })
 )
 
@@ -30,7 +33,7 @@ describe('OrganizationSignInButton', () => {
       person: { id: 'p1' },
       b2b: { contractName: ' Caetano Corp ' },
     })
-    render(<OrganizationSignInButton icon={icon} isRepresentative />)
+    render(<OrganizationSignInButton icon={icon} isB2BBuyer />)
     expect(screen.getByRole('button', { name: 'account' })).toHaveTextContent(
       'Caetano Corp'
     )
@@ -41,9 +44,21 @@ describe('OrganizationSignInButton', () => {
       person: { id: 'p1' },
       b2b: { contractName: '' },
     })
-    render(<OrganizationSignInButton icon={icon} isRepresentative />)
+    render(<OrganizationSignInButton icon={icon} isB2BBuyer />)
     expect(screen.getByRole('button', { name: 'account' })).toHaveTextContent(
       'Company'
     )
   })
+
+  it.each([
+    ['isB2BBuyer', { isB2BBuyer: true }],
+    ['the deprecated isRepresentative', { isRepresentative: true }],
+  ])('opens the drawer as a B2B buyer from %s', (_label, props) => {
+    mockUseSession.mockReturnValue({ person: { id: 'p1' }, b2b: {} })
+    render(<OrganizationSignInButton icon={icon} {...props} />)
+    fireEvent.click(screen.getByRole('button', { name: 'account' }))
+    expect(mockOrganizationDrawer.mock.lastCall?.[0]).toMatchObject({
+      isB2BBuyer: true,
+    })
+  })
 })

package/test/components/navigation/Navbar.browser.test.tsx

@@ -146,11 +146,11 @@ describe('Navbar', () => {
     )
   })
 
-  it('forwards isSignInResolved=true and isOrganizationEnabled for a resolved B2B representative session', () => {
+  it('forwards isSignInResolved=true and isOrganizationEnabled for a resolved B2B session', () => {
     mockStoreConfig.experimental.enableFaststoreMyAccount = true
     mockUseScreenResize.mockReturnValue({ isDesktop: true })
     mockUseSession.mockReturnValue({
-      b2b: { isRepresentative: true },
+      b2b: { unitId: 'unit-1', customerId: 'contract-1' },
       isSessionReady: true,
       hasValidated: true,
     })
@@ -161,7 +161,7 @@ describe('Navbar', () => {
       expect.objectContaining({
         isSignInResolved: true,
         isOrganizationEnabled: true,
-        isRepresentative: true,
+        isB2BBuyer: true,
       })
     )
 

package/test/components/navigation/NavbarAccountArea.browser.test.tsx

@@ -11,17 +11,17 @@ vi.mock(
   () => ({
     OrganizationSignInButton: ({
       icon,
-      isRepresentative,
+      isB2BBuyer,
     }: {
       icon: { alt: string; icon: string }
-      isRepresentative?: boolean
+      isB2BBuyer?: boolean
     }) => (
       <button
         type="button"
         data-testid="organization-signin"
         data-icon={icon.alt}
       >
-        {isRepresentative ? 'Representative' : 'Not representative'}
+        {isB2BBuyer ? 'B2B buyer' : 'Not a B2B buyer'}
       </button>
     ),
   })
@@ -85,7 +85,7 @@ describe('NavbarAccountArea', () => {
       <NavbarAccountArea
         isSignInResolved
         isOrganizationEnabled
-        isRepresentative
+        isB2BBuyer
         signInButton={signInButton}
         ButtonSignIn={ButtonSignInStub}
         skeletonAttr="data-fs-navbar-signin-skeleton"
@@ -93,7 +93,7 @@ describe('NavbarAccountArea', () => {
     )
 
     expect(screen.getByTestId('organization-signin')).toHaveTextContent(
-      'Representative'
+      'B2B buyer'
     )
     expect(screen.queryByTestId('storefront-signin')).toBeNull()
   })

package/test/components/navigation/NavbarSlider.browser.test.tsx

@@ -120,10 +120,10 @@ describe('NavbarSlider', () => {
     )
   })
 
-  it('forwards isSignInResolved=true and isOrganizationEnabled once the B2B representative session resolves', () => {
+  it('forwards isSignInResolved=true and isOrganizationEnabled once the B2B session resolves', () => {
     mockStoreConfig.experimental.enableFaststoreMyAccount = true
     mockUseSession.mockReturnValue({
-      b2b: { isRepresentative: true },
+      b2b: { unitId: 'unit-1', customerId: 'contract-1' },
       isSessionReady: true,
       hasValidated: true,
     })
@@ -134,7 +134,7 @@ describe('NavbarSlider', () => {
       expect.objectContaining({
         isSignInResolved: true,
         isOrganizationEnabled: true,
-        isRepresentative: true,
+        isB2BBuyer: true,
       })
     )
 

package/test/experimental/myAccountCmsServerSideProps.test.ts

@@ -2,7 +2,7 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 const mockValidateUser = vi.fn()
 const mockGetMyAccountRedirect = vi.fn()
-const mockGetIsRepresentative = vi.fn()
+const mockGetIsB2BBuyer = vi.fn()
 const mockFetchMyAccountPageContent = vi.fn()
 const mockGetGlobalSectionsData = vi.fn()
 const mockExecute = vi.fn()
@@ -20,8 +20,8 @@ vi.mock('src/utils/myAccountRedirect', () => ({
     mockGetMyAccountRedirect(...args),
 }))
 
-vi.mock('src/sdk/account/getIsRepresentative', () => ({
-  getIsRepresentative: (...args: unknown[]) => mockGetIsRepresentative(...args),
+vi.mock('src/sdk/account/getIsB2BBuyer', () => ({
+  getIsB2BBuyer: (...args: unknown[]) => mockGetIsB2BBuyer(...args),
 }))
 
 vi.mock('src/server/cms/fetchMyAccountPageContent', () => ({
@@ -78,7 +78,7 @@ function buildContext(overrides: Record<string, unknown> = {}) {
 describe('myAccountCmsServerSideProps', () => {
   beforeEach(() => {
     vi.clearAllMocks()
-    mockGetIsRepresentative.mockReturnValue(false)
+    mockGetIsB2BBuyer.mockReturnValue(false)
     mockGetGlobalSectionsData.mockReturnValue([
       Promise.resolve({ sections: [], settings: {} }),
       Promise.resolve({ sections: [] }),
@@ -176,7 +176,7 @@ describe('myAccountCmsServerSideProps', () => {
     expect(result).toMatchObject({
       props: {
         accountName: 'Ada',
-        isRepresentative: false,
+        isB2BBuyer: false,
         navigationLabels: { profileLabel: 'Profile' },
         pageSections: [
           {

package/test/pages/api/logout.test.ts

@@ -0,0 +1,106 @@
+import type { NextApiRequest, NextApiResponse } from 'next'
+import { beforeEach, describe, expect, it, vi, type Mock } from 'vitest'
+
+const mockConfig = vi.hoisted(() => ({
+  default: {
+    experimental: { refreshToken: true } as Record<string, unknown>,
+  } as { experimental: Record<string, unknown>; secureSubdomain?: string },
+}))
+
+vi.mock('discovery.config', () => ({ __esModule: true, ...mockConfig }))
+
+function createReqRes(cookie = 'VtexIdclientAutCookie_test-store=token') {
+  const res = {
+    status: vi.fn().mockReturnThis(),
+    json: vi.fn(),
+    end: vi.fn(),
+    setHeader: vi.fn(),
+  }
+
+  return {
+    req: {
+      method: 'POST',
+      headers: { host: 'store.example.com', cookie },
+    } as unknown as NextApiRequest,
+    res: res as unknown as NextApiResponse & { setHeader: Mock },
+  }
+}
+
+const setCookiesFrom = (res: { setHeader: Mock }): string[] =>
+  (res.setHeader.mock.calls.find(
+    ([name]) => String(name).toLowerCase() === 'set-cookie'
+  )?.[1] as string[] | undefined) ?? []
+
+async function getHandler() {
+  const mod = await import('../../../src/pages/api/fs/logout')
+  return mod.default
+}
+
+describe('/api/fs/logout', () => {
+  beforeEach(() => {
+    vi.resetModules()
+    mockConfig.default.experimental = { refreshToken: true }
+    mockConfig.default.secureSubdomain = undefined
+  })
+
+  it('expires vid_rt on every path it can be scoped to, even though the request never carries it', async () => {
+    const handler = await getHandler()
+    const { req, res } = createReqRes()
+
+    await handler(req, res)
+
+    const vidRt = setCookiesFrom(res).filter((cookie) =>
+      cookie.startsWith('vid_rt=;')
+    )
+
+    for (const path of [
+      '/api/vtexid/refreshtoken/webstore',
+      '/api/authenticator/refreshtoken/webstore',
+      '/api/authenticator/v1/refreshtoken/webstore',
+    ]) {
+      expect(vidRt.some((cookie) => cookie.includes(`path=${path}`))).toBe(true)
+    }
+  })
+
+  it('does not touch vid_rt when refresh token is disabled', async () => {
+    mockConfig.default.experimental = {}
+    const handler = await getHandler()
+    const { req, res } = createReqRes()
+
+    await handler(req, res)
+
+    expect(
+      setCookiesFrom(res).some((cookie) => cookie.startsWith('vid_rt='))
+    ).toBe(false)
+  })
+
+  it('expires the auth cookie on the parent shared with a sibling checkout host', async () => {
+    mockConfig.default.secureSubdomain = 'https://secure.example.com'
+    const handler = await getHandler()
+    const { req, res } = createReqRes()
+
+    await handler(req, res)
+
+    expect(
+      setCookiesFrom(res).filter((cookie) =>
+        cookie.startsWith('VtexIdclientAutCookie_test-store=;')
+      )
+    ).toEqual(
+      expect.arrayContaining([expect.stringContaining('; domain=example.com;')])
+    )
+  })
+
+  it('does not touch the parent domain when checkout runs on the store host', async () => {
+    mockConfig.default.secureSubdomain = 'https://store.example.com'
+    const handler = await getHandler()
+    const { req, res } = createReqRes()
+
+    await handler(req, res)
+
+    expect(
+      setCookiesFrom(res).some((cookie) =>
+        cookie.includes('; domain=example.com;')
+      )
+    ).toBe(false)
+  })
+})