Service name different on some OSes; set $lp_openldap_service based on $::osfamily.
Cleanup params.pp
Add LSB warning to params.pp
Change LSB warning to fail in params.pp
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