Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: vijay sawant <[email protected]>
  • Loading branch information
dosas and vijaysawant authored Apr 2, 2024
1 parent e931f8d commit 19311cc
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions tests/foreman/api/test_parameters.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,18 +16,21 @@

@pytest.fixture
def param_name():
"""Generate name string for common parameter and return to test cases"""
"""Generate name for common parameter"""
return gen_string('alpha')


@pytest.fixture
def param_value():
"""Generate value string for common parameter and return to test cases"""
"""Generate value for common parameter"""
return gen_string('alpha')


@pytest.fixture
def cp(param_name, param_value, module_target_sat):
"""Create common parameter object, yields it for test cases, delete object before teardown"""
"""Create common parameter"""
cp = module_target_sat.api.CommonParameter(name=param_name, value=param_value).create()

Expand All @@ -38,6 +41,7 @@ def cp(param_name, param_value, module_target_sat):

@pytest.fixture
def host(param_name, param_value, module_org, module_location, module_target_sat, cp, hostgroup):
"""Create host object, yeilds it for test cases, delete object before teardown"""
"""Create host"""
host = module_target_sat.api.Host(organization=module_org, location=module_location).create()

Expand All @@ -48,6 +52,7 @@ def host(param_name, param_value, module_org, module_location, module_target_sat

@pytest.fixture
def hostgroup(param_name, param_value, module_org, module_target_sat):
"""Create hostgroup object, yeilds it for test cases, delete object before teardown"""
"""Create hostgroup"""
hg = module_target_sat.api.HostGroup(
organization=[module_org],
Expand Down

0 comments on commit 19311cc

Please sign in to comment.