summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRob Crittenden <rcritten@redhat.com>2011-07-01 15:21:03 -0400
committerRob Crittenden <rcritten@redhat.com>2011-07-11 19:21:51 -0400
commit28e85ee47aa6e71262b9bf50758da079e2089b76 (patch)
tree7e2f4287d61c46ed7391f09e1c84fe810934c7a3
parent2415ba6d37371bc63131d02107fa4aee68e3f365 (diff)
downloadfreeipa-28e85ee47aa6e71262b9bf50758da079e2089b76.tar.gz
freeipa-28e85ee47aa6e71262b9bf50758da079e2089b76.tar.xz
freeipa-28e85ee47aa6e71262b9bf50758da079e2089b76.zip
Fix test failure in updater when adding values to a single-value attr
The ipaldap.py code was updated to consider the schema when making changes and does a REPLACE on single-value attributes. So when you do an add in an update it will effectively replace the value instead of ignoring it.
-rw-r--r--tests/test_install/4_update.update1
-rw-r--r--tests/test_install/test_updates.py2
2 files changed, 2 insertions, 1 deletions
diff --git a/tests/test_install/4_update.update b/tests/test_install/4_update.update
index d02cfcd3c..0aef7494f 100644
--- a/tests/test_install/4_update.update
+++ b/tests/test_install/4_update.update
@@ -1,3 +1,4 @@
+# Replace the existing SINGLE-VALUE gecos with a new value thru add
dn: uid=tuser, cn=test, cn=accounts, $SUFFIX
add:gecos: Test User New2
diff --git a/tests/test_install/test_updates.py b/tests/test_install/test_updates.py
index 58cfc11a4..eb376f191 100644
--- a/tests/test_install/test_updates.py
+++ b/tests/test_install/test_updates.py
@@ -126,7 +126,7 @@ class test_update(object):
Test the updater adding a new value to a single-valued attribute
"""
modified = self.updater.update([self.testdir + "4_update.update"])
- assert(modified == False)
+ assert(modified == True)
def test_5_update(self):
"""