New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Decoupling #124
base: develop
Are you sure you want to change the base?
Decoupling #124
Conversation
It's hard to see the changes here because of the changes in develop. Can you rebase/merge them? |
Started decoupling functionality from the main Locomotiveô class
removing .classpath file tracking Updated gitignore to not track .classpath files
@evant done, my bad, used to doing trunk based development and I forget that |
return isPresentWait(PageUtil.buildBy(config, id)); | ||
} | ||
|
||
public boolean isPresentWait(By by) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the functionality pre-0.19.0, something may have gotten hecked up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah you are right, I must've hecked something up with the rebase. I wanted to keep the Conductor
class name going forward but yeah I hecked it up.
…tion (#113) * Refactor isPresentWait * Add Exceptions to error messages * Remove old isPresentWait() tests - test_is_present_wait_retries_and_fail() - test_is_present_wait_retries_and_find_item() They were testing an old implementation with custom logic. Now we are only giving a wrapper around battle tested Appium methods so none are needed right now.
Related Issue
Put issue number in this format
closes #XXXX
to auto-close the issue or task this PR addresses (if true).Description
Please write a short description of your changes.
Additional Info
Provide any additional info here.
Checklist