don't fail lazyloading for isVisible and isExisting - closes #2003

lib/webdriverio.js

@@ -352,9 +352,10 @@ let WebdriverIO = function (args, modifier) {
                      * otherwise the context would be ignored and a different element could
                      * selected.
                      * Ignore this behavior for all waitFor calls as we want to refetch elements
-                     * in case they haven't been found.
+                     * in case they haven't been found. Also not throw an error for isExisting
+                     * and isVisible since they use that information to return their result.
                      */
-                    if (!isWaitFor && lastResult.state === 'failure') {
+                    if (!isWaitFor && !name.match(/^is(Existing|Visible)/) && lastResult.state === 'failure') {
                         let message = lastResult.message
 
                         /**

test/fixtures/specs/element.spec.js

@@ -23,6 +23,23 @@ describe('element as first class citizen', () => {
         expect(browser.element('.moreNesting').element('.findme').getText()).to.be.equal('MORE NESTED')
     })
 
+    it('can lazy load elements but does not fail for isExisting and isVisible', () => {
+        const elem = $('#notExisting')
+        expect(elem.isExisting()).to.be.equal(false)
+        expect(elem.isVisible()).to.be.equal(false)
+
+        let message
+        try {
+            elem.click()
+        } catch (e) {
+            message = e.message
+        }
+
+        expect(message).to.be.equal(
+            'An element could not be located on the page using the given search parameters ("#notExisting").'
+        )
+    })
+
     it('should allow to call waitForExist on elements result', () => {
         let element = browser.elements('//div[text()="Sorry, I\'m late!"]')
         let start = new Date().getTime()