summaryrefslogtreecommitdiff
path: root/help/test
diff options
context:
space:
mode:
authorAzul <azul@leap.se>2012-12-17 10:31:08 +0100
committerAzul <azul@leap.se>2012-12-17 10:31:08 +0100
commitfa6992640a83396980112fd012e2b3a58f10861b (patch)
tree2b517ac3d9e6bcb80345888fb0a8a6d840cbbc5f /help/test
parent836f85fc362d4080ac9e0fc17455ec5a5c03cfee (diff)
parentfa21fe9a5ba10f937cd21e83aa26a088f58e2e8a (diff)
Merge remote-tracking branch 'origin/master' into feature/certs-from-ca-deamon
Diffstat (limited to 'help/test')
-rw-r--r--help/test/functional/tickets_controller_test.rb26
1 files changed, 18 insertions, 8 deletions
diff --git a/help/test/functional/tickets_controller_test.rb b/help/test/functional/tickets_controller_test.rb
index 592bd8c..b29fa17 100644
--- a/help/test/functional/tickets_controller_test.rb
+++ b/help/test/functional/tickets_controller_test.rb
@@ -118,7 +118,7 @@ class TicketsControllerTest < ActionController::TestCase
login User.last
ticket = Ticket.find('stubtestticketid')
- ticket.created_by = User.last.id # TODO: hacky, but confirms it is their ticket
+ ticket.created_by = @current_user.id # TODO: hacky, but confirms it is their ticket
ticket.save
#they should be able to comment if it is their ticket:
@@ -176,25 +176,33 @@ class TicketsControllerTest < ActionController::TestCase
login :is_admin? => true, :email => nil
- get :index, {:admin_status => "mine", :open_status => "open"}
+ get :index, {:admin_status => "all", :open_status => "open"}
assert assigns(:all_tickets).count > 1 # at least 2 tickets
# if we close one ticket, the admin should have 1 less open ticket they admin
assert_difference('assigns[:all_tickets].count', -1) do
assigns(:tickets).first.close
assigns(:tickets).first.save
- get :index, {:admin_status => "mine", :open_status => "open"}
+ get :index, {:admin_status => "all", :open_status => "open"}
end
+ end
+
+ test "admin_status mine vs all" do
testticket = Ticket.create :title => 'temp testytest'
+ login :is_admin? => true, :email => nil
- # test admin_status 'mine' vs 'all'
get :index, {:admin_status => "all", :open_status => "open"}
assert assigns(:all_tickets).include?(testticket)
get :index, {:admin_status => "mine", :open_status => "open"}
assert !assigns(:all_tickets).include?(testticket)
+ end
- # admin should have one more ticket if a new tick gets an admin comment
+ test "commenting on a ticket adds to tickets that are mine" do
+ testticket = Ticket.create :title => 'temp testytest'
+ login :is_admin? => true, :email => nil
+
+ get :index, {:admin_status => "mine", :open_status => "open"}
assert_difference('assigns[:all_tickets].count') do
put :update, :id => testticket.id, :ticket => {:comments_attributes => {"0" => {"body" =>"NEWER comment"}}}
get :index, {:admin_status => "mine", :open_status => "open"}
@@ -205,10 +213,12 @@ class TicketsControllerTest < ActionController::TestCase
assert_equal assigns(:ticket).comments.last.posted_by, @current_user.id
assigns(:ticket).destroy
+ end
- # test ordering
+ test "admin ticket ordering" do
- get :index, {:admin_status => "mine", :open_status => "open", :sort_order => 'created_at_desc'}
+ login :is_admin? => true, :email => nil
+ get :index, {:admin_status => "all", :open_status => "open", :sort_order => 'created_at_desc'}
# this will consider all tickets, not just those on first page
first_tick = assigns(:all_tickets).all.first
@@ -216,7 +226,7 @@ class TicketsControllerTest < ActionController::TestCase
assert first_tick.created_at > last_tick.created_at
# and now reverse order:
- get :index, {:admin_status => "mine", :open_status => "open", :sort_order => 'created_at_asc'}
+ get :index, {:admin_status => "all", :open_status => "open", :sort_order => 'created_at_asc'}
assert_equal first_tick, assigns(:all_tickets).last
assert_equal last_tick, assigns(:all_tickets).first