Fix #2663 - refactor to use shell_out in rpm provider

Fixes #2663 by using shell_out instead of popen4
We don't normally use the specific `subject` method - this should just be `let(:provider)`.

I think `subject` is useful if you want to use the `subject` method in the spec, but we also normally don't do that.