fix group count bug

Change-Id: Ia9ca6ba4217785445c474e0a5128c0b9cc72dc4d
Reviewed-on: https://gerrit.instructure.com/5369
Tested-by: Hudson <hudson@instructure.com>
Reviewed-by: Zach Wily <zach@instructure.com>

Author: jenseng

app/controllers/conversations_controller.rb

@@ -263,7 +263,7 @@ def can_add_notes_to?(course)
   def matching_contexts(search, exclude = [])
     avatar_url = avatar_url_for_group(true)
     course_user_counts = @current_user.enrollment_visibility[:user_counts]
-    group_user_counts = @contexts[:groups].inject({}){ |hash, group| hash[group.id] = group.users.size; hash }
+    group_user_counts = @current_user.groups.inject({}){ |hash, group| hash[group.id] = group.users.size; hash }
     @contexts.values.map(&:values).flatten.
       select{ |context| search.downcase.strip.split(/\s+/).all?{ |part| context[:name].downcase.include?(part) } }.
       select{ |context| context[:active] }.

spec/controllers/conversations_controller_spec.rb

@@ -243,13 +243,20 @@ def conversation(num_other_users = 1)
   describe "GET 'find_recipients'" do
     it "should assign variables" do
       course_with_student_logged_in(:active_all => true)
-      other = User.create(:name => 'testuser')
+      @course.update_attribute(:name, "this_is_a_test_course")
+
+      other = User.create(:name => 'this_is_a_test_user')
       enrollment = @course.enroll_student(other)
       enrollment.workflow_state = 'active'
       enrollment.save
 
-      get 'find_recipients', :search => other.name
+      group = @course.groups.create(:name => 'this_is_a_test_group')
+      group.users = [@user, other]
+
+      get 'find_recipients', :search => 'this_is_a_test_'
       response.should be_success
+      response.body.should include(@course.name)
+      response.body.should include(group.name)
       response.body.should include(other.name)
     end
   end