Fix parametrization in test_positive_reboot_all_pxe_hosts

[Gauravtalreja1]

Problem Statement

Missing indirect=True in parametrization for provision_multiple_hosts fixture which causes,

tests/foreman/api/test_discoveredhost.py:427: in test_positive_reboot_all_pxe_hosts
    for host in provision_multiple_hosts:
E   TypeError: 'int' object is not iterable

Solution

using getattr to set count as default for this fixture, and removing @pytest.mark.parametrize for this fixture, along with mark for pxe_loader fixture which also has bios set as default.

[Gauravtalreja1]

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_reboot_all_pxe_hosts
provisioning: true

[Gauravtalreja1]

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_reboot_all_pxe_hosts
provisioning: true

[JacobCallahan]

ACK on the intent of the changes, but can you determine why the test is failing in PRT? If unrelated to your change, then the failure isn't a blocker.

[Gauravtalreja1]

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_reboot_all_pxe_hosts
provisioning: true

[Satellite-QE]

PRT Result

Build Number: 5698
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_discoveredhost.py -k test_positive_reboot_all_pxe_hosts
Test Result : =========== 14 deselected, 10 warnings, 1 error in 863.88s (0:14:23) ===========

[Gauravtalreja1]

Yes, the failure isn't related to the changes of this PR, but it is related to a BZ 2264195, I've opened a quick fix theforeman/foreman_discovery#623 in upstream already and handled the changes here.
And, now this PR is ready for review