New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 2820 - Fix CI tests #4365
Conversation
8bd57a3
to
5e4691c
Compare
from lib389._constants import TASK_WAIT | ||
from lib389._constants import DEFAULT_SUFFIX | ||
from lib389._mapped_object import DSLdapObjects | ||
from lib389.idm.user import UserAccounts | ||
|
||
pytestmark = pytest.mark.tier1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
originally tickets/ticket47973_test.py was under tier2.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The rest looks good to me. But maybe @Firstyear will something to say...
Bug Description: tickets/ticket47973_test.py test had failures in CI nightly runs Fix Description: Fixed the failure and also changed the code to use DSLdapObject moved the code into the schema test suite. Relates: 389ds#2820 Reviewed by: Simon(droideck)
5e4691c
to
12943cd
Compare
Please, avoid 'git push --force' option as it's hard to follow what changes were made.
|
@droideck opps, sure i will keep that in mind. Normal push was giving me some error i guess but sure I will completely avoid it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@droideck opps, sure i will keep that in mind. Normal push was giving me some error i guess but sure I will completely avoid it.
No problem:) Ping me on IRC if you have some annoying issue and we'll try to figure it out.
|
||
# While we wait for the task to complete search for our user | ||
search_count = 0 | ||
while search_count < 10: | ||
# Now check the user is still being returned | ||
# Check if task is complete | ||
assert user.exists() | ||
task_entry = topo.standalone.tasks.entry | ||
if topo.standalone.tasks.checkTask(task_entry)[0]: | ||
if task.get_exit_code() == 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
didn't use task.wait() here as the test needs it to be a race condition. Also made changes in other tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Bug Description: tickets/ticket47973_test.py test had failures in CI nightly runs
Fix Description: Fixed the failure and also changed the code to use DSLdapObject
moved the code into the schema test suite.
Relates: #2820
Reviewed by: ??