mirror of
https://salsa.debian.org/freeipa-team/freeipa.git
synced 2024-12-24 16:10:02 -06:00
Xfailed a test that fails due to 6250
Also created a decorator that removes the segment that the next test does not expect. https://fedorahosted.org/freeipa/ticket/6250 Reviewed-By: Martin Basti <mbasti@redhat.com>
This commit is contained in:
parent
49fbbb0641
commit
3e4740f788
@ -27,6 +27,26 @@ def find_segment(master, replica):
|
||||
return '-to-'.join(segment)
|
||||
|
||||
|
||||
def remove_segment(master, host1, host2):
|
||||
"""
|
||||
This removes a segment between host1 and host2 on master. The function is
|
||||
needed because test_add_remove_segment expects only one segment, but due to
|
||||
track tickete N 6250, the test_topology_updated_on_replica_install_remove
|
||||
leaves 2 topology segments
|
||||
"""
|
||||
def wrapper(func):
|
||||
def wrapped(*args, **kwargs):
|
||||
try:
|
||||
func(*args, **kwargs)
|
||||
finally:
|
||||
segment = find_segment(host1, host2)
|
||||
master.run_command(['ipa', 'topologysegment-del',
|
||||
DOMAIN_SUFFIX_NAME, segment],
|
||||
raiseonerr=False)
|
||||
return wrapped
|
||||
return wrapper
|
||||
|
||||
|
||||
@pytest.mark.skipif(config.domain_level == 0, reason=reasoning)
|
||||
class TestTopologyOptions(IntegrationTest):
|
||||
num_replicas = 2
|
||||
@ -64,6 +84,10 @@ class TestTopologyOptions(IntegrationTest):
|
||||
)
|
||||
return result
|
||||
|
||||
@pytest.mark.xfail(reason="Trac 6250", strict=True)
|
||||
@remove_segment(config.domains[0].master,
|
||||
config.domains[0].master,
|
||||
config.domains[0].replicas[1])
|
||||
def test_topology_updated_on_replica_install_remove(self):
|
||||
"""
|
||||
Install and remove a replica and make sure topology information is
|
||||
|
Loading…
Reference in New Issue
Block a user