Fix clean button locator in PF4 search.

[lhellebr]

As it was, we were getting an error when searching for the element, getting False for is_displayed. Consequently, the clear button was not clicked and a generic browser clear was called instead. The text disappeared for a bit and then it got re-filled to the search field.
It caused failures in tests such as
tests/foreman/ui/test_dashboard.py::test_positive_task_status
Changing type and locator fixes the issue.

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/ui/test_dashboard.py::test_positive_task_status tests/foreman/ui/test_contentview.py::test_positive_end_to_end tests/foreman/ui/test_media.py::test_positive_end_to_end

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/ui/test_dashboard.py::test_positive_task_status tests/foreman/ui/test_contentview.py::test_positive_add_rh_content tests/foreman/ui/test_media.py::test_positive_end_to_end

[lhellebr]

Cause of failure unrelated, it also fails in snap automation:

>           raise NoSuchElementException(f"Could not find an element {repr(locator)}") from None
E           selenium.common.exceptions.NoSuchElementException: Message: Could not find an element Locator(by='xpath', locator="//a[contains(@href, '/content_views/new')]"); For documentation on this error, please visit: https://www.selenium.dev/documentation/webdriver/troubleshooting/errors#no-such-element-exception

Rerunning without the offending test.

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/ui/test_dashboard.py::test_positive_task_status tests/foreman/ui/test_media.py::test_positive_end_to_end

[lhellebr]

The failures in PRT don't seem related. They only fail in master, not 6.14.z, and the same failure happens in master when I revert my commit.

I wanted to use PRT to check whether I cause regression or not but in the end, I ran on 6.14.z branch (with this commit cherrypicked) locally:

$ pytest tests/foreman/ui/test_dashboard.py::test_positive_task_status  tests/foreman/ui/test_media.py::test_positive_end_to_end
[...]
========================================= 2 passed, 6 warnings in 211.27s (0:03:31) ==========================================

There are many navigation related issues in Stream so I think this is merge worthy as it is. Alternatively, we can only merge into 6.14.z (that means future work).

[adarshdubey-star]

Do we need it in older satellite versions as well @lhellebr?

[lhellebr]

In 6.13.z branch, the locator is clear_button = Text(locator=(".//button[contains(@class,'autocomplete-clear-button')]")) - different enough for me to consider it a different issue (if it even is an issue).