Check Hypervisor host subscription status and hypervisor host and virtual guest mapping in UI

[yanpliu]

Test Cases: PASS

(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_hyperv_sca.py  -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 17 deselected, 11 warnings in 461.63s (0:07:41)
=============================================================================
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_esx_sca.py  -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 17 deselected, 11 warnings in 461.63s (0:07:41)
2024-06-04 03:49:38 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_libvirt_sca.py  -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 2 deselected, 11 warnings in 534.11s (0:08:54)
2024-06-04 04:15:38 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_nutanix_sca.py  -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 6 deselected, 11 warnings in 530.85s (0:08:50)
2024-06-04 04:27:38 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_kubevirt_sca.py  -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 2 deselected, 11 warnings in 634.01s (0:10:34)
2024-06-04 04:39:01 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.

[vsedmik]

Codewisely the changes look good, I just wonder why we copy-paste the code 5-times? Even the previous code of these test cases looks the same. Would it make sense to use some helper or parametrize the test case so that we don't have same test/code in 5 different test modules?

[yanpliu]

Codewisely the changes look good, I just wonder why we copy-paste the code 5-times? Even the previous code of these test cases looks the same. Would it make sense to use some helper or parametrize the test case so that we don't have same test/code in 5 different test modules?

@vsedmik Thank you very much for your suggestions, have already optimized the duplicated code

[yanpliu]

Codewisely the changes look good, I just wonder why we copy-paste the code 5-times? Even the previous code of these test cases looks the same. Would it make sense to use some helper or parametrize the test case so that we don't have same test/code in 5 different test modules?

@vsedmik Thank you very much for your suggestions, have already optimized the duplicated code

Test Results: PASS

(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_esx_sca.py -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.         
 1 passed, 17 deselected, 11 warnings in 688.48s (0:11:28) 
2024-06-12 05:23:13 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_hyperv_sca.py -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 2 deselected, 11 warnings in 617.46s (0:10:17)
2024-06-12 20:26:01 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.
(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_libvirt_sca.py -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 2 deselected, 11 warnings in 548.94s (0:09:08)
2024-06-12 20:41:43 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.

(robottelo_vv_master) [root@dell-per740-68-vm-05 robottelo]# pytest ./tests/foreman/virtwho/ui/test_nutanix_sca.py -k test_positive_deploy_configure_by_id_script[id] --disable-pytest-warnings -q
.                                                                                                                                                                                                           [100%]
1 passed, 6 deselected, 11 warnings in 532.25s (0:08:52)
2024-06-12 21:25:49 - robottelo - WARNING - missing grid_url or session_id. unable to clean video files.