feat(chat): move from edit to move endpoint for conversations/prompts rename (Issue #3052)

apps/chat-e2e/src/assertions/api/apiAssertion.ts

@@ -1,6 +1,6 @@
 import { ChatBody } from '@/chat/types/chat';
+import { MoveModel } from '@/chat/types/common';
 import { DialAIEntityModel } from '@/chat/types/models';
-import { Prompt } from '@/chat/types/prompt';
 import { ExpectedConstants, ExpectedMessages } from '@/src/testData';
 import { Message } from '@epam/ai-dial-shared';
 import { expect } from '@playwright/test';
@@ -123,30 +123,30 @@ export class ApiAssertion {
       .toBe(expectedMessage);
   }
 
-  public async assertRequestPromptName(request: Prompt, expectedValue: string) {
-    expect
-      .soft(request.name, ExpectedMessages.promptRequestNameIsValid)
-      .toBe(expectedValue);
-  }
-
-  public async assertRequestPromptDescription(
-    request: Prompt,
-    expectedValue: string,
+  public async assertMoveRequest(
+    request: MoveModel,
+    expectedDestination: string,
+    expectedSource: string,
+    isOverwritten = false,
   ) {
     expect
       .soft(
-        request.description,
-        ExpectedMessages.promptRequestDescriptionIsValid,
+        request.destinationUrl.endsWith(`/${expectedDestination}`),
+        ExpectedMessages.moveDestinationIsValid,
       )
-      .toBe(expectedValue);
-  }
-
-  public async assertRequestPromptContent(
-    request: Prompt,
-    expectedValue: string,
-  ) {
+      .toBeTruthy();
     expect
-      .soft(request.content, ExpectedMessages.promptRequestContentIsValid)
-      .toBe(expectedValue);
+      .soft(
+        request.sourceUrl.endsWith(`/${expectedSource}`),
+        ExpectedMessages.moveSourceIsValid,
+      )
+      .toBeTruthy();
+    isOverwritten
+      ? expect
+          .soft(isOverwritten, ExpectedMessages.moveSourceIsValid)
+          .toBeTruthy()
+      : expect
+          .soft(isOverwritten, ExpectedMessages.moveOverwriteIsValid)
+          .toBeFalsy();
   }
 }

apps/chat-e2e/src/testData/expectedConstants.ts

@@ -57,20 +57,11 @@ export const ExpectedConstants = {
     'Deleting will stop sharing and other users will no longer see this conversation.',
   renameSharedFolderMessage:
     'Renaming will stop sharing and other users will no longer see this folder.',
-  renameSharedConversationMessage:
-    'Renaming will stop sharing and other users will no longer see this conversation.',
   deleteSharedPromptMessage:
     'Are you sure that you want to delete a prompt?\n' +
     'Deleting will stop sharing and other users will no longer see this prompt.',
-  sharedConversationModelChangeDialogTitle: 'Confirm model changing',
-  renameSharedConversationDialogTitle: 'Confirm renaming conversation',
-  renameSharedPromptDialogTitle: 'Confirm renaming prompt',
   notAllowedToMoveParentToChild:
     "It's not allowed to move parent folder in child folder",
-  sharedConversationModelChangeMessage:
-    'Model changing will stop sharing and other users will no longer see this conversation.',
-  renameSharedPromptMessage:
-    'Renaming will stop sharing and other users will no longer see this prompt.',
   deletePromptConfirmationModalTitle: 'Confirm deleting prompt',
   deletePromptConfirmationModalMessage:
     'Are you sure that you want to delete a prompt?',
@@ -116,9 +107,9 @@ export const ExpectedConstants = {
   responseFileUrlContentPattern: (model: string) =>
     new RegExp('/appdata/' + model + '/images/.*\\.png', 'g'),
   shareConversationText:
-    'This link is temporary and will be active for 3 days. This conversation and future changes to it will be visible to users who follow the link. Only owner will be able to make changes. Renaming or changing the model will stop sharing.',
+    'This link is temporary and will be active for 3 days. This conversation and future changes to it will be visible to users who follow the link. Only owner will be able to make changes.',
   sharePromptText:
-    'This link is temporary and will be active for 3 days. This prompt and future changes to it will be visible to users who follow the link. Only owner will be able to make changes. Renaming will stop sharing.',
+    'This link is temporary and will be active for 3 days. This prompt and future changes to it will be visible to users who follow the link. Only owner will be able to make changes.',
   shareApplicationText:
     'This application and its updates will be visible to users with the link. Renaming or changing the version will stop sharing.',
   shareConversationFolderText:
@@ -300,6 +291,9 @@ export const API = {
   promptsHost: () => `${API.listingHost}/prompts`,
   filesListingHost: () => `${API.listingHost}/files`,
   fileHost: '/api/files',
+  conversationHost: '/api/conversations',
+  promptHost: '/api/prompts',
+  moveHost: '/api/ops/resource/move',
   importFileRootPath: (bucket: string) => `files/${bucket}`,
   modelFilePath: (modelId: string) => `appdata/${modelId}/images`,
   importFilePath: (bucket: string, modelId: string) =>

apps/chat-e2e/src/testData/expectedMessages.ts

@@ -91,9 +91,9 @@ export enum ExpectedMessages {
   chatRequestAddonsAreValid = 'Chat API request addons are valid',
   chatRequestMessageIsValid = 'Chat API request message is valid',
   chatRequestAttachmentIsValid = 'Chat API request attachment is valid',
-  promptRequestNameIsValid = 'Prompt API request name is valid',
-  promptRequestDescriptionIsValid = 'Prompt API request description is valid',
-  promptRequestContentIsValid = 'Prompt API request content is valid',
+  moveDestinationIsValid = 'Move API request destination is valid',
+  moveSourceIsValid = 'Move API request source is valid',
+  moveOverwriteIsValid = 'Move API request overwrite is valid',
   sendMessageButtonDisabled = 'Send message button is disabled',
   sendMessageButtonEnabled = 'Send message button is enabled',
   tooltipContentIsValid = 'Tooltip content is valid',

apps/chat-e2e/src/tests/defaultModelSettings.test.ts

@@ -414,7 +414,6 @@ dialTest(
     });
     await dialHomePage.waitForPageLoaded();
     await chat.changeAgentButton.click();
-    nonDefaultModel = ModelsUtil.getModel('gpt-4-turbo-2024-04-09')!;
     await talkToAgentDialog.selectAgent(nonDefaultModel, marketplacePage);
     await dialHomePage.mockChatTextResponse(
       MockedChatApiResponseBodies.simpleTextBody,

apps/chat-e2e/src/tests/folderNameNumeration.test.ts

@@ -117,6 +117,7 @@ dialTest(
         await folderDropdownMenu.selectMenuOption(MenuOptions.rename);
         await folderConversations.editFolderNameWithTick(
           GeneratorUtil.randomString(5),
+          { isHttpMethodTriggered: false },
         );
 
         await chatBar.createNewFolder();

apps/chat-e2e/src/tests/parametrizedReplay.test.ts

@@ -266,7 +266,9 @@ dialTest(
     await dialTest.step(
       'Export conversation and then delete all Dial entities',
       async () => {
-        await conversations.openEntityDropdownMenu(conversation.name, 2);
+        await conversations.openEntityDropdownMenu(conversation.name, {
+          exactMatch: true,
+        });
         await conversationDropdownMenu.selectMenuOption(MenuOptions.export);
         exportedData = await dialHomePage.downloadData(
           () =>

apps/chat-e2e/src/tests/promptFolderNamesakes.test.ts

@@ -41,7 +41,9 @@ dialTest(
         ExpectedConstants.newFolderWithIndexTitle(1),
       );
       await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
-      await folderPrompts.editFolderNameWithTick(duplicatedFolderName);
+      await folderPrompts.editFolderNameWithTick(duplicatedFolderName, {
+        isHttpMethodTriggered: false,
+      });
     });
 
     await dialTest.step(
@@ -207,6 +209,7 @@ dialTest(
       await folderDropdownMenu.selectMenuOption(MenuOptions.rename);
       await folderPrompts.editFolderNameWithTick(
         ExpectedConstants.newFolderWithIndexTitle(2),
+        { isHttpMethodTriggered: false },
       );
     });
 

apps/chat-e2e/src/tests/promptFolderNumeration.test.ts

@@ -163,7 +163,9 @@ dialTest(
           ExpectedConstants.newFolderWithIndexTitle(folderNumber),
         );
         await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
-        await folderPrompts.editFolderNameWithTick('Renamed Folder');
+        await folderPrompts.editFolderNameWithTick('Renamed Folder', {
+          isHttpMethodTriggered: false,
+        });
 
         await promptBar.createNewFolder();
         await expect
@@ -211,6 +213,7 @@ dialTest(
       await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
       await folderPrompts.editFolderNameWithTick(
         ExpectedConstants.newPromptFolderWithIndexTitle(999),
+        { isHttpMethodTriggered: false },
       );
     });
 
@@ -285,7 +288,9 @@ dialTest(
           ExpectedConstants.newFolderWithIndexTitle(i),
         );
         await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
-        await folderPrompts.editFolderNameWithTick(duplicatedFolderName);
+        await folderPrompts.editFolderNameWithTick(duplicatedFolderName, {
+          isHttpMethodTriggered: false,
+        });
         await expect(
           folderPrompts.getFolderByName(duplicatedFolderName, i),
           ExpectedMessages.folderNameUpdated,

apps/chat-e2e/src/tests/promptFoldersSpecialChars.test.ts

@@ -81,7 +81,9 @@ dialTest(
     });
 
     await dialTest.step('Rename it to contain special characters', async () => {
-      await folderPrompts.editFolderNameWithTick(newNameWithSpecialChars);
+      await folderPrompts.editFolderNameWithTick(newNameWithSpecialChars, {
+        isHttpMethodTriggered: false,
+      });
       await promptBarFolderAssertion.assertFolderState(
         { name: newNameWithSpecialChars },
         'visible',
@@ -174,7 +176,9 @@ dialTest(
       async () => {
         await folderPrompts.openFolderDropdownMenu(expectedName);
         await folderDropdownMenu.selectMenuOption(MenuOptions.rename);
-        await folderPrompts.editFolderNameWithTick(newNameWithEmojis);
+        await folderPrompts.editFolderNameWithTick(newNameWithEmojis, {
+          isHttpMethodTriggered: false,
+        });
         await promptBarFolderAssertion.assertFolderState(
           { name: newNameWithEmojis },
           'visible',

apps/chat-e2e/src/tests/promptMaximumNameLength.test.ts

@@ -75,10 +75,6 @@ dialTest(
     await dialTest.step('Rename the prompt to a long name', async () => {
       await prompts.openEntityDropdownMenu(expectedName);
       await promptDropdownMenu.selectMenuOption(MenuOptions.edit);
-      await promptModalDialog.setField(
-        promptModalDialog.name,
-        nameUnder160Symbols,
-      );
       // Wait for the API request to update the prompt name
       await promptModalDialog.updatePromptDetailsWithButton(
         nameUnder160Symbols,
@@ -138,14 +134,18 @@ dialTest(
           ExpectedConstants.newPromptFolderWithIndexTitle(1),
         );
         await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
-        await folderPrompts.editFolderNameWithTick(longName);
+        await folderPrompts.editFolderNameWithTick(longName, {
+          isHttpMethodTriggered: false,
+        });
 
         // Rename folder_child
         await folderPrompts.openFolderDropdownMenu(
           ExpectedConstants.newPromptFolderWithIndexTitle(2),
         );
         await promptDropdownMenu.selectMenuOption(MenuOptions.rename);
-        await folderPrompts.editFolderNameWithTick(longName);
+        await folderPrompts.editFolderNameWithTick(longName, {
+          isHttpMethodTriggered: false,
+        });
       },
     );
 

apps/chat-e2e/src/tests/promptUsage.test.ts

@@ -1,4 +1,3 @@
-import { Conversation } from '@/chat/types/chat';
 import { Prompt } from '@/chat/types/prompt';
 import { ShareByLinkResponseModel } from '@/chat/types/share';
 import dialTest from '@/src/core/dialFixtures';
@@ -510,7 +509,6 @@ dialTest(
   async ({
     dialHomePage,
     promptData,
-    conversationData,
     dataInjector,
     agentSettings,
     conversationSettingsModal,
@@ -533,24 +531,17 @@ dialTest(
       `{{${aVar}}}`,
       `{{${bVar}|${bVarDefaultValue}}}`,
     );
-    let conversation: Conversation;
 
-    await dialTest.step(
-      'Prepare prompt with vars and empty conversation',
-      async () => {
-        prompt = promptData.preparePrompt(promptContent);
-        conversation = conversationData.prepareEmptyConversation();
-        await dataInjector.createPrompts([prompt]);
-        await dataInjector.createConversations([conversation]);
-      },
-    );
+    await dialTest.step('Prepare prompt with vars', async () => {
+      prompt = promptData.preparePrompt(promptContent);
+      await dataInjector.createPrompts([prompt]);
+    });
 
     await dialTest.step(
       `Type / in system prompt field, select created prompt and verify variable modal with default values is displayed`,
       async () => {
         await dialHomePage.openHomePage();
         await dialHomePage.waitForPageLoaded();
-        await conversations.selectConversation(conversation.name);
         await chat.configureSettingsButton.click();
         await agentSettings.setSystemPrompt('/');
         const promptsList = agentSettings.getPromptList();
@@ -574,7 +565,7 @@ dialTest(
         await agentSettingAssertion.assertSystemPromptValue(
           promptTemplate(aVarValue, bVarDefaultValue),
         );
-        await conversationSettingsModal.cancelButton.click();
+        await conversationSettingsModal.applyChangesButton.click();
       },
     );