Code review comment for lp://qastaging/~abreu-alexandre/webbrowser-app/add-ap-tests-for-navigation-and-popup

Revision history for this message
Olivier Tilloy (osomon) wrote :

445 + self.assertThat(lambda: webview.url,
446 + Eventually(Contains("/href-link")))

In case like this where comparing the value of a property on an object, the use of a lambda is superfluous, i.e. this will work:

    self.assertThat(webview.url, Eventually(Contains("/href-link")))

There are a number of places in the new tests that can be simplified.

« Back to merge proposal