(#12412) Refactor puppet_resource('group') tests
Previously, the Linux-specific logic for validating
puppet_resource('group') was contained within each 'group' test case. In
order to support executing resource acceptance tests on non-Linux
agents, I could have added if-then-else statements to each test case,
though that would lead to logic-sprawl.
This commit refactors the existing set of 'group' acceptance tests to
use native commands to validate that the tests work on any supported
agent platform. Specifically, that puppet_resource('group') can create,
delete, query, and query_all, and that it doesn't try to
delete a non-existent group, or add a group that already exists.
The goal of these tests is to validate that puppet_resource "works"
with group resources, and can therefore be trusted to validate the
results of subsequence acceptance tests.
The commit also preserves tests that exercise unique behaviors,
e.g. should_modify_gid.rb.