Merge pull request #3907 from RedisInsight/e2e/feature/RI-6061

E2e/feature/ri 6061

Author: mariasergeenko

tests/e2e/helpers/telemetry.ts

@@ -65,6 +65,6 @@ export class Telemetry {
         const requestBody = JSON.parse(request.request.body.toString());
 
         // Verify that event has correct property value
-        await t.expect(requestBody.eventData[property]).eql(value);
+        await t.expect(String(requestBody.eventData[property])).eql(value);
     }
 }

tests/e2e/pageObjects/browser-page.ts

@@ -107,7 +107,8 @@ export class BrowserPage extends InstancePage {
     backToBrowserBtn = Selector('[data-testid=back-right-panel-btn]');
     loadAllBtn = Selector('[data-testid=load-all-value-btn]');
     downloadAllValueBtn = Selector('[data-testid=download-all-value-btn]');
-    openTutorialsBtn = Selector('[data-testid=explore-msg-btn]');
+    openTutorialsBtn = Selector('[data-testid=explore-msg-btn]')
+    keyItem = Selector('[data-testid*="node-item"][data-testid*="keys:"]');
     //CONTAINERS
     streamGroupsContainer = Selector('[data-testid=stream-groups-container]');
     streamConsumersContainer = Selector('[data-testid=stream-consumers-container]');

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

@@ -1,6 +1,7 @@
 import { Selector, t } from 'testcafe';
 import { DatabaseHelper } from '../../../../helpers/database';
 import { BrowserPage, MyRedisDatabasePage } from '../../../../pageObjects';
+import { Telemetry } from '../../../../helpers/telemetry';
 import {
     commonUrl,
     ossStandaloneBigConfig,
@@ -19,6 +20,32 @@ const myRedisDatabasePage = new MyRedisDatabasePage();
 const databaseHelper = new DatabaseHelper();
 const databaseAPIRequests = new DatabaseAPIRequests();
 const apiKeyRequests = new APIKeyRequests();
+const telemetry = new Telemetry();
+
+const telemetryEvents = ['SEARCH_MODE_CHANGED', 'SEARCH_INDEX_ADD_BUTTON_CLICKED', 'SEARCH_INDEX_ADDED'];
+const logger = telemetry.createLogger();
+const expectedPropertiesMode = [
+    'current',
+    'databaseId',
+    'previous',
+    'provider',
+    'view'
+];
+
+const expectedPropertiesCreateIndex = [
+    'databaseId',
+    'provider',
+    'view'
+];
+
+const expectedPropertiesAddedIndex = [
+    'countOfFieldNames',
+    'countOfPrefixes',
+    'dataType',
+    'databaseId',
+    'provider',
+    'view'
+];
 
 const patternModeTooltipText = 'Filter by Key Name or Pattern';
 const redisearchModeTooltipText = 'Search by Values of Keys';
@@ -167,7 +194,7 @@ test
         await Common.checkURLContainsText(externalPageLinkFirst);
         await Common.checkURLContainsText(externalPageLinkSecond);
     });
-test
+test.requestHooks(logger)
     .before(async() => {
         await databaseHelper.acceptLicenseTermsAndAddDatabaseApi(ossStandaloneBigConfig);
     })
@@ -177,10 +204,17 @@ test
     })('Index creation', async t => {
         const createIndexLink = 'https://redis.io/docs/latest/commands/ft.create/?utm_source=redisinsight&utm_medium=app&utm_campaign=browser_search';
 
-        // Verify that user can cancel index creation
+        // Verify that telemetry event 'SEARCH_MODE_CHANGED' sent
         await t.click(browserPage.redisearchModeBtn);
+        await telemetry.verifyEventHasProperties(telemetryEvents[0], expectedPropertiesMode, logger);
+
+        // Verify that user can cancel index creation
         await t.click(browserPage.selectIndexDdn);
         await t.click(browserPage.createIndexBtn);
+
+        // Verify that telemetry event 'SEARCH_INDEX_ADD_BUTTON_CLICKED' sent
+        await telemetry.verifyEventHasProperties(telemetryEvents[1], expectedPropertiesCreateIndex, logger);
+
         await t.expect(browserPage.newIndexPanel.exists).ok('New Index panel is not displayed');
         await t.click(browserPage.cancelIndexCreationBtn);
         await t.expect(browserPage.newIndexPanel.exists).notOk('New Index panel is displayed');
@@ -197,8 +231,8 @@ test
         await goBackHistory();
 
         // Verify that user can create an index with multiple prefixes
-        // await t.click(browserPage.selectIndexDdn);
-        // await t.click(browserPage.createIndexBtn);
+        await t.click(browserPage.selectIndexDdn);
+        await t.click(browserPage.createIndexBtn);
         await t.click(browserPage.indexNameInput);
         await t.typeText(browserPage.indexNameInput, indexName);
         await t.click(browserPage.prefixFieldInput);
@@ -214,6 +248,10 @@ test
         await t.click(browserPage.indexIdentifierInput);
         await t.typeText(browserPage.indexIdentifierInput, 'k0');
         await t.click(browserPage.confirmIndexCreationBtn);
+
+        await telemetry.verifyEventHasProperties(telemetryEvents[2], expectedPropertiesAddedIndex, logger);
+        await telemetry.verifyEventPropertyValue(telemetryEvents[2], 'countOfPrefixes', '3', logger);
+
         await t.expect(browserPage.newIndexPanel.exists).notOk('New Index panel is displayed');
         await t.click(browserPage.selectIndexDdn);
         await browserPage.selectIndexByName(indexName);

tests/e2e/tests/web/regression/browser/list-key.e2e.ts

@@ -6,17 +6,27 @@ import { DatabaseAPIRequests } from '../../../../helpers/api/api-database';
 import { populateListWithElements } from '../../../../helpers/keys';
 import { Common } from '../../../../helpers/common';
 import { APIKeyRequests } from '../../../../helpers/api/api-keys';
+import { Telemetry } from '../../../../helpers/telemetry';
 
 const browserPage = new BrowserPage();
 const databaseHelper = new DatabaseHelper();
 const databaseAPIRequests = new DatabaseAPIRequests();
 const apiKeyRequests = new APIKeyRequests();
+const telemetry = new Telemetry();
 
 const dbParameters = { host: ossStandaloneConfig.host, port: ossStandaloneConfig.port };
 const keyName = `TestListKey-${ Common.generateWord(10) }`;
 const elementForSearch = `SearchField-${ Common.generateWord(5) }`;
 const keyToAddParameters = { elementsCount: 500000, keyName, elementStartWith: 'listElement' };
 
+const telemetryEvent = 'LIST_VIEW_OPENED';
+const logger = telemetry.createLogger();
+
+const expectedProperties = [
+    'databaseId',
+    'provider'
+];
+
 fixture `List Key verification`
     .meta({ type: 'regression' })
     .page(commonUrl)
@@ -29,11 +39,16 @@ fixture `List Key verification`
         await apiKeyRequests.deleteKeyByNameApi(keyName, ossStandaloneConfig.databaseName);
         await databaseAPIRequests.deleteStandaloneDatabaseApi(ossStandaloneConfig);
     });
-test
+test.requestHooks(logger)
     .meta({ rte: rte.standalone })('Verify that user can search per exact element index in List key in DB with 1 million of fields', async t => {
         // Add 1000000 elements to the list key
         await populateListWithElements(dbParameters.host, dbParameters.port, keyToAddParameters);
         await populateListWithElements(dbParameters.host, dbParameters.port, keyToAddParameters);
+
+        // Verify that telemetry event 'TREE_VIEW_KEY_VALUE_VIEWED' sent
+        await t.click(browserPage.browserViewButton);
+        await telemetry.verifyEventHasProperties(telemetryEvent, expectedProperties, logger);
+
         // Add custom element to the list key
         await browserPage.openKeyDetails(keyName);
         await browserPage.addElementToList(elementForSearch);

tests/e2e/tests/web/regression/tree-view/tree-view.e2e.ts

@@ -1,5 +1,6 @@
 import { DatabaseHelper } from '../../../../helpers/database';
 import { BrowserPage, WorkbenchPage } from '../../../../pageObjects';
+import { Telemetry } from '../../../../helpers/telemetry';
 import {
     commonUrl,
     ossStandaloneBigConfig,
@@ -16,8 +17,17 @@ const browserPage = new BrowserPage();
 const workbenchPage = new WorkbenchPage();
 const databaseHelper = new DatabaseHelper();
 const databaseAPIRequests = new DatabaseAPIRequests();
+const telemetry = new Telemetry();
 
 let keyNames: string[] = [];
+const telemetryEvent = 'TREE_VIEW_KEY_VALUE_VIEWED';
+const logger = telemetry.createLogger();
+
+const expectedProperties = [
+    'databaseId',
+    'keyType',
+    'provider'
+];
 
 fixture `Tree view verifications`
     .meta({ type: 'regression', rte: rte.standalone })
@@ -101,17 +111,21 @@ test
         await t.expect(totalKeysValue).gte(10, 'the info message after upload does not appear');
     });
 
-test('Verify that user can see the total number of keys, the number of keys scanned, the “Scan more” control displayed at the top of Tree view and Browser view', async t => {
-    await browserPage.selectFilterGroupType(KeyTypesTexts.Hash);
+test.requestHooks(logger)('Verify that user can see the total number of keys, the number of keys scanned, the “Scan more” control displayed at the top of Tree view and Browser view', async t => {
+    await browserPage.selectFilterGroupType(KeyTypesTexts.ReJSON);
     // Verify the controls on the Browser view
     await t.expect(browserPage.totalKeysNumber.visible).ok('The total number of keys is not displayed on the Browser view');
     await t.expect(browserPage.scannedValue.visible).ok('The number of keys scanned is not displayed on the Browser view');
     await t.expect(browserPage.scanMoreButton.visible).ok('The scan more button is not displayed on the Browser view');
-    // Verify the controls on the Tree view
+    //Verify the controls on the Tree view
     await t.click(browserPage.treeViewButton);
     await t.expect(browserPage.totalKeysNumber.visible).ok('The total number of keys is not displayed on the Tree view');
     await t.expect(browserPage.scannedValue.visible).ok('The number of keys scanned is not displayed on the Tree view');
     await t.expect(browserPage.scanMoreButton.visible).ok('The scan more button is not displayed on the Tree view');
+
+    // Verify that telemetry event 'TREE_VIEW_KEY_VALUE_VIEWED' sent
+    await t.click(browserPage.keyItem);
+    await telemetry.verifyEventHasProperties(telemetryEvent, expectedProperties, logger);
 });
 test('Verify that when user deletes the key he can see the key is removed from the folder, the number of keys is reduced, the percentage is recalculated', async t => {
     const mainFolder = browserPage.TreeView.getFolderSelectorByName('device');