Update API endpoint for host facts

[shweta83]

Problem Statement

API endpoint for facts values : /api/hosts/:host_id/facts was incorrectly placed causing test failure.

Solution

Updated.

Related Issues

[shweta83]

trigger: test-robottelo
pytest: tests/foreman/endtoend/test_api_endtoend.py::TestAvailableURLs::test_positive_get_links

[Satellite-QE]

PRT Result

Build Number: 7597
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/endtoend/test_api_endtoend.py::TestAvailableURLs::test_positive_get_links --external-logging
Test Result : ================== 1 failed, 9 warnings in 681.28s (0:11:21) ===================

[pondrejk]

trigger: test-robottelo
pytest: tests/foreman/endtoend/test_api_endtoend.py::TestAvailableURLs::test_positive_get_links
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.15'

[Satellite-QE]

PRT Result

Build Number: 7617
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/endtoend/test_api_endtoend.py::TestAvailableURLs::test_positive_get_links --external-logging
Test Result : ======================== 6 warnings, 1 error in 55.94s =========================

[shweta83]

The reason of failure is the API endpoint is missing from /api/v2. After discussion with developer, the json created in /api/v2 gets automatically generated by rails and it usually fetches few API endpoints. This is not a very reliable source of truth to check all endpoints and he has suggested to test apidoc/v2 instead.
Any thoughts here @jyejare @JacobCallahan ?

[pondrejk]

Hi, aside from what was said above (I'm definitely for finding a better way to do this -- possibly by using https://github.com/JacobCallahan/apix), the humble aim of test_positive_get_links is to let us know about api changes on the product side (only first level depth, but still something). The current failing state on 6.15 was introduced by https://github.com/SatelliteQE/robottelo/pull/15265/files#diff-99f40169f11495c51e665ec0a4299a4f4206cd4c72664afd264e227987ea95c8R358 not by the product change. So I'd vote for restoring the match between API_PATHS and api/v2 so that the test can perform its original purpose.

[shweta83]

Hi, aside from what was said above (I'm definitely for finding a better way to do this -- possibly by using https://github.com/JacobCallahan/apix), the humble aim of test_positive_get_links is to let us know about api changes on the product side (only first level depth, but still something). The current failing state on 6.15 was introduced by https://github.com/SatelliteQE/robottelo/pull/15265/files#diff-99f40169f11495c51e665ec0a4299a4f4206cd4c72664afd264e227987ea95c8R358 not by the product change. So I'd vote for restoring the match between API_PATHS and api/v2 so that the test can perform its original purpose.

Sure thing. I can remove the endpoint for now so that the sanity passes and we can come up with another solution.