fix(tests): no async forEach in merge patients

Removed the forEach() async function that caused spurious test timeouts. It's now a for...of loop.
Third-Culture-Software · Aug 6, 2019 · c4b257a · c4b257a
1 parent b78ab2a
commit c4b257a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/test/end-to-end/patient/registry.merge.page.js b/test/end-to-end/patient/registry.merge.page.js
@@ -15,9 +15,9 @@ function PatientMergePage() {
   };
 
   page.gridSelectRows = async function gridSelectRows(...lines) {
-    lines.forEach(async element => {
-      await GU.selectRow(gridId, element);
-    });
+    for (const line of lines) { // eslint-disable-line
+      await GU.selectRow(gridId, line); // eslint-disable-line
+    }
   };
 
   page.selectPatientToKeep = async function selectPatientToKeep(reference) {

diff --git a/test/end-to-end/patient/registry.merge.spec.js b/test/end-to-end/patient/registry.merge.spec.js
@@ -25,5 +25,4 @@ describe('Merge Patients', () => {
     await Page.merge();
     await components.notification.hasSuccess();
   });
-
 });
-Original file line number
+Diff line change
@@ Expand Up / @@ -25,5 +25,4 @@ describe('Merge Patients', () => { @@
         await Page.merge();
         await components.notification.hasSuccess();
       });
     });