Merge pull request #394 from ekohl/autorequire-db

Autorequire mongodb_database for mongodb_user

Author: alexjfisher

lib/puppet/type/mongodb_user.rb

@@ -88,6 +88,10 @@ def insync?(_is)
     'mongodb'
   end
 
+  autorequire(:mongodb_database) do
+    self[:database]
+  end
+
   validate do
     if self[:password_hash].nil? && self[:password].nil? && provider.password.nil? && provider.password_hash.nil?
       err("Either 'password_hash' or 'password' should be provided")

manifests/db.pp

@@ -39,7 +39,6 @@
     username      => $user,
     database      => $db_name,
     roles         => $roles,
-    require       => Mongodb_database[$db_name],
   }
 
 }

spec/defines/db_spec.rb

@@ -14,9 +14,10 @@
     end
 
     it 'contains mongodb_user with mongodb_database requirement' do
-      is_expected.to contain_mongodb_user('User testuser on db testdb').with('username' => 'testuser',
-                                                                             'database' => 'testdb',
-                                                                             'require'  => 'Mongodb_database[testdb]')
+      is_expected.to contain_mongodb_user('User testuser on db testdb'). \
+        with_username('testuser'). \
+        with_database('testdb'). \
+        that_requires('Mongodb_database[testdb]')
     end
 
     it 'contains mongodb_user with proper roles' do
@@ -53,9 +54,10 @@
     end
 
     it 'contains mongodb_user with mongodb_database requirement' do
-      is_expected.to contain_mongodb_user('User testuser on db testdb').with('username' => 'testuser',
-                                                                             'database' => 'testdb',
-                                                                             'require'  => 'Mongodb_database[testdb]')
+      is_expected.to contain_mongodb_user('User testuser on db testdb'). \
+        with_username('testuser'). \
+        with_database('testdb'). \
+        that_requires('Mongodb_database[testdb]')
     end
 
     it 'contains mongodb_user with proper roles' do