search

SatelliteQE / robottelo

Robottelo is a test suite that exercises The Foreman.
GNU General Public License v3.0
61 stars 114 forks source link

Update API e2e test to use RegistrationCommand entity to register #16914

Closed Gauravtalreja1 closed 1 week ago

Gauravtalreja1 commented 1 week ago

Problem Statement

API e2e test to use hammer to register instead of API/nailgun

Solution

Related Issues

https://github.com/SatelliteQE/nailgun/pull/1225

Gauravtalreja1 commented 1 week ago

trigger: test-robottelo pytest: tests/foreman/endtoend/test_api_endtoend.py -k test_positive_end_to_end nailgun: 1225

Satellite-QE commented 1 week ago

PRT Result

Build Number: 9359
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/endtoend/test_api_endtoend.py -k test_positive_end_to_end --external-logging
Test Result : ========== 1 passed, 5 deselected, 18 warnings in 5434.66s (1:30:34) ===========
Gauravtalreja1 commented 1 week ago

trigger: test-robottelo pytest: tests/foreman/endtoend/test_api_endtoend.py -k test_positive_end_to_end

Satellite-QE commented 1 week ago

PRT Result

Build Number: 9367
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/endtoend/test_api_endtoend.py -k test_positive_end_to_end --external-logging
Test Result : ========== 1 passed, 5 deselected, 18 warnings in 1510.28s (0:25:10) ===========
shweta83 commented 1 week ago

@Gauravtalreja1 Can you please update the description of the PR and commit message to add package option in registration command since you added that in the test. Rest all looks good.

Gauravtalreja1 commented 1 week ago

@shweta83 Sure, I updated the description for future references, but, I would differ from adding it to the commit message since it will unnecessarily make it longer