Merge pull request #2778 from SalesforceCommerceCloud/sz-W-19007043-comment-attribution

@W-19007043 - remove AI attribution comments PR 2

Author: szirpesf

.cursor/rules/testing/unit-tests-generic.mdc

@@ -7,22 +7,6 @@ USE WHEN writing unit tests for components in template packages
 
 # 🧪 Generic Component Test Rules
 
-# CRITICAL: AI Attribution Requirements
-* **IMPORTANT** All individual test methods generated or modified by Cursor MUST include an AI attribution comment directly above the test stating the following: 
-"DO NOT REMOVE THIS COMMENT! This test was generated by Cursor". The comment should go on the test method and not the test class. Failure to add an AI Attribution Comment will be considered a failure of test generation.
-* The AI attribution comment MUST include a comment declaring the LLM model that was used in writing the test on its own line.
-
-*Sample AI Attribution Comment*
-```
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor 
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
-test('renders component correctly', () => {
-    // test implementation
-})
-```
-
 ## Structure & Best Practices
 - Use `describe` blocks to group tests, `test` for individual cases
 - Use `beforeEach` for setup, clear mocks after each test

packages/pwa-storefront-mcp/src/utils/utils.test.js

@@ -31,19 +31,11 @@ describe('Utils', () => {
     })
 
     describe('isMonoRepo', () => {
-        /*
-         * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-         * This test was generated with the following model: GPT-4.1
-         */
         test('returns true if lerna.json exists', () => {
             jest.spyOn(fs, 'existsSync').mockReturnValueOnce(true)
             expect(isMonoRepo()).toBe(true)
         })
 
-        /*
-         * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-         * This test was generated with the following model: GPT-4.1
-         */
         test('returns false if lerna.json does not exist', () => {
             jest.spyOn(fs, 'existsSync').mockReturnValueOnce(false)
             expect(isMonoRepo()).toBe(false)

packages/template-retail-react-app/app/components/drawer-menu/drawer-menu.test.js

@@ -73,10 +73,6 @@ describe('DrawerMenu', () => {
         expect(spinner).toBeInTheDocument()
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('renders drawer in closed state', () => {
         renderWithProviders(<DrawerMenu isOpen={false} root={mockCategories.root} />)
 
@@ -85,10 +81,6 @@ describe('DrawerMenu', () => {
         expect(drawerContent).not.toBeInTheDocument()
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('calls onClose when close button is clicked', async () => {
         const {user} = renderWithProviders(
             <DrawerMenu isOpen={true} root={mockCategories.root} onClose={mockOnClose} />
@@ -100,10 +92,6 @@ describe('DrawerMenu', () => {
         expect(mockOnClose).toHaveBeenCalledTimes(1)
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('calls onLogoClick when logo is clicked', async () => {
         const {user} = renderWithProviders(
             <DrawerMenu isOpen={true} root={mockCategories.root} onLogoClick={mockOnLogoClick} />
@@ -116,10 +104,6 @@ describe('DrawerMenu', () => {
         expect(mockOnLogoClick).toHaveBeenCalledTimes(1)
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('displays sign in link for guest users', () => {
         useCustomerType.mockReturnValue({isRegistered: false})
 
@@ -132,10 +116,6 @@ describe('DrawerMenu', () => {
         expect(signInLink).toHaveClass('chakra-link')
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('displays user account menu for registered users', () => {
         useCustomerType.mockReturnValue({isRegistered: true})
 
@@ -145,10 +125,6 @@ describe('DrawerMenu', () => {
         expect(myAccountButton).toBeInTheDocument()
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('expands user account menu and shows account options', async () => {
         useCustomerType.mockReturnValue({isRegistered: true})
 
@@ -164,10 +140,6 @@ describe('DrawerMenu', () => {
         })
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('handles logout action for registered users', async () => {
         useCustomerType.mockReturnValue({isRegistered: true})
 
@@ -187,10 +159,6 @@ describe('DrawerMenu', () => {
         })
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('renders category navigation when root categories are provided', () => {
         renderWithProviders(<DrawerMenu isOpen={true} root={mockCategories.root} />)
 
@@ -201,10 +169,6 @@ describe('DrawerMenu', () => {
         expect(categoryNav).toHaveAttribute('aria-atomic', 'true')
     })
 
-    /*
-     * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
-     * This test was generated with the following model: Claude Sonnet 4
-     */
     test('renders with custom itemsKey and itemsCountKey props', () => {
         const customRoot = {
             customItems: [{id: 'test-item', name: 'Test Item', customItems: []}]

packages/template-retail-react-app/app/components/header/index.test.js

@@ -227,29 +227,17 @@ test('shows dropdown menu when an authenticated users hover on the account icon'
     })
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('handles sign out functionality correctly', async () => {
     // Skip this complex test - focus on simpler testable behaviors
     expect(true).toBe(true)
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('shows loading spinner during sign out process', async () => {
     // Skip this test as it requires complex auth setup
     // Focus on testing the loading state logic separately
     expect(true).toBe(true)
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('handles keyboard navigation with Tab+Shift in account menu', async () => {
     // Test keyboard navigation without requiring auth
     const {user} = renderWithProviders(<Header />)
@@ -261,10 +249,6 @@ test('handles keyboard navigation with Tab+Shift in account menu', async () => {
     expect(accountIcon).toBeInTheDocument()
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('handles mouse leave events without crashing', async () => {
     const {user} = renderWithProviders(<Header />)
 
@@ -277,10 +261,6 @@ test('handles mouse leave events without crashing', async () => {
     expect(accountIcon).toBeInTheDocument()
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('renders with various props correctly', async () => {
     const mockHandlers = {
         onMenuClick: jest.fn(),
@@ -301,10 +281,6 @@ test('renders with various props correctly', async () => {
     expect(screen.getByLabelText(/My cart/)).toBeInTheDocument()
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('handles responsive behavior correctly', async () => {
     // Test desktop behavior
     useMediaQuery.mockReturnValue([true]) // isDesktop = true
@@ -323,10 +299,6 @@ test('handles responsive behavior correctly', async () => {
     expect(screen.getByLabelText('My Account')).toBeInTheDocument()
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('renders children in body container', async () => {
     const testContent = 'Test Children Content'
 
@@ -342,10 +314,6 @@ test('renders children in body container', async () => {
     })
 })
 
-/*
- * DO NOT REMOVE THIS COMMENT! This test was generated by Cursor
- * This test was generated with the following model: Claude 3.5 Sonnet
- */
 test('handles search functionality', async () => {
     renderWithProviders(<Header />)