test_entry should not be rejected if username is missing, but it should not set the uid

to create situation where the entry exists but does not have a uid.

Author: dmlond

spec/support/ldap_shared_examples.rb

@@ -1,11 +1,9 @@
 shared_context 'mocked ldap' do |returns:|
   let(:entry_users) { send(returns) }
   let(:expected_entries) {
-    entry_users.reject{ |u|
-      !u.has_key? :username
-    }.map { |test_user|
+    entry_users.map { |test_user|
       expected_entry = Net::LDAP::Entry.new
-      expected_entry[:uid] = test_user[:username]
+      expected_entry[:uid] = test_user[:username] if test_user[:username]
       expected_entry[:givenName] = test_user[:first_name]
       expected_entry[:sn] = test_user[:last_name]
       expected_entry[:mail] = test_user[:email] if test_user[:email]