update default method to check if element displayed to always return …

htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/HtmlElement.java

@@ -208,7 +208,12 @@ public WebElement findElement(By by) {
      */
     @Override
     public boolean isDisplayed() {
-        return wrappedElement.isDisplayed();
+        try{
+            return getWrappedElement().isDisplayed();
+        } catch (Exception e) {
+            //ignore and return false to indicate element is not visible.
+        }
+        return false;
     }
 
     /**

htmlelements-java/src/main/java/ru/yandex/qatools/htmlelements/element/TypifiedElement.java

@@ -91,9 +91,15 @@ public String toString() {
      * @return Whether or not the element is displayed
      */
     public boolean isDisplayed() {
-        return getWrappedElement().isDisplayed();
+        try{
+            return getWrappedElement().isDisplayed();
+        } catch (Exception e) {
+            //ignore and return false to indicate element is not visible.
+        }
+        return false;
     }
 
+
     /**
      * Is the element currently enabled or not? This will generally return true for everything but
      * disabled input elements.

htmlelements-java/src/test/java/ru/yandex/qatools/htmlelements/ElementDisplayedTest.java

@@ -0,0 +1,58 @@
+package ru.yandex.qatools.htmlelements;
+
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import org.junit.Assert;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.ExternalResource;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
+import org.openqa.selenium.phantomjs.PhantomJSDriver;
+
+import ru.yandex.qatools.htmlelements.element.HtmlElement;
+import ru.yandex.qatools.htmlelements.element.TypifiedElement;
+import ru.yandex.qatools.htmlelements.loader.HtmlElementLoader;
+import ru.yandex.qatools.htmlelements.testelements.DummyBlock;
+
+/**
+ * Test that validates the {@link TypifiedElement} isDisplayed method always
+ * returns a boolean.
+ * @author Michael Suzuki
+ */
+public class ElementDisplayedTest {
+
+    private WebDriver driver;
+
+    @Rule
+    public ExternalResource manage = new ExternalResource() {
+        @Override
+        protected void before() throws Throwable {
+            driver = new PhantomJSDriver();
+        }
+
+        @Override
+        protected void after() {
+            driver.quit();
+        }
+    };
+
+    @Test
+    public void shouldReturnFalseAndNotAnException() throws Exception {
+        Assert.assertFalse(HtmlElementLoader.create(DummyBlock.class, driver).isDisplayed());
+    }
+
+    @Test
+    public void shouldReturnTrueForVisibleElements() throws Exception {
+        WebElement element = mock(WebElement.class);
+        when(element.isDisplayed()).thenReturn(true);
+        HtmlElement elementVisble = new HtmlElement();
+        elementVisble.setWrappedElement(element);
+        Assert.assertTrue(elementVisble.isDisplayed());
+
+        when(element.isDisplayed()).thenReturn(false);
+        Assert.assertFalse(elementVisble.isDisplayed());
+    }
+
+}