refactor `crawler.py::do_something()`

[nnrepos]

the function do_something is not very clear:

• it interacts with with elements on the driver's page, clicks it /send_keys to it, and then returns the attributes of the first child that was not interacted with yet.
• it relies on a global variable already_clicked_elems.
• it doesn't have a docstring or a meaningful name.

i think that, before creating a test for that method, we should first refactor it. a few options:

• refactor crawler.py into a Crawler class, and keep the already_clicked_elems as a member;
• refactor just do_something by making it yield a child's attributes, that way a set is no longer needed;
• leave the global variable there for now and just focus on documentation.

either way, i think the function should be:

• renamed to something like interact_with_clickable_chlid,
• given proper documentation + docstring,
• given type hints.

let me know what you think 🧇

[marco-c]

i think that, before creating a test for that method, we should first refactor it. a few options:

* refactor `crawler.py` into a `Crawler` class, and keep the `already_clicked_elems` as a member;

* refactor just `do_something` by making it `yield` a child's attributes, that way a `set` is no longer needed;

* leave the global variable there for now and just focus on documentation.

Let's start by adding documentation, as that's the definitely necessary step.

either way, i think the function should be:

* renamed to something like `interact_with_clickable_chlid`,

The function is supposed to be performing some random action on a page. interact_with_clickable_child is too narrow, maybe something like perform_action_on_page would work better?

[nnrepos]

The function is supposed to be performing some random action on a page. interact_with_clickable_child is too narrow, maybe something like perform_action_on_page would work better?
i guess the initial intention of this function was to do something arbitrary, but right now it does something specific. i think it's better to document what a function actually does, rather than what it was meant to do. perform_action_on_page is still not very meaningful, but i'll do as you ask 🥲