Merge branch 'master' of github.com:diaspora/diaspora_rails

app/controllers/people_controller.rb

@@ -2,7 +2,13 @@ class PeopleController < ApplicationController
   before_filter :authenticate_user!
   
   def index
+    unless params[:q]
       @people = Person.friends.paginate :page => params[:page], :order => 'created_at DESC'
+      render :index
+    else
+      @people = Person.search_for_friends(params[:q])
+      render :json => @people.to_json(:only => :_id)
+    end
   end
   
   def show

app/models/person.rb

@@ -32,7 +32,9 @@ class Person
 
   scope :friends, where(:_type => "Person", :active => true)
   
-
+  def self.search_for_friends(query)
+    Person.all('$where' => "function() { return this.profile.first_name.match(/^#{query}/i) || this.profile.last_name.match(/^#{query}/i); }")
+  end
  
   def real_name
     "#{profile.first_name.to_s} #{profile.last_name.to_s}"

spec/controllers/people_controller_spec.rb

@@ -0,0 +1,61 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+ 
+describe PeopleController do
+  render_views
+  before do
+    @user = Factory.create(:user, :profile => Profile.new( :first_name => "bob", :last_name => "smith"))
+    request.env['warden'] = mock_model(Warden, :authenticate? => @user, :authenticate! => @user, :authenticate => @user)
+
+    sign_in :user, @user   
+  end
+
+  it "index should yield search results for substring of person name" do
+    friend_one = Factory.create(:person)
+    friend_two = Factory.create(:person)
+    friend_three = Factory.create(:person)
+    friend_four = Factory.create(:person)
+
+    friend_one.profile.first_name = "Robert"
+    friend_one.profile.last_name = "Grimm"
+    friend_one.profile.save
+
+    friend_two.profile.first_name = "Eugene"
+    friend_two.profile.last_name = "Weinstein"
+    friend_two.save
+
+    friend_three.profile.first_name = "Yevgeniy"
+    friend_three.profile.last_name = "Dodis"
+    friend_three.save
+
+    friend_four.profile.first_name = "Casey"
+    friend_four.profile.last_name = "Grippi"
+    friend_four.save
+
+
+    puts Person.friends.count
+
+    get :index, :q => "Eu"
+
+
+    assigns[:people].include?(friend_two).should == true
+    assigns[:people].include?(friend_one).should == false
+    assigns[:people].include?(friend_three).should == false
+    assigns[:people].include?(friend_four).should == false
+
+    get :index, :q => "Wei"
+    assigns[:people].include?(friend_two).should == true
+    assigns[:people].include?(friend_one).should == false
+    assigns[:people].include?(friend_three).should == false
+    assigns[:people].include?(friend_four).should == false
+
+    get :index, :q => "Gri"
+    assigns[:people].include?(friend_one).should == true
+    assigns[:people].include?(friend_four).should == true
+    assigns[:people].include?(friend_two).should == false
+    assigns[:people].include?(friend_three).should == false
+
+    get :index
+    assigns[:people].should == Person.friends.all
+  end
+
+end