mongodb_user ignores database parameter when removing a user

[SeanHood]

Pull Request (PR) description

Currently setting ensure => absent for a user doesn't work for users created in other databases than admin

This Pull Request (PR) fixes the following issues

This PR fixes this by passing the database value to mongo_eval

[SeanHood]

I have a feeling, this patch needs to be applied to get tests running. Should this be in this PR or a separate one?

diff --git a/.fixtures.yml b/.fixtures.yml
index 9937c6d..97dc093 100644
--- a/.fixtures.yml
+++ b/.fixtures.yml
@@ -1,7 +1,7 @@
 fixtures:
   repositories:
-    "stdlib": "git://github.com/puppetlabs/puppetlabs-stdlib.git"
-    "apt": "git://github.com/puppetlabs/puppetlabs-apt.git"
+    "stdlib": "https://github.com/puppetlabs/puppetlabs-stdlib.git"
+    "apt": "https://github.com/puppetlabs/puppetlabs-apt.git"
     "systemd": "https://github.com/voxpupuli/puppet-systemd.git"
     "zypprepo": "https://github.com/voxpupuli/puppet-zypprepo.git"
     yumrepo_core:

[smortex]

I just merged #643 which include it. Let's re-trigger a build

[smortex]

Hum, one day I will understand what GitHub does with the repo before running CI. I though it would merge the branch and the main branch but it seems not to…

Can you please rebase your changes on top of the main branch?

From your working directory:

git fetch origin       # Download the latest code we have here
git rebase origin/master # Move your commits on top of the main branch
git push -f            # Send the changes (-f is required because we re-wrote history)

Thanks!

[SeanHood]

Different failures, I'll take a look at those now.

[smortex]

LGTM. Not a user of this module so I will let another Vox member approve and merge this. Thanks!