Turns out we probably want to redirect to an empty cart if the order …

…is complete.

core/app/controllers/checkout_controller.rb

@@ -35,7 +35,6 @@ def object_params
  def load_order
  @order = current_order
  redirect_to cart_path and return if @order.nil? or @order.empty?
- redirect_to order_path(@order) if @order.complete?
  @order.state = params[:state] if params[:state]
  state_callback(:before)
  end

core/spec/controllers/checkout_controller_spec.rb

@@ -111,16 +111,15 @@
  end
 
  context "when order is complete" do
- before(:each) { order.stub(:complete?).and_return true }
-
+ before { controller.stub! :current_order => nil }
  it "should not change the state if order is completed" do
  order.should_not_receive(:update_attribute)
  post :update, {:state => "confirm"}
  end
 
- it "should redirect to the order_path" do
+ it "should redirect to the cart_path" do
  post :update, {:state => "confirm"}
- response.should redirect_to order_path(assigns[:order])
+ response.should redirect_to cart_path
  end
  end