-
Notifications
You must be signed in to change notification settings - Fork 246
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
Test: sssclt test case #7457
base: master
Are you sure you want to change the base?
Test: sssclt test case #7457
Conversation
Check for error if memcache_size_sid is set in nss section. Signed-off-by: Deepak Das <ddas@redhat.com>
9ce763a
to
1d02cac
Compare
@alexey-tikhonov, I'm still determining. I told @madhuriupadhye, we can approve the test but tag it as deferred. I'm going to perform the housekeeping with a higher priority so we can get new cases in. |
@danlavu, any decision? |
All of the sssctl tests need to be reworked, but it's not a priority. Let's go ahead and add the test and it'll get reworked with everything else. |
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.
Other than the line break the test looks good, thank you.
client.sssd.nss["memcache_size_sid"] = "10" | ||
client.sssd.start(check_config=False) | ||
result = client.sssctl.config_check() | ||
assert result.rc == 0, "Config-check failed" |
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.
For readability, add a line break before the assertions.
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
@DDDas7 rebase, add a blank line, look at covscan errors, this PR is close to being ready to be merged. Let's not forget about it. |
Check for error if memcache_size_sid is set in nss section.