missed a few

Author: lerouxb

packages/compass-e2e-tests/tests/collection-bulk-delete.test.ts

@@ -70,13 +70,13 @@ describe('Bulk Delete', function () {
     ).to.equal('{ i: 5 }');
 
     // Check that it will update the expected number of documents
-    expect(browser.$(Selectors.BulkDeleteModalTitle).getText()).to.equal(
+    expect(await browser.$(Selectors.BulkDeleteModalTitle).getText()).to.equal(
       'Delete 1 document'
     );
 
-    expect(browser.$(Selectors.BulkDeleteModalDeleteButton).getText()).to.equal(
-      'Delete 1 document'
-    );
+    expect(
+      await browser.$(Selectors.BulkDeleteModalDeleteButton).getText()
+    ).to.equal('Delete 1 document');
 
     // Press delete
     await browser.clickVisible(Selectors.BulkDeleteModalDeleteButton);
@@ -130,13 +130,13 @@ describe('Bulk Delete', function () {
     await browser.$(Selectors.BulkDeleteModal).waitForDisplayed();
 
     // Check that it will update the expected number of documents
-    expect(browser.$(Selectors.BulkDeleteModalTitle).getText()).to.equal(
+    expect(await browser.$(Selectors.BulkDeleteModalTitle).getText()).to.equal(
       'Delete 1 document'
     );
 
-    expect(browser.$(Selectors.BulkDeleteModalDeleteButton).getText()).to.equal(
-      'Delete 1 document'
-    );
+    expect(
+      await browser.$(Selectors.BulkDeleteModalDeleteButton).getText()
+    ).to.equal('Delete 1 document');
 
     // Press delete
     await browser.clickVisible(Selectors.BulkDeleteModalDeleteButton);

packages/compass-e2e-tests/tests/collection-bulk-update.test.ts

@@ -67,17 +67,17 @@ describe('Bulk Update', () => {
     });
 
     // Make sure the query is shown in the modal.
-    expect(browser.$(Selectors.BulkUpdateReadonlyFilter).getText()).to.equal(
-      '{ i: 5 }'
-    );
+    expect(
+      await browser.$(Selectors.BulkUpdateReadonlyFilter).getText()
+    ).to.equal('{ i: 5 }');
 
     // Check that it will update the expected number of documents
-    expect(browser.$(Selectors.BulkUpdateTitle).getText()).to.equal(
-      'Update 1 document'
-    );
-    expect(browser.$(Selectors.BulkUpdateUpdateButton).getText()).to.equal(
+    expect(await browser.$(Selectors.BulkUpdateTitle).getText()).to.equal(
       'Update 1 document'
     );
+    expect(
+      await browser.$(Selectors.BulkUpdateUpdateButton).getText()
+    ).to.equal('Update 1 document');
 
     // Check that the modal starts with the default update text
     expect(
@@ -217,9 +217,9 @@ describe('Bulk Update', () => {
     await browser.$(Selectors.BulkUpdateModal).waitForDisplayed();
 
     // Make sure the query is shown in the modal.
-    expect(browser.$(Selectors.BulkUpdateReadonlyFilter).getText()).to.match(
-      /{\s+i:\s+{\s+\$gt:\s+5\s+}\s+}/
-    );
+    expect(
+      await browser.$(Selectors.BulkUpdateReadonlyFilter).getText()
+    ).to.match(/{\s+i:\s+{\s+\$gt:\s+5\s+}\s+}/);
 
     // Check that the modal starts with the expected update text
     expect(

packages/compass-e2e-tests/tests/collection-import.test.ts

@@ -304,8 +304,10 @@ describe('Collection import', function () {
 
     // confirm
     // this selector is very brittle, so just make sure it works
-    expect(browser.$(Selectors.InsertConfirm).isDisplayed()).to.be.true;
-    expect(browser.$(Selectors.InsertConfirm).getText()).to.equal('Insert');
+    expect(await browser.$(Selectors.InsertConfirm).isDisplayed()).to.be.true;
+    expect(await browser.$(Selectors.InsertConfirm).getText()).to.equal(
+      'Insert'
+    );
     await browser.$(Selectors.InsertConfirm).waitForEnabled();
     await browser.clickVisible(Selectors.InsertConfirm);
 

packages/compass-e2e-tests/tests/connection-form.test.ts

@@ -104,9 +104,9 @@ describe('Connection form', function () {
 
     delete expectedState.connectionString;
     await browser.setConnectFormState(expectedState);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
-      connectionString
-    );
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(connectionString);
   });
 
   it('parses and formats a URI for multiple hosts', async function () {
@@ -140,9 +140,9 @@ describe('Connection form', function () {
 
     delete expectedState.connectionString;
     await browser.setConnectFormState(expectedState);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
-      connectionString
-    );
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(connectionString);
   });
 
   it('parses and formats a URI for mongodb+srv scheme', async function () {
@@ -176,9 +176,9 @@ describe('Connection form', function () {
 
     delete expectedState.connectionString;
     await browser.setConnectFormState(expectedState);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
-      connectionString
-    );
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(connectionString);
   });
 
   it('parses and formats a URI for username/password authentication', async function () {
@@ -310,9 +310,9 @@ describe('Connection form', function () {
 
     delete expectedState.connectionString;
     await browser.setConnectFormState(expectedState);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
-      connectionString
-    );
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(connectionString);
   });
 
   it('parses and formats a URI for LDAP authentication', async function () {
@@ -477,9 +477,9 @@ describe('Connection form', function () {
 
     delete expectedState.connectionString;
     await browser.setConnectFormState(expectedState);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
-      connectionString
-    );
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(connectionString);
   });
 
   it('does not update the URI for SSH tunnel with password authentication', async function () {

packages/compass-e2e-tests/tests/show-kerberos-password-field.test.ts

@@ -52,7 +52,9 @@ describe('showKerberosPasswordField', function () {
       authMethod: 'GSSAPI',
     };
     await browser.setConnectFormState(state);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(
       'mongodb://localhost:27017/?authMechanism=GSSAPI&authSource=%24external'
     );
 
@@ -71,7 +73,9 @@ describe('showKerberosPasswordField', function () {
       authMethod: 'GSSAPI',
     };
     await browser.setConnectFormState(state);
-    expect(browser.$(Selectors.ConnectionFormStringInput).getValue()).to.equal(
+    expect(
+      await browser.$(Selectors.ConnectionFormStringInput).getValue()
+    ).to.equal(
       'mongodb://localhost:27017/?authMechanism=GSSAPI&authSource=%24external'
     );