The last major code commit for ACI functionality uncovered a couple of inco...

mkeyes

02.41.2007 b27721e452ab73064fb1a652797cc36a0f7e22e0

The last major code commit for ACI functionality uncovered a couple of incorrect return codes that were tested. These are fixed here.

When using targattrfilters, all the values of a multi-valued attribute must match the filters in the targattrfilters statement, not just one of them. 

 opends/tests/functional-tests/testcases/aci/multiple_aci_tests.xml

@@ -3702,7 +3702,8 @@
                #@TestPurpose               Test with separate targetattrs
                #@TestPreamble              Admin adds an aci with the target equal to the dn of one existing branch.
                #@TestStep                  Client replaces l in an entry with the targeted branch dn.
                #@TestStep                  Client replaces mail in an entry with the targeted branch dn.
                #@TestStep                  Client replaces title in an entry with the targeted branch dn.
                #@TestStep                  Client replaces another title in an entry with the targeted branch dn.
                #@TestStep                  Client replaces l in an entry that is not with the targeted branch dn.
                #@TestStep                  Remove aci.
                #@TestStep                  Client replaces l from an entry with the previously targeted branch dn.
@@ -3765,7 +3766,7 @@
                    'entryToBeModified'   : '%s/aci/multiple_aci_tests/replace_title36.ldif' % STAGED_DATA_DIR }
                </call>
    
                <if expr="RC != 0">
                <if expr="RC != 50">
                    <tcstatus result="'fail'"/>
                </if>
    
@@ -3781,7 +3782,7 @@
                    'entryToBeModified'   : '%s/aci/multiple_aci_tests/replace_title36.1.ldif' % STAGED_DATA_DIR }
                </call>
    
                <if expr="RC != 0">
                <if expr="RC != 50">
                    <tcstatus result="'fail'"/>
                </if>
    
@@ -3849,7 +3850,8 @@
                #@TestPurpose               Test with separate targetattrs
                #@TestPreamble              Admin adds an aci with the target equal to the dn of one existing branch.
                #@TestStep                  Client replaces l in an entry with the targeted branch dn.
                #@TestStep                  Client replaces mail in an entry with the targeted branch dn.
                #@TestStep                  Client replaces title in an entry with the targeted branch dn.
                #@TestStep                  Client replaces another title in an entry with the targeted branch dn.
                #@TestStep                  Client replaces l in an entry that is not with the targeted branch dn.
                #@TestStep                  Remove aci.
                #@TestStep                  Client replaces l from an entry with the previously targeted branch dn.
@@ -3912,7 +3914,7 @@
                    'entryToBeModified'   : '%s/aci/multiple_aci_tests/replace_title37.ldif' % STAGED_DATA_DIR }
                </call>
    
                <if expr="RC != 0">
                <if expr="RC != 50">
                    <tcstatus result="'fail'"/>
                </if>
    
@@ -3928,7 +3930,7 @@
                    'entryToBeModified'   : '%s/aci/multiple_aci_tests/replace_title37.1.ldif' % STAGED_DATA_DIR }
                </call>
    
                <if expr="RC != 0">
                <if expr="RC != 50">
                    <tcstatus result="'fail'"/>
                </if>