fixes for ttls test

vlad-dargel · vlad-dargel · commit e0a420373689 · 2024-08-16T14:19:15.000+02:00
diff --git a/tests/e2e/src/page-objects/components/editor-view/DoubleColumnKeyDetailsView.ts b/tests/e2e/src/page-objects/components/editor-view/DoubleColumnKeyDetailsView.ts
@@ -13,10 +13,10 @@ export class DoubleColumnKeyDetailsView extends KeyDetailsView {
     By.xpath(
       `//*[contains(@data-testid, "edit-${keyType}-button-${name}")] | //*[contains(@data-testid, "${keyType}_edit-btn-${name}")]`,
     )
-  getValueInput = (keyType: string, name: string): By =>
-    By.xpath(`//*[contains(@data-testid, "${keyType}_content-value-${name}")]`)
-  getWrapperOfValueInput  = (keyType: string, name: string): By =>
-    By.xpath(`//*[contains(@data-testid, "${keyType}_content-value-${name}")]/parent::*`)
+  getWrapperOfValueInput = (keyType: string, name: string): By =>
+    By.xpath(
+      `//*[contains(@data-testid, "${keyType}_content-${name}")]/parent::*`,
+    )
 
   /**
    * Edit key value from details
@@ -31,7 +31,10 @@ export class DoubleColumnKeyDetailsView extends KeyDetailsView {
     editorLocator: By,
     keyType: string,
   ): Promise<void> {
-    await InputActions.hoverElement(this.getWrapperOfValueInput(keyType, name),1000)
+    await InputActions.hoverElement(
+      this.getWrapperOfValueInput(keyType, name),
+      1000,
+    )
     const editLocator = this.getEditButton(keyType, name)
     await ButtonActions.clickElement(editLocator)
     await InputActions.typeText(editorLocator, value)
@@ -51,7 +54,10 @@ export class DoubleColumnKeyDetailsView extends KeyDetailsView {
     editorLocator: By,
     keyType: string,
   ): Promise<void> {
-    await InputActions.hoverElement(this.getWrapperOfValueInput(keyType, name),1000)
+    await InputActions.hoverElement(
+      this.getWrapperOfValueInput(keyType, name),
+      1000,
+    )
     const editLocator = this.getEditButton(keyType, name)
     await ButtonActions.clickElement(editLocator)
     await InputActions.slowType(editorLocator, value)
diff --git a/tests/e2e/src/page-objects/components/editor-view/HashKeyDetailsView.ts b/tests/e2e/src/page-objects/components/editor-view/HashKeyDetailsView.ts
@@ -9,12 +9,16 @@ import { Key } from 'vscode-extension-tester'
  * Hash Key details view
  */
 export class HashKeyDetailsView extends DoubleColumnKeyDetailsView {
-  hashFieldValueEditor = By.xpath(`//*[contains(@data-testid, 'hash_value-editor')]`)
+  hashFieldValueEditor = By.xpath(
+    `//*[contains(@data-testid, 'hash_value-editor')]`,
+  )
   hashFieldTtlEditor = By.xpath(`//*[@data-testid = 'inline-item-editor']`)
   hashFieldsList = By.xpath(
     `//*[contains(@data-testid, 'hash-field-') and not(contains(@data-testid,'value'))]/div`,
   )
-  hashValuesList = By.xpath(`//*[contains(@data-testid, 'hash_content-value-')]`)
+  hashValuesList = By.xpath(
+    `//*[contains(@data-testid, 'hash_content-value-')]`,
+  )
   truncatedValue = By.xpath(
     `//*[contains(@data-testid, 'hash_content-value-')]//*[@class = 'truncate']`,
   )
@@ -26,7 +30,9 @@ export class HashKeyDetailsView extends DoubleColumnKeyDetailsView {
     `//*[@data-testid = 'add-hash-field-panel']//*[contains(@data-testid, 'hash-value-')]`,
   )
   saveHashFieldButton = By.xpath(`//*[@data-testid = 'save-fields-btn']`)
-  showTtlCheckbox = By.xpath(`//*[@data-testid = 'show-ttl-column-checkbox']/parent::*/parent::*`)
+  showTtlCheckbox = By.xpath(
+    `//*[@data-testid = 'show-ttl-column-checkbox']/parent::*/parent::*`,
+  )
 
   getFieldTtlInputByField = (field: string): By =>
     By.xpath(`//div[@data-testid="hash-ttl_content-value-${field}"]`)
@@ -46,6 +52,8 @@ export class HashKeyDetailsView extends DoubleColumnKeyDetailsView {
       this.hashFieldValueEditor,
       KeyTypesShort.Hash,
     )
+    // Wait until value changes
+    await CommonDriverExtension.driverSleep(300)
   }
 
   /**
@@ -65,16 +73,23 @@ export class HashKeyDetailsView extends DoubleColumnKeyDetailsView {
    * @param fieldName The field name
    * @param fieldTtl The hash field ttl value for Redis databases 7.3.4 and higher
    */
-  async editHashKeyTtl(fieldName: string, fieldTtl: string = ''): Promise<void> {
+  async editHashKeyTtl(
+    fieldName: string,
+    fieldTtl: string = '',
+  ): Promise<void> {
     const enteredText = await this.getElementText(
-      this.getFieldTtlInputByField(fieldName))
+      this.getFieldTtlInputByField(fieldName),
+    )
     const count = enteredText.length
 
-    await InputActions.hoverElement(this.getFieldTtlInputByField(fieldName),1000)
+    await InputActions.hoverElement(
+      this.getFieldTtlInputByField(fieldName),
+      1000,
+    )
     await ButtonActions.clickElement(this.getEditFieldByField(fieldName))
 
     // clear the input
-    for(let i = 0; i < count; i++){
+    for (let i = 0; i < count; i++) {
       await InputActions.pressKey(this.hashFieldTtlEditor, Key.BACK_SPACE)
     }
     await InputActions.typeText(this.hashFieldTtlEditor, fieldTtl)
diff --git a/tests/e2e/src/page-objects/components/editor-view/KeyDetailsView.ts b/tests/e2e/src/page-objects/components/editor-view/KeyDetailsView.ts
@@ -78,7 +78,7 @@ export class KeyDetailsView extends WebView {
     By.xpath(`//*[@data-testid[starts-with(., '${keyName.split('-')[0]}') and contains(., '${columnName}')]]//*[@data-testid='value-as-json']
     | //*[@data-testid[starts-with(., '${keyName.split('-')[0]}-field-') and contains(., '${columnName}')]]//*[@data-testid='value-as-json']`)
   getEditBtnByKeyName = (keyName: string): By =>
-    By.xpath(`//vscode-button[contains(@data-testid, 'edit_') and contains(@data-testid, '${keyName}')]`)
+    By.xpath(`//vscode-button[contains(@data-testid, 'edit') and contains(@data-testid, '${keyName}')]`)
   /**
    * get key size
    */
diff --git a/tests/e2e/src/page-objects/components/editor-view/SortedSetKeyDetailsView.ts b/tests/e2e/src/page-objects/components/editor-view/SortedSetKeyDetailsView.ts
@@ -12,7 +12,7 @@ export class SortedSetKeyDetailsView extends DoubleColumnKeyDetailsView {
     `//*[contains(@data-testid, 'zset-member-value-')]`,
   )
   truncatedValue = By.xpath(
-    `//*[contains(@data-testid, 'zset-member-value-')]//*[contains(@aria-describedby, 'popup')]`,
+    `//*[contains(@data-testid, 'zset-score-value-')]//*[@class = 'truncate']`,
   )
   scoreSortedSetFieldsList = By.xpath(
     `//*[contains(@data-testid, 'zset_content-value-')]`,
diff --git a/tests/e2e/src/page-objects/components/tree-view/TreeView.ts b/tests/e2e/src/page-objects/components/tree-view/TreeView.ts
@@ -260,7 +260,8 @@ export class TreeView extends WebView {
       value,
     )
     await ButtonActions.clickElement(this.keyTreeFilterApplyBtn)
-    await this.waitForElementVisibility(this.loadingIndicator, 1000, false)
+    await this.waitForElementVisibility(this.loadingIndicator, 2000, true)
+    await this.waitForElementVisibility(this.loadingIndicator, 3000, false)
   }
 
   /**
diff --git a/tests/e2e/src/tests/browser/formatters.ts b/tests/e2e/src/tests/browser/formatters.ts
@@ -1,6 +1,12 @@
 import { expect } from 'chai'
 import { describe, it } from 'mocha'
-import { before, beforeEach, after, afterEach, Locator } from 'vscode-extension-tester'
+import {
+  before,
+  beforeEach,
+  after,
+  afterEach,
+  Locator,
+} from 'vscode-extension-tester'
 import {
   CliAPIRequests,
   DatabaseAPIRequests,
@@ -16,7 +22,8 @@ import {
 } from '@e2eSrc/page-objects/components'
 import {
   ButtonActions,
-  DatabasesActions, InputActions,
+  DatabasesActions,
+  InputActions,
   KeyDetailsActions,
 } from '@e2eSrc/helpers/common-actions'
 import { formatters, phpData } from '../../test-data/formatters-data'
@@ -224,6 +231,7 @@ describe('Formatters', () => {
       await keyDetailsView.selectFormatter(formatter.format)
       await hashKeyDetailsView.editHashKeyValue(invalidText)
       await ButtonActions.clickElement(keyDetailsView.saveButton)
+      await CommonDriverExtension.driverSleep(500)
       // Verify that invalid value can be saved
       expect(
         await hashKeyDetailsView.getElementText(
@@ -487,7 +495,7 @@ describe('Formatters', () => {
         keysData,
         Config.ossStandaloneConfig.databaseName,
         value,
-        value
+        value,
       )
       // Refresh database
       await treeView.refreshDatabaseByName(
@@ -512,8 +520,20 @@ describe('Formatters', () => {
           await KeyDetailsActions.openKeyDetailsByKeyNameInIframe(key.keyName)
           await keyDetailsView.selectFormatter(formatter.format)
           await CommonDriverExtension.driverSleep(300)
+          // Hover over value to see edit button for hash and list
+          if (
+            key.keyType === KeyTypesShort.Hash ||
+            key.keyType === KeyTypesShort.List
+          ) {
+            await InputActions.hoverElement(
+              doubleColumnKeyDetailsView.getWrapperOfValueInput(
+                key.keyType,
+                value,
+              ),
+              1000,
+            )
+          }
           // Verify that edit button disabled for Hash, List, String keys
-          await InputActions.hoverElement(doubleColumnKeyDetailsView.getWrapperOfValueInput(key.keyName, value),1000)
           expect(await keyDetailsView.isElementDisabled(editBtn, 'class')).eql(
             true,
             `Key ${key.keyType} is enabled for ${formatter.format} formatter`,
@@ -530,6 +550,13 @@ describe('Formatters', () => {
           await KeyDetailsActions.openKeyDetailsByKeyNameInIframe(key.keyName)
           await keyDetailsView.selectFormatter(formatter.format)
           // Verify that edit button enabled for ZSet
+          await InputActions.hoverElement(
+            doubleColumnKeyDetailsView.getWrapperOfValueInput(
+              key.keyType,
+              value,
+            ),
+            1000,
+          )
           expect(
             await keyDetailsView.isElementDisabled(editBtnLocator, 'class'),
           ).eql(
diff --git a/tests/e2e/src/tests/browser/large-key-details-values.ts b/tests/e2e/src/tests/browser/large-key-details-values.ts
@@ -297,6 +297,8 @@ describe('Large key details verification', () => {
     await KeyDetailsActions.openKeyDetailsByKeyNameInIframe(
       bigStringKeyParameters.keyName,
     )
+    // Wait until large key fully loaded
+    CommonDriverExtension.driverSleep(1000)
     expect(
       await stringKeyDetailsView.isElementDisabled(
         stringKeyDetailsView.editKeyValueButton,

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ export class SortedSetKeyDetailsView extends DoubleColumnKeyDetailsView {`
`12`	`12`	`//*[contains(@data-testid, 'zset-member-value-')]`,
`13`	`13`	`)`
`14`	`14`	`truncatedValue = By.xpath(`
`15`		- `//[contains(@data-testid, 'zset-member-value-')]//[contains(@aria-describedby, 'popup')]`,
	`15`	+ `//[contains(@data-testid, 'zset-score-value-')]//[@class = 'truncate']`,
`16`	`16`	`)`
`17`	`17`	`scoreSortedSetFieldsList = By.xpath(`
`18`	`18`	`//*[contains(@data-testid, 'zset_content-value-')]`,
Original file line number	Diff line number	Diff line change
`@@ -260,7 +260,8 @@ export class TreeView extends WebView {`
`260`	`260`	`value,`
`261`	`261`	`)`
`262`	`262`	`await ButtonActions.clickElement(this.keyTreeFilterApplyBtn)`
`263`		`- await this.waitForElementVisibility(this.loadingIndicator, 1000, false)`
	`263`	`+ await this.waitForElementVisibility(this.loadingIndicator, 2000, true)`
	`264`	`+ await this.waitForElementVisibility(this.loadingIndicator, 3000, false)`
`264`	`265`	`}`
`265`	`266`
`266`	`267`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -297,6 +297,8 @@ describe('Large key details verification', () => {`
`297`	`297`	`await KeyDetailsActions.openKeyDetailsByKeyNameInIframe(`
`298`	`298`	`bigStringKeyParameters.keyName,`
`299`	`299`	`)`
	`300`	`+ // Wait until large key fully loaded`
	`301`	`+ CommonDriverExtension.driverSleep(1000)`
`300`	`302`	`expect(`
`301`	`303`	`await stringKeyDetailsView.isElementDisabled(`
`302`	`304`	`stringKeyDetailsView.editKeyValueButton,`