Refactored passing in the correct parameters and improved the tickets…

… action also.
baphled · May 17, 2010 · 8ba3c5d · 8ba3c5d
1 parent a3406a8
commit 8ba3c5d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/app/controllers/parking_controller.rb b/app/controllers/parking_controller.rb
@@ -1,12 +1,15 @@
 class ParkingController < ActionController::Base
   layout "application"
 
+  def index
+  end
+
   def new
     @resource = Resource.new
   end
 
   def tickets
-    @resource = Resource.new(:name => 'baphled', :project_id => '50164')
+    @resource = Resource.find(params[:resource][:id])
     @tickets = @resource.tickets(params[:tag])
   end
 

diff --git a/spec/controllers/parking_controller_spec.rb b/spec/controllers/parking_controller_spec.rb
@@ -4,8 +4,10 @@
   describe "GET, tickets" do
     context "requesting a list of tickets" do
       it "should do a search using the LightHouse API for tags" do
-        Lighthouse::Ticket.should_receive(:find).with(:all, :params => { :project_id => '50164', :q => "state:open tagged:feature" })
-        get :tickets, {:tag => 'feature'}
+        @resource = mock_model(Resource, :name => "baphled", :project_id => '50164').as_null_object
+        Resource.stub!(:find).and_return @resource
+        @resource.should_receive(:tickets).with("feature")
+        get :tickets, {:tag => 'feature', :resource => {:id => 1}}
       end
     end
   end