Merge pull request #8504 from GabrielNagy/PUP-10857-2

(PUP-10857) Correctly retrieve the groups of an user

Author: luchihoratiu

acceptance/tests/resource/user/should_manage_groups.rb

@@ -8,7 +8,7 @@
   confine :except, :platform => /eos-/ # See ARISTA-37
   confine :except, :platform => /cisco_/ # See PUP-5828
 
-  tag 'audit:medium',
+  tag 'audit:high',
       'audit:acceptance' # Could be done as integration tests, but would
                          # require changing the system running the test
                          # in ways that might require special permissions

lib/puppet/provider/user/useradd.rb

@@ -71,7 +71,7 @@ def comment
 
   def groups
      return localgroups if @resource.forcelocal?
-     get(:groups)
+     super
   end
 
   def finduser(key, value)

spec/unit/provider/user/useradd_spec.rb

@@ -392,7 +392,7 @@
 
     it "should fall back to nameservice groups when forcelocal is false" do
       resource[:forcelocal] = false
-      allow(provider).to receive(:get).with(:groups).and_return('remote groups')
+      allow(Puppet::Util::POSIX).to receive(:groups_of).with('myuser').and_return(['remote groups'])
       expect(provider).not_to receive(:localgroups)
       expect(provider.groups).to eq('remote groups')
     end