save screenshot files only if they are present

e2e/util/util.js

@@ -335,13 +335,37 @@ exports.waitUntilUrlIsShowed = function (urlToWait, timeout) {
 exports.waitUntilElementIsVisible = function (elementToCheck, timeout) {
     var waitTimeout = timeout || DEFAULT_TIMEOUT;
 
-    return browser.wait(until.visibilityOf(elementToCheck), waitTimeout, 'Element is not visible ' + elementToCheck.locator());
+    this.waitUntilElementIsPresent(elementToCheck, timeout);
+
+    var isDisplayed = false;
+    return browser.wait(() => {
+        elementToCheck.isDisplayed().then(
+            () => {
+                isDisplayed = true;
+            },
+            (err) => {
+                isDisplayed = false;
+            }
+        );
+        return isDisplayed;
+    }, waitTimeout, 'Element is not visible ' + elementToCheck.locator());
 };
 
 exports.waitUntilElementIsPresent = function (elementToCheck, timeout) {
     var waitTimeout = timeout || DEFAULT_TIMEOUT;
 
-    return browser.wait(until.presenceOf(elementToCheck), waitTimeout, 'Element is not present ' + elementToCheck.locator());
+    var isPresent = false;
+    return browser.wait(() => {
+        elementToCheck.isPresent().then(
+            () => {
+                isPresent = true;
+            },
+            (err) => {
+                isPresent = false;
+            }
+        );
+        return isPresent;
+    }, waitTimeout, 'Element is not present ' + elementToCheck.locator());
 };
 
 /**

e2e/z-finish.e2e.ts

@@ -40,31 +40,32 @@ describe('Save screenshot at the end', () => {
                 hostEcm: TestConfig.adf.url
             });
 
-            alfrescoJsApi.login(TestConfig.adf.adminEmail, TestConfig.adf.adminPassword);
-
-            let folder = await alfrescoJsApi.nodes.addNode('-my-', {
-                'name': 'Screenshot-e2e-' + buildNumber,
-                'nodeType': 'cm:folder'
-            }, {}, {});
-
             let files = fs.readdirSync(path.join(__dirname, '../e2e-output/screenshots'));
 
-            for (const fileName of files) {
+            if (files && files.length > 0) {
+                alfrescoJsApi.login(TestConfig.adf.adminEmail, TestConfig.adf.adminPassword);
 
-                let pathFile = path.join(__dirname, '../e2e-output/screenshots', fileName);
+                let folder = await alfrescoJsApi.nodes.addNode('-my-', {
-                let file = fs.createReadStream(pathFile);
+                    'name': 'Screenshot-e2e-' + buildNumber,
+                    'nodeType': 'cm:folder'
+                }, {}, {});
 
-                await  alfrescoJsApi.upload.uploadFile(
+                for (const fileName of files) {
-                    file,
-                    '',
-                    folder.entry.id,
-                    null,
-                    {
-                        'name': file.name,
-                        'nodeType': 'cm:content'
-                    }
-                );
 
+                    let pathFile = path.join(__dirname, '../e2e-output/screenshots', fileName);
+                    let file = fs.createReadStream(pathFile);
+
+                    await  alfrescoJsApi.upload.uploadFile(
+                        file,
+                        '',
+                        folder.entry.id,
+                        null,
+                        {
+                            'name': file.name,
+                            'nodeType': 'cm:content'
+                        }
+                    );
+                }
             }
         }