[6.18.z] Convert Lightspeed inventory sync from CLI to UI

[Satellite-QE]

Cherrypick of PR: #20028

Problem Statement

The CLI test for syncing Lightpseed inventory status is failing due to its reliance on the foreman-rake rh_cloud_inventory:sync rake task. This command syncs the inventory for all organization on a Satellite and does not respect options passed in to restrict the scope of the command to a single org. As a result, in CI, when the test searches for an inventory sync task on which to make assertions, it often erroneously finds a sync task for an organization other than the one being tested.

Additionally, syncing the inventory via the foreman-rake CLI is not a documented approach, and the foreman-rake CLI is typically not a supported customer use case.

Solution

Convert the CLI inventory sync test to a UI test, which uses a supported workflow and restricts the sync to the selected org by default.

[Satellite-QE]

trigger: test-robottelo
pytest: tests/foreman/ui/test_rhcloud_inventory.py::test_sync_inventory_status

[sourcery-ai]

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

• Consider extracting the shared wait_for + ForemanTask search logic into a reusable helper to reduce duplication across your inventory sync tests.
• It would strengthen the test to include a UI-based assertion (e.g. verifying host statuses or sync state in the UI) after the sync, not just checking the API task output.
• Capture the timestamp immediately after triggering the sync in the UI session rather than before, to avoid potential race conditions if UI interactions delay task execution.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Consider extracting the shared wait_for + ForemanTask search logic into a reusable helper to reduce duplication across your inventory sync tests.
- It would strengthen the test to include a UI-based assertion (e.g. verifying host statuses or sync state in the UI) after the sync, not just checking the API task output.
- Capture the timestamp immediately after triggering the sync in the UI session rather than before, to avoid potential race conditions if UI interactions delay task execution.

---

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.}

[Satellite-QE]

PRT Result

Build Number: 13240
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_rhcloud_inventory.py::test_sync_inventory_status --external-logging
Test Result : ================= 4 passed, 61 warnings in 1300.67s (0:21:40) ==================