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

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)
+  })
+})