Review requests

Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>

changelogs/fragments/alpine_group_force.yml

@@ -1,3 +1,3 @@
 ---
 bugfixes:
-    - user - raise an error if force=true is used while deleting the group on Alpine and BusyBox (https://github.com/ansible/ansible/issues/85565).
+    - user - raise an error if force=true is used while deleting the group on BusyBox based distros (https://github.com/ansible/ansible/issues/85565).

lib/ansible/modules/group.py

@@ -38,7 +38,7 @@ options:
         description:
             - Whether to delete a group even if it is the primary group of a user.
             - Only applicable on platforms which implement a C(--force) flag on the group deletion command.
-            - Not applicable on BusyBox and Alpine Linux.
+            - Not applicable on macOS, *BSD and BusyBox based distros.
         type: bool
         default: false
         version_added: "2.15"
@@ -401,6 +401,8 @@ class FreeBsdGroup(Group):
     GROUPFILE = '/etc/group'
 
     def group_del(self):
+        if self.module.params['force']:
+            self.module.fail_json(msg='The force option is not supported for group deletion on this platform.')
         cmd = [self.module.get_bin_path('pw', True), 'groupdel', self.name]
         return self.execute_command(cmd)
 
@@ -477,6 +479,8 @@ class DarwinGroup(Group):
         return (rc, out, err)
 
     def group_del(self):
+        if self.module.params['force']:
+            self.module.fail_json(msg='The force option is not supported for group deletion on this platform.')
         cmd = [self.module.get_bin_path('dseditgroup', True)]
         cmd += ['-o', 'delete']
         cmd += ['-L', self.name]
@@ -531,6 +535,8 @@ class OpenBsdGroup(Group):
     GROUPFILE = '/etc/group'
 
     def group_del(self):
+        if self.module.params['force']:
+            self.module.fail_json(msg='The force option is not supported for group deletion on this platform.')
         cmd = [self.module.get_bin_path('groupdel', True), self.name]
         return self.execute_command(cmd)
 
@@ -583,6 +589,8 @@ class NetBsdGroup(Group):
     GROUPFILE = '/etc/group'
 
     def group_del(self):
+        if self.module.params['force']:
+            self.module.fail_json(msg='The force option is not supported for group deletion on this platform.')
         cmd = [self.module.get_bin_path('groupdel', True), self.name]
         return self.execute_command(cmd)
 
@@ -653,7 +661,7 @@ class BusyBoxGroup(Group):
 
     def group_del(self):
         if self.module.params['force']:
-            self.module.fail_json(msg='force is not a valid option for this platform.')
+            self.module.fail_json(msg='The force option is not supported for group deletion on this platform.')
 
         cmd = [self.module.get_bin_path('delgroup', True), self.name]
         return self.execute_command(cmd)

test/integration/targets/group/tasks/tests.yml

@@ -407,7 +407,7 @@
           assert:
             that:
               - force_delete is failed
-              - "'force is not a valid option' in force_delete.msg"
+              - "'The force option is not supported' in force_delete.msg"
 
       always:
         - name: Cleanup group
@@ -415,7 +415,7 @@
             name: groupdeltest
             state: absent
 
-      when: ansible_distribution in ["Alpine"]
+      when: ansible_distribution in ["MacOSX", "Alpine", "FreeBSD"]
 
     # create system group