Early exit for ExplicitWait

[tucknats]

Looking at the code for explicitWait inside findElement method, https://github.com/sukgu/shadow-automation-selenium/blob/master/src/main/java/io/github/sukgu/Shadow.java#L232, I see a couple of problems

1. If the element is found, the code should exit early otherwise we're waiting the full time. Even worse, the code is going to repeat the whole execution so many times.
2. Also, I am not sure if any of the 3 methods [executerGetObject, fixLocator, isPresent] will throw an exception if the element is not found. The whole point of 'polling' is that the calls need to be forgiving. If I set explicitWait=10, pollingTime=1, the method 'findElement' shouldn't throw an error on the first 9 tries.

if(explicitWait > 0) {
			element = (WebElement) executerGetObject(String.format("return getObject(\"%s\");", cssSelector));
			fixLocator(driver, cssSelector, element);
			visible = isPresent(element);
			
			for(int i = 0 ; i < explicitWait && !visible;) {
				try {
					Thread.sleep(pollingTime * 1000);
					element = (WebElement) executerGetObject(String.format("return getObject(\"%s\");", cssSelector));
					fixLocator(driver, cssSelector, element);
					visible = isPresent(element);
					i = i + pollingTime;
				} catch (InterruptedException e) {
					
				}
			}
		}

[tucknats]

NVM, completely missed the '!visible' part :)