fixes for regression

Author: vlad-dargel

tests/e2e/test-data/upload-tutorials/sample.zip renamed to tests/e2e/test-data/upload-tutorials/tutorialTest.zip

@@ -264,6 +264,7 @@ test
         await myRedisDatabasePage.clickOnDBByName(simpleDbName); // click standalone database
         await cliPage.sendCommandInCli(`FT.DROPINDEX ${indexNameSimpleDb}`);
         await t.click(browserPage.patternModeBtn);
+        await t.click(browserPage.browserViewButton);
         await browserPage.deleteKeysByNames(keyNames);
 
         //delete database

tests/e2e/tests/critical-path/browser/search-capabilities.e2e.ts

@@ -114,7 +114,8 @@ test('Verify that user can edit and run automatically added "Aggregate" script i
     await t.expect(workbenchPage.queryTableResult.textContent).contains(aggregationResultField, 'The aggregation field name is not in the Search result');
     await t.expect(workbenchPage.queryTableResult.textContent).contains('100', 'The aggregation max value is in not the Search result');
 });
-test('Verify that when the “Manual” option clicked, user can see the Editor is automatically prepopulated with the information', async t => {
+// Outdated after https://redislabs.atlassian.net/browse/RI-4279
+test.skip('Verify that when the “Manual” option clicked, user can see the Editor is automatically prepopulated with the information', async t => {
     const information = [
         '// Workbench is the advanced Redis command-line interface that allows to send commands to Redis, read and visualize the replies sent by the server.',
         '// Enter multiple commands at different rows to run them at once.',

tests/e2e/tests/critical-path/workbench/default-scripts-area.e2e.ts

@@ -6,14 +6,13 @@ import {
 import { MyRedisDatabasePage, WorkbenchPage } from '../../../pageObjects';
 import { commonUrl, ossStandaloneConfig } from '../../../helpers/conf';
 import { deleteStandaloneDatabaseApi } from '../../../helpers/api/api-database';
-import {Common} from '../../../helpers/common';
 
 const myRedisDatabasePage = new MyRedisDatabasePage();
 const workbenchPage = new WorkbenchPage();
-const common = new Common();
 const filePath = path.join('..', '..', '..', 'test-data', 'upload-tutorials', 'sample.zip');
-const tutorialName = `tutorialName-${common.generateWord(10)}`;
-const link = 'https://drive.google.com/uc?export=download&id=1mlyDKWLu12L02FblOPh15EwG2Vy_FhJ7';
+const tutorialName = 'tutorialTest';
+const tutorialName2 = 'tutorialTestByLink';
+const link = 'https://drive.google.com/uc?id=1puRUoT8HmyZCekkeWNxBzXe_48TzXcJc&export=download';
 let folder1 = 'folder-1';
 let folder2 = 'folder-2';
 let internalLinkName1 = 'probably-1';
@@ -29,7 +28,7 @@ fixture `Upload custom tutorials`
     .afterEach(async() => {
         await deleteStandaloneDatabaseApi(ossStandaloneConfig);
     });
-// https://redislabs.atlassian.net/browse/RI-4186, https://redislabs.atlassian.net/browse/RI-4198
+// https://redislabs.atlassian.net/browse/RI-4186, https://redislabs.atlassian.net/browse/RI-4198, https://redislabs.atlassian.net/browse/RI-4302
 test('Verify that user can upload tutorial with local zip file without manifest.json', async t => {
     // Verify that user can upload custom tutorials on docker version
     folder1 = 'folder-1';
@@ -39,8 +38,6 @@ test('Verify that user can upload tutorial with local zip file without manifest.
     // Verify that user can see the “MY TUTORIALS” section in the Enablement area.
     await t.expect(workbenchPage.customTutorials.visible).ok('custom tutorials sections is not visible');
     await t.click(workbenchPage.tutorialOpenUploadButton);
-    // Verify that User can enter a tutorial name
-    await t.typeText(workbenchPage.tutorialNameField, tutorialName);
     await t.expect(workbenchPage.tutorialSubmitButton.hasAttribute('disabled')).ok('submit button is not disabled');
     // Verify that User can request to add a new custom Tutorial by uploading a .zip archive from a local folder
     await t.setFilesToUpload(workbenchPage.tutorialImport, [filePath]);
@@ -50,10 +47,12 @@ test('Verify that user can upload tutorial with local zip file without manifest.
     await t.click(workbenchPage.tutorialAccordionButton.withText(tutorialName));
     await t.expect((await workbenchPage.getAccordionButtonWithName(folder1)).visible).ok(`${folder1} is not visible`);
     await t.expect((await workbenchPage.getAccordionButtonWithName(folder2)).visible).ok(`${folder2} is not visible`);
+    await t.click(await workbenchPage.getAccordionButtonWithName(folder1));
     await t.expect((await workbenchPage.getInternalLinkWithManifest(internalLinkName1)).visible)
         .ok(`${internalLinkName1} is not visible`);
+    await t.click(await workbenchPage.getAccordionButtonWithName(folder2));
     await t.expect((await workbenchPage.getInternalLinkWithManifest(internalLinkName2)).visible)
-        .ok(`${internalLinkName2} is not visible`);
+        .ok(`${internalLinkName1} is not visible`);
     await t.expect(workbenchPage.scrolledEnablementArea.exists).notOk('enablement area is visible before clicked');
     await t.click((await workbenchPage.getInternalLinkWithManifest(internalLinkName1)));
     await t.expect(workbenchPage.scrolledEnablementArea.visible).ok('enablement area is not visible after clicked');
@@ -66,21 +65,23 @@ test('Verify that user can upload tutorial with local zip file without manifest.
     await t.expect((workbenchPage.tutorialAccordionButton.withText(tutorialName).exists))
         .notOk(`${tutorialName} tutorial is not uploaded`);
 });
-// https://redislabs.atlassian.net/browse/RI-4186, https://redislabs.atlassian.net/browse/RI-4213
+// https://redislabs.atlassian.net/browse/RI-4186, https://redislabs.atlassian.net/browse/RI-4213, https://redislabs.atlassian.net/browse/RI-4302
 test('Verify that user can upload tutorial with URL with manifest.json', async t => {
-    internalLinkName1 = 'working_probably';
+    const labelFromManifest = 'LabelFromManifest';
+    internalLinkName1 = 'manifest-id';
 
     await t.click(workbenchPage.tutorialOpenUploadButton);
-    await t.typeText(workbenchPage.tutorialNameField, tutorialName);
     // Verify that user can upload tutorials using a URL
     await t.typeText(workbenchPage.tutorialLinkField, link);
     await t.click(workbenchPage.tutorialSubmitButton);
-    await t.expect(workbenchPage.tutorialAccordionButton.withText(tutorialName).with({ timeout: 20000 }).visible)
-        .ok(`${tutorialName} tutorial is not uploaded`);
-    await t.click(workbenchPage.tutorialAccordionButton.withText(tutorialName));
+    await t.expect(workbenchPage.tutorialAccordionButton.withText(tutorialName2).with({ timeout: 20000 }).visible)
+        .ok(`${tutorialName2} tutorial is not uploaded`);
+    await t.click(workbenchPage.tutorialAccordionButton.withText(tutorialName2));
     // Verify that User can see the same structure in the tutorial uploaded as described in the .json manifest
     await t.expect((await workbenchPage.getInternalLinkWithoutManifest(internalLinkName1)).visible)
-        .ok(`${internalLinkName1} folder is not visible`);
+        .ok(`${internalLinkName1} folder specified in manifest is not visible`);
+    await t.expect(await (await workbenchPage.getInternalLinkWithoutManifest(internalLinkName1)).textContent)
+        .eql(labelFromManifest, `${labelFromManifest} tutorial specified in manifest is not visible`);
     await t.click((await workbenchPage.getInternalLinkWithoutManifest(internalLinkName1)));
     await t.expect(workbenchPage.scrolledEnablementArea.visible).ok('enablement area is not visible after clicked');
     await t.click(workbenchPage.closeEnablementPage);
@@ -89,6 +90,6 @@ test('Verify that user can upload tutorial with URL with manifest.json', async t
     await t.click(workbenchPage.tutorialDeleteButton);
     await t.expect(workbenchPage.tutorialDeleteButton.exists).notOk('Delete popup is still visible');
     // Verify that when User delete the tutorial, then User can see this tutorial and relevant markdown files are deleted from: the Enablement area in Workbench
-    await t.expect((workbenchPage.tutorialAccordionButton.withText(tutorialName).exists))
-        .notOk(`${tutorialName} tutorial is not uploaded`);
+    await t.expect((workbenchPage.tutorialAccordionButton.withText(tutorialName2).exists))
+        .notOk(`${tutorialName2} tutorial is not uploaded`);
 });