Fix hardware model delete functionality (6.15.z)

[dosas]

For foreman 3.9 (6.15.z) the 'Actions' header string was removed (see picture):

The delete action is now accessible via an svg.

This should fix https://github.com/SatelliteQE/robottelo/blob/master/tests/foreman/ui/test_hardwaremodel.py

[omkarkhatavkar]

@lhellebr @jameerpathan111 @LadislavVasina1 @pondrejk Can anyone from your team run these PRT tests for this PR?

[lhellebr]

Have you verified this change is intended?
Where else is this used, what tests should we run PRT for? Just ui/test_hardwaremodel.py::test_positive_end_to_end?

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/ui/test_hardwaremodel.py::test_positive_end_to_end

[dosas]

The mentioned test will still fail since there is a follow up error:

AssertionError: assert_message: failed to find matching notifications. Available notifications: []

The error fixed within this PR is about not being able to open the delete pop-up

[omkarkhatavkar]

@dosas Thanks for giving the fix for this. I verified the test. It seems there there is failure in case of deleting the already used model with wrong error message. While I was still able to record the test execution see that delete it working okay.

[omkarkhatavkar]

@dosas will wait for any other opinion, else we will merge this PR.

[lhellebr]

The test fails with error Request failed with status code 422 because the hardware model is already in use. Actually, a different error is expected:

        if err_message:
            view.flash.assert_message(f"Danger alert: {err_message}")

That's an issue unrelated to the PR that should be fixed:

1. A correct error message should be expected
2. The message should say what the problem was, not just 422

[dosas]

Despite the label this was never cherry-picked into 6.15.z. Does somebody know what went wrong?