#90: refactored collection index to not use Should(Have.Count(...)

yashaka / NSelene

Consise API to Selenium for .Net (the port of Selene in python / Selenide in Java)

MIT License

68 stars 21 forks source link

#90: refactored collection index to not use Should(Have.Count(...) #103

Closed wjgerritsen-0001 closed 2 days ago

wjgerritsen-0001 commented 1 month ago

Refactor the index method in the Collection. Now it will only timeout in case of real failure, see added testcase.

Not fully sure about the error message, see below:

Timed out after 0,25s, while waiting for:
    Browser.All(#will-not-appear)[100].Should(Be.InDom)
Reason:
    element was not found in collection by index 100 (actual collection size is 0)

yashaka commented 1 month ago

Will try to review it this weekend! thank you for the PR!