From d95043b63b5266e3e075b9869e01a5d47819c208 Mon Sep 17 00:00:00 2001
From: Luke Bennett <lbennett@gitlab.com>
Date: Fri, 17 May 2019 13:43:03 +0100
Subject: [PATCH] Resolve CE/EE diffs in new group member view

Part of single codebase changes.
---
 app/helpers/groups/group_members_helper.rb      |  7 +++++++
 .../group_members/_new_group_member.html.haml   |  2 +-
 .../helpers/groups/group_members_helper_spec.rb | 17 +++++++++++++++++
 3 files changed, 25 insertions(+), 1 deletion(-)
 create mode 100644 app/helpers/groups/group_members_helper.rb
 create mode 100644 spec/helpers/groups/group_members_helper_spec.rb

diff --git a/app/helpers/groups/group_members_helper.rb b/app/helpers/groups/group_members_helper.rb
new file mode 100644
index 00000000000..a5d2f76820f
--- /dev/null
+++ b/app/helpers/groups/group_members_helper.rb
@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Groups::GroupMembersHelper
+  def group_member_select_options
+    { multiple: true, class: 'input-clamp', scope: :all, email_user: true }
+  end
+end
diff --git a/app/views/groups/group_members/_new_group_member.html.haml b/app/views/groups/group_members/_new_group_member.html.haml
index c8cdc2cc3e4..8b511f6866f 100644
--- a/app/views/groups/group_members/_new_group_member.html.haml
+++ b/app/views/groups/group_members/_new_group_member.html.haml
@@ -1,7 +1,7 @@
 = form_for @group_member, url: group_group_members_path(@group), html: { class: 'users-project-form users-group-form' } do |f|
   .row
     .col-md-4.col-lg-6
-      = users_select_tag(:user_ids, multiple: true, class: 'input-clamp', scope: :all, email_user: true)
+      = users_select_tag(:user_ids, group_member_select_options)
       .form-text.text-muted.append-bottom-10
         Search for members by name, username, or email, or invite new ones using their email address.
 
diff --git a/spec/helpers/groups/group_members_helper_spec.rb b/spec/helpers/groups/group_members_helper_spec.rb
new file mode 100644
index 00000000000..898c330c498
--- /dev/null
+++ b/spec/helpers/groups/group_members_helper_spec.rb
@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe Groups::GroupMembersHelper do
+  describe '.group_member_select_options' do
+    let(:group) { create(:group) }
+
+    before do
+      helper.instance_variable_set(:@group, group)
+    end
+
+    it 'returns an options hash' do
+      expect(helper.group_member_select_options).to include(multiple: true, scope: :all, email_user: true)
+    end
+  end
+end