IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

Bugfixes in managed topology tests

Browse Source 
Fixed a false negative related to replication taking some time: added
wait_for_replication call before checking for new object in replicas.

Reviewed-By: Martin Babinsky <mbabinsk@redhat.com>
This commit is contained in:
Oleg Fayans 2016-04-06 11:20:38 +02:00 committed by Martin Basti
parent 1974f20aec
commit 1c79c1ea2d
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ipatests/test_integration/test_topology.py
View File

@ -3,7 +3,6 @@
# #
import re import re
import time
import pytest import pytest
@ -122,7 +121,10 @@ class TestTopologyOptions(IntegrationTest):
self.replicas[1].hostname) self.replicas[1].hostname)
returncode, error = tasks.destroy_segment(self.master, deleteme) returncode, error = tasks.destroy_segment(self.master, deleteme)
assert returncode == 0, error assert returncode == 0, error
# make sure replica1 does not have segment that was deleted on master # Wait till replication ends and make sure replica1 does not have
# segment that was deleted on master
replica1_ldap = self.replicas[0].ldap_connect()
tasks.wait_for_replication(replica1_ldap)
result3 = self.replicas[0].run_command(['ipa', 'topologysegment-find', result3 = self.replicas[0].run_command(['ipa', 'topologysegment-find',
DOMAIN_SUFFIX_NAME]).stdout_text DOMAIN_SUFFIX_NAME]).stdout_text
assert(deleteme not in result3), "%s: segment still exists" % deleteme assert(deleteme not in result3), "%s: segment still exists" % deleteme
@ -131,7 +133,8 @@ class TestTopologyOptions(IntegrationTest):
self.master.run_command(['ipa', 'user-add', 'someuser', self.master.run_command(['ipa', 'user-add', 'someuser',
'--first', 'test', '--first', 'test',
'--last', 'user']) '--last', 'user'])
time.sleep(60) # replication requires some time dest_ldap = self.replicas[1].ldap_connect()
tasks.wait_for_replication(dest_ldap)
result4 = self.replicas[1].run_command(['ipa', 'user-find']) result4 = self.replicas[1].run_command(['ipa', 'user-find'])
assert('someuser' in result4.stdout_text), 'User not found: someuser' assert('someuser' in result4.stdout_text), 'User not found: someuser'
# We end up having a line topology: master <-> replica1 <-> replica2 # We end up having a line topology: master <-> replica1 <-> replica2