set up basic puppetlabs spec
Lint fixes on tests
Add basic top level spec
Add client and server tests to top spec
Fix missing ensure_real in template evaluation
Convert operatingsystem to osfamily
Add default for ssl param to client class
Fix service dependencies in client class

At least in PE 2.5.1, the '~>' method was failing with a mysterious
error: undefined method `<<' for {}:Hash on node snarf.ucsd.edu

Switched to explicit subscribe metaparam
set default param for ssl to false in server
Remove spec check for ensure=present on server
Set ssl to false in manifests/server/config.pp
Add explicit include of ldap::params

numerous things reference variables in it, but it wasn't being
explicitly included. This causes unpredictable behavior due to include
order
Work around parser bug with chained resources

Mulitple chained resource declarations also cause the Hash bug to rear
it's head in PE 2.5.1

Switched from '->' to explicit 'require' and 'before' metaparams
Move tests into spec
Remove unused ensure param
Include ldap::params in defined type

Fix up potentially missing variables
Fail if missing parameters

The templates will show a bunch of "undefs" if these parameters are
undefined
Add simple spec for ldap::defines::domain
Remove bad default values for acl

The template will not evaluate without these parameters set to undef.

Also did a line length cleanup for lint purposes
Include ldap::params in acl define

Needed to make sure that we have variables defined