robottelo
robottelo copied to clipboard
First Ansible Roles API test
Requires https://github.com/SatelliteQE/nailgun/pull/841
First test surrounding Ansible Roles
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
@sambible any updates on this PR?
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
Addressed comments and implemented @ogajduse change request. Going to run the test again with runner, but should be good to go when we can merge the Nailgun PR
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role' nailgun: 841
Nailgun PR is merged, this can go In now.
@sambible rebase and we can merge!
I guess we need to rebase again
trigger: test-robottelo pytest: tests/foreman/api/test_ansible.py -k 'test_positive_assign_ansible_role'
@sambible Can you check on the PRT failure and revisit this PR? It would be nice to have this merged.
I vote we just close this PR as tests/foreman/api/test_ansible.py::test_positive_ansible_job_on_host has already covered what this test is doing. There are also equivalent tests in CLI and UI now, so I don't see the need to keep this test when it is already set up for those other tests.