Skip to content
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

feat: expose WaitVisible method from the rod library #1101

Merged
merged 5 commits into from
Oct 13, 2021
Merged

feat: expose WaitVisible method from the rod library #1101

merged 5 commits into from
Oct 13, 2021

Conversation

pmareke
Copy link
Contributor

@pmareke pmareke commented Oct 9, 2021

Fixes #1096

To-Do:

  • Add new Action called ActionWaitVisible
  • Add the implementation for ActionWaitVisible

Signed-off-by: pedro.lopez.mareque@gmail.com

@pmareke
Copy link
Contributor Author

pmareke commented Oct 9, 2021

Also do I need to add one test for this action? Done!

@ehsandeep ehsandeep changed the base branch from master to dev October 9, 2021 10:59
Copy link
Contributor

@forgedhallpass forgedhallpass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also add a negative test scenario where the timeout expires and the element is not found? Have you also tested it with a headless template?

@pmareke
Copy link
Contributor Author

pmareke commented Oct 9, 2021

Could you please also add a negative test scenario where the timeout expires and the element is not found? Have you also tested it with a headless template?

I added the timeout test but I don't know what do you mean with a headless template, can you give some example?

@ehsandeep ehsandeep added the Status: Review Needed The issue has a PR attached to it which needs to be reviewed label Oct 13, 2021
@forgedhallpass forgedhallpass merged commit 88ee74d into projectdiscovery:dev Oct 13, 2021
@forgedhallpass
Copy link
Contributor

@pmareke Thank you for your contribution!

I added the timeout test but I don't know what do you mean with a headless template, can you give some example?

  1. By timeout I meant an optional parameter for the WaitVisible method. I've added it in c0ef419, where I've also extracted the duplicated logic from your test.

  2. Nuclei is a vulnerability scanner that relies on rules defined in yaml templates.
    I've added an example in the ticket, see [feature] Add/Expose Headless action that waits until an element appears #1096 (comment)

@pmareke
Copy link
Contributor Author

pmareke commented Oct 13, 2021

@pmareke Thank you for your contribution!

I added the timeout test but I don't know what do you mean with a headless template, can you give some example?

  1. By timeout I meant an optional parameter for the WaitVisible method. I've added it in c0ef419, where I've also extracted the duplicated logic from your test.
  2. Nuclei is a vulnerability scanner that relies on rules defined in yaml templates.
    I've added an example in the ticket, see [feature] Add/Expose Headless action that waits until an element appears #1096 (comment)

Thanks to you @forgedhallpass, I really enjoyed the PR!

@forgedhallpass
Copy link
Contributor

Thanks to you @forgedhallpass, I really enjoyed the PR!

Very glad to hear that! I am looking forward to your future contributions in case you decide to do so. Let me know if your eyes catch something or if you have new ideas. Thanks again for your efforts :)

@forgedhallpass forgedhallpass added Status: Completed Nothing further to be done with this issue. Awaiting to be closed. hacktoberfest-accepted and removed Status: Review Needed The issue has a PR attached to it which needs to be reviewed labels Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Status: Completed Nothing further to be done with this issue. Awaiting to be closed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature] Add/Expose Headless action that waits until an element appears
3 participants