<feed xmlns='http://www.w3.org/2005/Atom'>
<title>sssd.git/src/tests/sysdb-tests.c, branch sssd-1_12_2</title>
<subtitle>sssd with jhrozek's patches</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/'/>
<entry>
<title>sysdb: add sysdb_attrs_add_val_safe() and sysdb_attrs_add_string_safe()</title>
<updated>2014-10-16T15:56:07+00:00</updated>
<author>
<name>Sumit Bose</name>
<email>sbose@redhat.com</email>
</author>
<published>2014-10-08T14:12:52+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=8a2a503fa5c01ea037d28b7c902b8821a11084bd'/>
<id>8a2a503fa5c01ea037d28b7c902b8821a11084bd</id>
<content type='text'>
sysdb_attrs_add_val_safe() works like sysdb_attrs_add_val() but checks
if the attribute value to add already exists. In this case the value
list is not changed. This is useful if values are added from different
sources at different times to avoid LDB_ERR_ATTRIBUTE_OR_VALUE_EXISTS
errors from ldb_modify() later on.

sysdb_attrs_add_string_safe() does the same for string arguments

Reviewed-by: Pavel Březina &lt;pbrezina@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
sysdb_attrs_add_val_safe() works like sysdb_attrs_add_val() but checks
if the attribute value to add already exists. In this case the value
list is not changed. This is useful if values are added from different
sources at different times to avoid LDB_ERR_ATTRIBUTE_OR_VALUE_EXISTS
errors from ldb_modify() later on.

sysdb_attrs_add_string_safe() does the same for string arguments

Reviewed-by: Pavel Březina &lt;pbrezina@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: add test for sysdb_get_user_attr with subdomain user</title>
<updated>2014-10-09T15:14:41+00:00</updated>
<author>
<name>Pavel Březina</name>
<email>pbrezina@redhat.com</email>
</author>
<published>2014-10-07T14:57:40+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=25072dd5274f07255cdc930f6257c4cbf74692de'/>
<id>25072dd5274f07255cdc930f6257c4cbf74692de</id>
<content type='text'>
Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>SYSDB: move sysdb_get_real_name() from sysdb.c to sysdb_search.c</title>
<updated>2014-10-06T12:19:37+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2014-09-21T14:55:21+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=d7d3ee1b8ab7a05129c83da8a185351d7c751c1c'/>
<id>d7d3ee1b8ab7a05129c83da8a185351d7c751c1c</id>
<content type='text'>
The sysdb.c should be reserved for utility and setup functions. Search
functions belong to sysdb_search.c Keeping functions in specialized
modules helps to maintain nice dependencies and in overall makes unit
testing easier.

Moreover, the function was not unit tested, which needed fixing.

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The sysdb.c should be reserved for utility and setup functions. Search
functions belong to sysdb_search.c Keeping functions in specialized
modules helps to maintain nice dependencies and in overall makes unit
testing easier.

Moreover, the function was not unit tested, which needed fixing.

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: Add a test for storing custom attrs with automatic ID</title>
<updated>2014-09-18T16:00:53+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2014-09-17T11:53:48+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=89e18b38259e9c5f1aa1a35ef30f13aee5ea6bd2'/>
<id>89e18b38259e9c5f1aa1a35ef30f13aee5ea6bd2</id>
<content type='text'>
Reviewed-by: Daniel Gollub &lt;dgollub@brocade.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed-by: Daniel Gollub &lt;dgollub@brocade.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>TESTS: Add a case-insensitive group search sysdb test</title>
<updated>2014-09-17T13:51:45+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2014-09-17T10:12:29+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=d580dde031bca7588555e8f2857a0fcc2977f1e0'/>
<id>d580dde031bca7588555e8f2857a0fcc2977f1e0</id>
<content type='text'>
A recent fix enabled searching for groups by name in a case-insensitive
domain. This patch adds a unit test to check that behaviour.

Reviewed-by: Sumit Bose &lt;sbose@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
A recent fix enabled searching for groups by name in a case-insensitive
domain. This patch adds a unit test to check that behaviour.

Reviewed-by: Sumit Bose &lt;sbose@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>SYSDB: SSS_LDB_SEARCH - macro around ldb_search</title>
<updated>2014-09-05T09:34:50+00:00</updated>
<author>
<name>Pavel Reichl</name>
<email>preichl@redhat.com</email>
</author>
<published>2014-07-11T14:21:59+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=61602026ed8c91efd166000562899670449f1b50'/>
<id>61602026ed8c91efd166000562899670449f1b50</id>
<content type='text'>
This patch amends previous patch 5153e8b9793dea1e212ca08af0f77ea1d023cbb7.

Macro SSS_LDB_SEARCH is used instead of using fuction sss_ldb_search as
a wrapper around ldb_search which could lead to premature expansion of
variadic parameters.

Part of solution for:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch amends previous patch 5153e8b9793dea1e212ca08af0f77ea1d023cbb7.

Macro SSS_LDB_SEARCH is used instead of using fuction sss_ldb_search as
a wrapper around ldb_search which could lead to premature expansion of
variadic parameters.

Part of solution for:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>TESTS: Add unit tests for the GPO interface</title>
<updated>2014-09-01T15:32:03+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2014-07-21T18:31:15+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=8cfd1e0d696a573a92ef011a64317b9054f5c45f'/>
<id>8cfd1e0d696a573a92ef011a64317b9054f5c45f</id>
<content type='text'>
Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>TOOLS: Always debug to stderr</title>
<updated>2014-07-09T15:36:17+00:00</updated>
<author>
<name>Jakub Hrozek</name>
<email>jhrozek@redhat.com</email>
</author>
<published>2014-07-09T09:24:21+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=6b57784f0f175275fd900eca21c77415e3a5ea52'/>
<id>6b57784f0f175275fd900eca21c77415e3a5ea52</id>
<content type='text'>
https://fedorahosted.org/sssd/ticket/2348

Programs that are supposed to only be executed on the foreground should
log to stderr automatically.

Reviewed-by: Michal Židek &lt;mzidek@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
https://fedorahosted.org/sssd/ticket/2348

Programs that are supposed to only be executed on the foreground should
log to stderr automatically.

Reviewed-by: Michal Židek &lt;mzidek@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>TESTS: sysdb_getnetgr - return ENOENT</title>
<updated>2014-06-25T14:04:17+00:00</updated>
<author>
<name>Pavel Reichl</name>
<email>preichl@redhat.com</email>
</author>
<published>2014-06-19T15:18:12+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=50c009a3d51521880ddf568e7173d1ed9d5c2685'/>
<id>50c009a3d51521880ddf568e7173d1ed9d5c2685</id>
<content type='text'>
Resolves:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
Reviewed-by: Jakub Hrozek &lt;jhrozek@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Resolves:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
Reviewed-by: Jakub Hrozek &lt;jhrozek@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>TESTS: add tests for sss_ldb_search</title>
<updated>2014-06-25T14:04:11+00:00</updated>
<author>
<name>Pavel Reichl</name>
<email>preichl@redhat.com</email>
</author>
<published>2014-06-19T14:38:59+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jhrozek/public_git/sssd.git/commit/?id=940dd08102ac8527ef9e367dc5d6fb88cd53a8a2'/>
<id>940dd08102ac8527ef9e367dc5d6fb88cd53a8a2</id>
<content type='text'>
Resolves:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
Reviewed-by: Jakub Hrozek &lt;jhrozek@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Resolves:
https://fedorahosted.org/sssd/ticket/1991

Reviewed-by: Lukáš Slebodník &lt;lslebodn@redhat.com&gt;
Reviewed-by: Jakub Hrozek &lt;jhrozek@redhat.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
