Closed mrk-han closed 5 years ago
The default implementation of isPresentWait uses waitForCondition, which uses configuration.getTimeout() for the parameter long timeOutInSeconds.
isPresentWait
waitForCondition
configuration.getTimeout()
long timeOutInSeconds
However, this is not immediately obvious or noted in a Javadoc for isPresentWait.
Furthermore, isPresentWait also ties its retries to configuration.getRetries
configuration.getRetries
This is problematic because of the ambiguity in naming of timeout and retries. See: https://github.com/willowtreeapps/conductor-mobile/issues/10
timeout
retries
The default implementation of
isPresentWaituseswaitForCondition, which usesconfiguration.getTimeout()for the parameterlong timeOutInSeconds.However, this is not immediately obvious or noted in a Javadoc for isPresentWait.
Furthermore, isPresentWait also ties its retries to
configuration.getRetriesThis is problematic because of the ambiguity in naming of
timeoutandretries. See: https://github.com/willowtreeapps/conductor-mobile/issues/10