Refactor relation_group_members method

This change refactors the
`relation_group_members ` method

app/finders/group_members_finder.rb

@@ -66,9 +66,13 @@ class GroupMembersFinder < UnionFinder
     group.members
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def relation_group_members(relation)
-    GroupMember.non_request.non_minimal_access
+    all_group_members(relation).non_minimal_access
+  end
+
+  # rubocop: disable CodeReuse/ActiveRecord
+  def all_group_members(relation)
+    GroupMember.non_request
       .where(source_id: relation.select(:id))
       .where.not(user_id: group.users.select(:id))
   end

ee/app/finders/ee/group_members_finder.rb

@@ -18,19 +18,13 @@ module EE::GroupMembersFinder
   def group_members_list
     return group.all_group_members if group.minimal_access_role_allowed?
 
-    group.members
+    super
   end
 
   override :relation_group_members
-  # rubocop: disable CodeReuse/ActiveRecord
   def relation_group_members(relation)
-    members = ::GroupMember.non_request
-      .where(source_id: relation.select(:id))
-      .where.not(user_id: group.users.select(:id))
-
-    return members if group.minimal_access_role_allowed?
+    return all_group_members(relation) if group.minimal_access_role_allowed?
 
-    members.non_minimal_access
+    super
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 end

ee/spec/finders/ee/group_members_finder_spec.rb

@@ -43,7 +43,7 @@ RSpec.describe GroupMembersFinder do
         stub_licensed_features(minimal_access_role: true)
       end
 
-      it 'returns only members with minimal access' do
+      it 'also returns members with minimal access' do
         result = finder.execute(include_relations: [:direct, :descendants])
 
         expect(result.to_a).to match_array([group_owner_membership, group_member_membership, dedicated_member_account_membership, group_minimal_access_membership])