@W-19053749: Removed edit button for auto-bonus products (#2815)

* Removed edit button for auto-bonus products

* removed extra comments

* Fixed lint issues

Author: ddiazccrz

packages/template-retail-react-app/app/pages/cart/partials/cart-secondary-button-group.jsx

@@ -104,16 +104,14 @@ const CartSecondaryButtonGroup = ({
                             />
                         </Button>
                     )}
-                    {/* Only show edit button if it's not a standard product */}
-                    {variant.id &&
-                        !variant.type?.item && ( // the variant.id ensures complete product data. Without it, Edit button appears briefly
-                            <Button variant="link" size="sm" onClick={() => onEditClick(variant)}>
-                                <FormattedMessage
-                                    defaultMessage="Edit"
-                                    id="cart_secondary_button_group.action.edit"
-                                />
-                            </Button>
-                        )}
+                    {variant.id && !variant.type?.item && !isBonusProduct && (
+                        <Button variant="link" size="sm" onClick={() => onEditClick(variant)}>
+                            <FormattedMessage
+                                defaultMessage="Edit"
+                                id="cart_secondary_button_group.action.edit"
+                            />
+                        </Button>
+                    )}
                 </ButtonGroup>
                 {!isBonusProduct && (
                     <Flex alignItems="center">

packages/template-retail-react-app/app/pages/cart/partials/cart-secondary-button-group.test.js

@@ -170,11 +170,10 @@ describe('CartSecondaryButtonGroup Edit button conditional rendering', () => {
     })
 })
 
-test('hides remove, wishlist and gift checkbox for bonus product', async () => {
+test('hides remove, wishlist, edit button and gift checkbox for bonus product', async () => {
     const {user} = renderWithProviders(<MockedComponent isBonusProduct={true} />)
 
-    expect(screen.getByRole('button', {name: /edit/i})).toBeInTheDocument()
-
+    expect(screen.queryByRole('button', {name: /edit/i})).not.toBeInTheDocument()
     expect(screen.queryByRole('button', {name: /remove/i})).not.toBeInTheDocument()
     expect(screen.queryByRole('button', {name: /add to wishlist/i})).not.toBeInTheDocument()
     expect(screen.queryByRole('checkbox', {name: /this is a gift/i})).not.toBeInTheDocument()