Protractor, with isDisplayed() I get NoSuchElementError: No element found using locator
Asked Answered
C

4

23

In protractor 2.0, I am checking in a expect() if one element is displayed. I expect a false, but the weird thing is that I get following error:

NoSuchElementError: No element found using locator: By.id("userForm")

My code is:

describe('closeModal', function() {
    it('should close the alert that appears after registration.', function(){
        element(by.id('closeAlertModalButton')).click();
        expect(element(by.id('userForm')).isDisplayed()).toBeFalsy();
    });
});

I understand that I get that error because element is not longer on the page (is what I want to confirm), but shouldn't I get a false and not a error?

Candlepin answered 7/5, 2015 at 11:35 Comment(1)
You should better use isElementPresent or isPresent - error you've got is part of WebDriver logicJaquith
P
36

isDisplayed() would check if an element is visible or not, but you need to check whether an element is present in DOM or not, use isElementPresent() or isPresent():

expect(browser.isElementPresent(element(by.id('userForm')))).toBe(false);
expect(element(by.id('userForm')).isPresent()).toBe(false);

See also:

Possession answered 7/5, 2015 at 11:42 Comment(4)
You're a "protractor angel!" I've learned a lot from your answers here! thank you! @PossessionTambratamburlaine
@IdanE and I've learned a lot from your questions! Thank you!Possession
If i do it in condition. it is not working. ex. if(browser.isElementPresent(element(by.id('userForm')))Rochkind
@Rochkind browser.isElementPresent(element(by.id('userForm')) returns a promise which is always "truthy". You should explicitly resolve the promise to check the actual presence value.Possession
J
2

This error is part of WebDriver behavior. For such cases you should better use isPresent or isElementPresent

Jaquith answered 7/5, 2015 at 11:42 Comment(0)
M
2

If element visible do A if not visible do B, disregard exception if element not found:

element.isDisplayed().then(function(visible){
    if (visible) {
        // do A when element visible
    }else{
        // do B when element not visible 
    }
}, function () {
    //suppress exception if element is not found on page
});
Mneme answered 10/4, 2018 at 17:8 Comment(0)
F
1

.isDisplayed() assumes the element is present (exists in the DOM)

so if you do

expect($('[ng-show=saving]').isDisplayed()).toBe(true);

but the element is not present, then instead of graceful failed expectation, $('[ng-show=saving]').isDisplayed() will throw an error causing the rest of it block not executed

Solution

If you assume, the element you're checking may not be present for any reason on the page, then go with a safe way below

/**
*  element is Present and is Displayed
*  @param    {ElementFinder}      $element       Locator of element
*  @return   {boolean}
*/
let isDisplayed = function ($element) {
  return (await $element.isPresent()) && (await $element.isDisplayed())
}

and use

expect(await isDisplayed( $('[ng-show=saving]') )).toBe(true);
Fondly answered 10/2, 2021 at 0:1 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.