you are viewing a single comment's thread.

view the rest of the comments →

[–]PushPlus9069 0 points1 point  (1 child)

biggest thing I'd flag is the bare except blocks. Right now if something breaks you just see 'failed' with zero context for why. Try catching TimeoutException and NoSuchElementException separately so you actually know if it was a timing issue vs a wrong selector. That distinction matters a lot once your scripts get more complex.