search

openshift / openldap

Apache License 2.0
32 stars 59 forks source link

added test schema #11

Closed stevekuznetsov closed 9 years ago

stevekuznetsov commented 9 years ago

@deads2k @liggitt This adds a testPerson objectClass that we can stuff arbitrary attributes into as we see fit.

The only issue with this implementation is that we need to get an OID. We should be able to subclass from an OID given to RedHat previously, however. If that's not possible, we could stomp on someone else's namespace, but that's bad form. Unfortunately as we need to be root to add schemas, this needs to happen at the image level and we cannot isolate this to happening in our test build.

deads2k commented 9 years ago

For testing, I'm fine with stomping. Most specs reserve some names for internal use though.

deads2k commented 9 years ago

Works for me. Remove the new test that is relying on functionality we don't seem to have in the image. This blocks https://github.com/openshift/origin/pull/4972

stevekuznetsov commented 9 years ago

@deads2k Removed test, rebased.

deads2k commented 9 years ago

lgtm, works locally.