Minor browser updates - Githubissues

IntelliTect / TestTools

A collection of tools for aiding in test automation

MIT License

10 stars 7 forks source link

Minor browser updates #28

Closed PandaMagnus closed 5 years ago

PandaMagnus commented 5 years ago

Update to Browser.WaitUntil. It was using an old version of the logic from before we could specify a return type. This version should be more accurate for 80% - 90% of all scenarios in Selenium.

PandaMagnus commented 5 years ago

Found some conditions where this doesn't work... going to do some more work on this this weekend.

PandaMagnus commented 5 years ago

Okay, issue is fixed. @adamskt @Keboo @kamonson any of you able to review soon-ish? I need to break out the Wait class into its own package for Verint and Avista, and I'd prefer to not have to deal with merging over this PR if possible. :)

PandaMagnus commented 5 years ago

Abandoning PR. Pivoted on the project a bit to work on other stuff first, and potentially will be shuffling some responsibility around.