test(quantic): Add tests for feedback modal edge cases in quanticSmartSnippet e2e tests

packages/quantic/cypress/e2e/default-2/generatedAnswer/generated-answer-selectors.ts

@@ -90,7 +90,7 @@ export const GeneratedAnswerSelectors: GeneratedAnswerSelector = {
     ),
   feedbackDoneButton: () =>
     GeneratedAnswerSelectors.feedbackModal().find(
-      '[data-cy="feedback-modal-footer__done"]'
+      '[data-testid="feedback-modal-footer__done"]'
     ),
   feedbackDocumentUrlInput: () =>
     GeneratedAnswerSelectors.feedbackModal().find('input[name="documentUrl"]'),

packages/quantic/cypress/e2e/default-2/smart-snippet/smart-snippet-selectors.ts

@@ -71,7 +71,9 @@ export const SmartSnippetSelectors: SmartSnippetSelector = {
       .get('lightning-modal')
       .find('[data-testid="feedback-modal-footer__cancel"]'),
   feedbackDoneButton: () =>
-    cy.get('lightning-modal').find('[data-cy="feedback-modal-footer__done"]'),
+    cy
+      .get('lightning-modal')
+      .find('[data-testid="feedback-modal-footer__done"]'),
   feedbackDetailsInput: () =>
     cy
       .get('lightning-modal')

packages/quantic/force-app/main/default/lwc/quanticFeedbackModal/success.html

@@ -9,6 +9,6 @@
     </div>
   </lightning-modal-body>
   <lightning-modal-footer>
-    <lightning-button data-cy="feedback-modal-footer__done" variant="brand" label={labels.done} onclick={closeModal}></lightning-button>
+    <lightning-button data-testid="feedback-modal-footer__done" variant="brand" label={labels.done} onclick={closeModal}></lightning-button>
   </lightning-modal-footer>
 </template>

packages/quantic/force-app/main/default/lwc/quanticFeedbackModalQna/templates/success.html

@@ -13,7 +13,7 @@
   </lightning-modal-body>
   <lightning-modal-footer>
     <lightning-button
-      data-cy="feedback-modal-footer__done"
+      data-testid="feedback-modal-footer__done"
       variant="brand"
       label={labels.done}
       onclick={closeModal}

packages/quantic/force-app/main/default/lwc/quanticSmartSnippet/__tests__/quanticSmartSnippet.test.js

@@ -337,21 +337,5 @@ describe('c-quantic-smart-snippet', () => {
         });
       });
     });
-
-    describe('when interacting with the feedback modal', () => {
-      describe('when trying to open the feedback modal after executing the same query', () => {
-        // eslint-disable-next-line jest/no-disabled-tests, jest/expect-expect
-        it.skip('should not open the feedback modal', async () => {
-          // TODO: Implement this test - SFINT-5933
-        });
-      });
-
-      describe('when trying to open the feedback modal after executing a query that gave a new answer', () => {
-        // eslint-disable-next-line jest/no-disabled-tests, jest/expect-expect
-        it.skip('should open the feedback modal', async () => {
-          // TODO: Implement this test - SFINT-5933
-        });
-      });
-    });
   });
 });

packages/quantic/force-app/main/default/lwc/quanticSmartSnippet/e2e/pageObject.ts

@@ -45,6 +45,10 @@ export class SmartSnippetObject {
     return this.page.getByTestId('feedback-modal-footer__cancel');
   }
 
+  get feedbackModalDoneButton(): Locator {
+    return this.page.getByTestId('feedback-modal-footer__done');
+  }
+
   get firstFeedbackOptionLabel(): Locator {
     const labelFirstOption = "Didn't answer my question at all";
     return this.page.getByText(labelFirstOption);
@@ -94,6 +98,10 @@ export class SmartSnippetObject {
     await this.feedbackSubmitButton.click();
   }
 
+  async clickFeedbackModalDoneButton(): Promise<void> {
+    await this.feedbackModalDoneButton.click();
+  }
+
   async waitForExpandSmartSnippetUaAnalytics(): Promise<Request> {
     return this.waitForSmartSnippetCustomUaAnalytics('expandSmartSnippet');
   }

packages/quantic/force-app/main/default/lwc/quanticSmartSnippet/e2e/quanticSmartSnippet.e2e.ts

@@ -1,5 +1,6 @@
-import {testSearch, testInsight} from './fixture';
+import {testSearch, testInsight, expect} from './fixture';
 import {useCaseTestCases} from '../../../../../../playwright/utils/useCase';
+import smartSnippetData from './data';
 
 const maximumSnippetHeight = 250;
 
@@ -113,6 +114,55 @@
           });
         });
       });
+
+      test.describe('when trying to open the feedback modal after executing the same query', () => {
+        test('should not allow to open the feedback modal', async ({
+          smartSnippet,
+          search,
+        }) => {
+          await smartSnippet.clickDislikeButton();
+          await smartSnippet.clickExplainWhyButton();
+          await smartSnippet.selectFirstFeedbackOptionLabel();
+          await smartSnippet.clickFeedbackSubmitButton();
+
+          await smartSnippet.clickFeedbackModalDoneButton();
+
+          const searchResponsePromise = search.waitForSearchResponse();
+          await search.performSearch();
+          await searchResponsePromise;
+
+          expect(smartSnippet.dislikeButton).not.toBeNull();
+          await smartSnippet.clickDislikeButton();
+          expect(smartSnippet.explainWhyButton).not.toBeVisible();
+        });
+      });
+
+      test.describe('when trying to open the feedback modal after executing a query that gave a new answer', () => {
+        test('should allow to open the feedback modal', async ({
+          smartSnippet,
+          search,
+        }) => {
+          await smartSnippet.clickDislikeButton();
+          await smartSnippet.clickExplainWhyButton();
+          await smartSnippet.selectFirstFeedbackOptionLabel();
+          await smartSnippet.clickFeedbackSubmitButton();
+
+          await smartSnippet.clickFeedbackModalDoneButton();
+
+          await search.mockSearchWithSmartSnippetResponse({
+            ...smartSnippetData,
+            question: 'new question',
+            answerSnippet: 'new answer',
+          });
+          const searchResponsePromise = search.waitForSearchResponse();
+          await search.performSearch();
+          await searchResponsePromise;
+
+          expect(smartSnippet.dislikeButton).not.toBeNull();
+          await smartSnippet.clickDislikeButton();
+          expect(smartSnippet.explainWhyButton).toBeVisible();
+        });
+      });
     });
   });
 });