Disassociate Hosts Coverage [SAT-31029]

[LadislavVasina1]

Problem Statement

There is a new feature that enables users to disassociate multiple hosts via the All Hosts page. (SAT-31029)

Solution

Create a test test_disassociate_multiple_hosts that covers this use case.

It creates a VMware compute resource, imports 2 VMs,
(Thank you @Gauravtalreja1 for the info on how to do that, this part of the test is inspired by test_positive_virt_card)
and disassociates them via the UI.
Checks that the disassociation went well are done via the API, and they are checking values based on this https://github.com/theforeman/foreman/blob/f4246cdaca1d0039d93de65cda9e52862c02f81b/app/models/host/managed.rb#L589

Related Issues

Needs:
SatelliteQE/airgun#1909
theforeman/foreman#10560

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k 'test_disassociate_multiple_hosts'
airgun: 1909
theforeman:
    foreman: 10560

[LadislavVasina1]

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k 'test_disassociate_multiple_hosts'
airgun: 1909
theforeman:
    foreman: 10560
env:
    ROBOTTELO_ui__record_video: true

[Satellite-QE]

PRT Result

Build Number: 11963
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_host.py -k test_disassociate_multiple_hosts --external-logging
Test Result : ========== 1 passed, 81 deselected, 14 warnings in 1513.75s (0:25:13) ==========

[LadislavVasina1]

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k 'test_disassociate_multiple_hosts'
airgun: 1909
theforeman:
    foreman: 10560
env:
    ROBOTTELO_ui__record_video: true

[Satellite-QE]

PRT Result

Build Number: 11968
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py -k test_disassociate_multiple_hosts --external-logging
Test Result : ========== 1 failed, 81 deselected, 9 warnings in 1262.79s (0:21:02) ===========

[LadislavVasina1]

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py -k 'test_disassociate_multiple_hosts'
airgun: 1909
theforeman:
    foreman: 10560
env:
    ROBOTTELO_ui__record_video: true

[Satellite-QE]

PRT Result

Build Number: 11974
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py -k test_disassociate_multiple_hosts --external-logging
Test Result : ========== 1 failed, 81 deselected, 9 warnings in 1273.21s (0:21:13) ===========

[LadislavVasina1]

It seems that the PRT is failing due to selenium UI session is obfuscated by the breached password warning; for more reference, see 1.7.2025 JPL meeting notes. (PRT managed to pass at least once ath the beginning here #18859 (comment))

[LadislavVasina1]

@pnovotny, thank you for the suggestions, I will push the changes in a minute

[LadislavVasina1]

[pnovotny]

LGTM

[vsedmik]

Looks good to me, great work!

I put only one non-blocking nit bellow.

[vsedmik]

This is the second definition of hostgroup_name. I would just remove the one from line 3021 and keep this one.

[LadislavVasina1]

@vsedmik True, thanks for that, I missed this.