Merge pull request kennethkalmer#19 from maetl/domain-find-by-name

Filter domains by name

app/controllers/domains_controller.rb

@@ -9,7 +9,11 @@ class DomainsController < InheritedResources::Base
   protected
 
   def collection
-    @domains = Domain.user( current_user ).paginate :page => params[:page]
+    if params[:name]
+      @domains = Domain.user( current_user ).by_name(params[:name]).paginate :page => params[:page]
+    else
+     @domains = Domain.user( current_user ).paginate :page => params[:page]
+    end
   end
 
   def resource

app/models/domain.rb

@@ -63,6 +63,7 @@ class Domain < ActiveRecord::Base
 
   # Scopes
   scope :user, lambda { |user| user.admin? ? nil : where(:user_id => user.id) }
+  scope :by_name, lambda { |name| where(:name => name) }
   default_scope order('name')
 
   class << self

spec/models/domain_spec.rb

@@ -110,8 +110,17 @@
   let(:aaron) { FactoryGirl.create(:aaron) }
   let(:quentin_domain) { FactoryGirl.create(:domain, :user => quentin) }
   let(:aaron_domain) { FactoryGirl.create(:domain, :name => 'example.org', :user => aaron) }
+  let(:aaron2_domain) { FactoryGirl.create(:domain, :name => 'example2.org', :user => aaron) }
   let(:admin) { FactoryGirl.create(:admin) }
 
+  it "should filter by name when domain exists" do
+    aaron_domain
+    aaron2_domain
+
+    Domain.by_name('example.org').should include(aaron_domain)
+    Domain.by_name('example.org').should_not include(aaron2_domain)
+  end
+
   it "should show all domains to an admin" do
     quentin_domain
     aaron_domain