lslebodn commented on a pull request
"""
On (07/09/16 10:26), mzidek-rh wrote:
Here is a fix + CI test for
https://fedorahosted.org/sssd/ticket/3179.
Michal
You can view, comment on, or merge this pull request online at:
https://github.com/SSSD/sssd/pull/20
I have just few comments to the integration test.
IMHO, it's not enough to test just that export
and import passed. It is an incomplete test.
We need to be sure that we corretly imported all datas.
In other word, check all user attributes.
It would be good to have more descriptive name of tests.
The "test_regr_3179_override" does not say a lot.
I know there are other tests with such name
but it does not mean we need to continue in this bad habit.
LS
"""
See the full comment at
https://github.com/SSSD/sssd/pull/20#issuecomment-245875622