Skip to content

Commit

Permalink
Fix LDAP test to catch the correct output.
Browse files Browse the repository at this point in the history
After 1.10 rebase this test started failing because we now use grouped
APIs by default.

Signed-off-by: Simo Sorce <[email protected]>
  • Loading branch information
simo5 committed Apr 17, 2018
1 parent 87440f4 commit ceea517
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/extended/ldap_groups.sh
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ for (( i=0; i<${#schema[@]}; i++ )); do
os::cmd::expect_success_and_text 'oc get group -o jsonpath={.items[*].metadata.name}' 'firstgroup secondgroup thirdgroup'
os::cmd::expect_success_and_text 'oc adm groups prune --whitelist=ldapgroupuids.txt --sync-config=sync-config-user-defined.yaml --confirm | wc -l' '0'
os::cmd::expect_success_and_text 'oc get group -o jsonpath={.items[*].metadata.name}' 'firstgroup secondgroup thirdgroup'
os::cmd::expect_success_and_text 'oc patch group secondgroup -p "{\"metadata\":{\"annotations\":{\"openshift.io/ldap.uid\":\"cn=garbage\"}}}"' 'group "secondgroup" patched'
os::cmd::expect_success_and_text 'oc patch group secondgroup -p "{\"metadata\":{\"annotations\":{\"openshift.io/ldap.uid\":\"cn=garbage\"}}}"' 'group.user.openshift.io "secondgroup" patched'
os::cmd::expect_success_and_text 'oc adm groups prune --whitelist=ldapgroupuids.txt --sync-config=sync-config-user-defined.yaml --confirm' 'group/secondgroup'
os::cmd::expect_success_and_text 'oc get group -o jsonpath={.items[*].metadata.name}' 'firstgroup thirdgroup'
os::cmd::expect_success_and_text 'oc delete groups --all' 'deleted'
Expand Down

0 comments on commit ceea517

Please sign in to comment.