Merge pull request #512 from coopdevs/feature/add-phones-search-managers

Feature/add phones search feature (only for managers)

Author: sauloperez

app/views/users/manage.html.erb

@@ -11,7 +11,7 @@
     <div class="col-md-12">
       <%= search_form_for(@search, class: "navbar-form navbar-left", url: manage_users_path) do |f| %>
         <div class="form-group">
-          <%= f.search_field :user_username_or_user_email_or_member_uid_search_contains, class: "form-control" %>
+          <%= f.search_field :user_username_or_user_email_or_user_phone_or_user_alt_phone_or_member_uid_search_contains, class: "form-control" %>
         </div>
         <button class="btn btn-default" type="submit">
           <%= t 'global.search' %>

spec/controllers/users_controller_spec.rb

@@ -146,6 +146,17 @@
         end
       end
     end
+
+    context 'when searching' do
+      it 'allows to search by phone' do
+        user = Fabricate(:user, phone: 123456789)
+        member = Fabricate(:member, user: user, organization: test_organization)
+
+        get :manage, q: { user_username_or_user_email_or_user_phone_or_user_alt_phone_or_member_uid_search_contains: 123456789 }
+
+        expect(assigns(:members)).to include(member)
+      end
+    end
   end
 
   describe "GET #show" do