diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index f9ee2d76..9dd18801 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -197,13 +197,17 @@ class IssuesControllerTest < Test::Unit::TestCase end def test_index_sort - get :index, :sort_key => 'tracker' + get :index, :sort => 'tracker,id:desc' assert_response :success - sort_params = @request.session['issuesindex_sort'] - assert sort_params.is_a?(Hash) - assert_equal 'tracker', sort_params[:key] - assert_equal 'ASC', sort_params[:order] + sort_params = @request.session['issues_index_sort'] + assert sort_params.is_a?(String) + assert_equal 'tracker,id:desc', sort_params + + issues = assigns(:issues) + assert_not_nil issues + assert !issues.empty? + assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id) end def test_gantt diff --git a/test/unit/helpers/sort_helper_test.rb b/test/unit/helpers/sort_helper_test.rb index 9132beb2..d2aca110 100644 --- a/test/unit/helpers/sort_helper_test.rb +++ b/test/unit/helpers/sort_helper_test.rb @@ -20,6 +20,11 @@ require File.dirname(__FILE__) + '/../../test_helper' class SortHelperTest < HelperTestCase include SortHelper + def setup + @session = nil + @sort_param = nil + end + def test_default_sort_clause_with_array sort_init 'attr1', 'desc' sort_update(['attr1', 'attr2']) @@ -34,6 +39,13 @@ class SortHelperTest < HelperTestCase assert_equal 'table1.attr1 DESC', sort_clause end + def test_default_sort_clause_with_multiple_columns + sort_init 'attr1', 'desc' + sort_update({'attr1' => ['table1.attr1', 'table1.attr2'], 'attr2' => 'table2.attr2'}) + + assert_equal 'table1.attr1 DESC, table1.attr2 DESC', sort_clause + end + def test_params_sort @sort_param = 'attr1,attr2:desc' @@ -45,13 +57,13 @@ class SortHelperTest < HelperTestCase end def test_invalid_params_sort - @sort_param = 'attr3' + @sort_param = 'invalid_key' sort_init 'attr1', 'desc' sort_update({'attr1' => 'table1.attr1', 'attr2' => 'table2.attr2'}) - assert_nil sort_clause - assert_equal '', @session['foo_bar_sort'] + assert_equal 'table1.attr1 DESC', sort_clause + assert_equal 'attr1:desc', @session['foo_bar_sort'] end def test_invalid_order_params_sort