read recommended repositories by it types

[vijaysawant]

Problem Statement
No support presented for recommended kickstart repositories in airgun entities & in views.

Solution
Update RedHatRepositoryEntity read method and RedHatRepositoriesView with required changes.

Related robottelo PR
SatelliteQE/robottelo#19676

[sourcery-ai]

Hey there - I've reviewed your changes - here's some feedback:

• Before selecting filter_type in read(), clear any existing filter selections to avoid carrying over stale UI state across calls.
• The locator for search_by_filter_type uses an index-based bs-select-2; consider switching to a more robust selector (e.g. data attributes or a unique class) to reduce fragility if the DOM order changes.
• With multiple optional search parameters (entity_name, category, recommended_repo, filter_type), you might simplify the signature by accepting a single options object or dict for better scalability.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Before selecting filter_type in read(), clear any existing filter selections to avoid carrying over stale UI state across calls.
- The locator for search_by_filter_type uses an index-based bs-select-2; consider switching to a more robust selector (e.g. data attributes or a unique class) to reduce fragility if the DOM order changes.
- With multiple optional search parameters (entity_name, category, recommended_repo, filter_type), you might simplify the signature by accepting a single options object or dict for better scalability.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[LadislavVasina1]

@vijaysawant, I thought you had confirmed via slack that view.search_by_filter_type.fill(['string1', 'string2']) worked for selecting RPM, Kickstart, etc.
Is it not true already?

[vijaysawant]

@vijaysawant, I thought you had confirmed via slack that view.search_by_filter_type.fill(['string1', 'string2']) worked for selecting RPM, Kickstart, etc. Is it not true already?

Hey @LadislavVasina1, The solution you suggested worked (passing list of strings). But as per nature of robottelo test case, passing single value to view.search_by_filter_type.fill() important as we are using parameterisation. That is the reason I didn't use list here.

[LadislavVasina1]

@vijaysawant Can you link the test you are talking about please?

[vijaysawant]

@vijaysawant Can you link the test you are talking about please?

Please see description, robottelo PR 19676 is open.

[sambible]

ACK Pending explanation of my comment. I'm assuming it has something to do with loading/navigation, but I'm not sure.