From c26b7a68afb725479e909822f0014c6fd3fd7b5f Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 22 Dec 2017 15:21:51 -0300 Subject: [PATCH] Bug 19564: Add tests for _convert_sort_fields Signed-off-by: Nicolas Legrand --- .../Koha_SearchEngine_Elasticsearch_Search.t | 25 +++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t b/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t index 865a6ce..de7ef95 100644 --- a/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t +++ b/t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t @@ -17,7 +17,7 @@ use Modern::Perl; -use Test::More tests => 12; +use Test::More tests => 13; use t::lib::Mocks; use Koha::SearchEngine::Elasticsearch::QueryBuilder; @@ -229,3 +229,26 @@ subtest 'build_query tests' => sub { ); }; +subtest "_convert_sort_fields" => sub { + plan tests => 1; + my @sort_by = $builder->_convert_sort_fields(qw( call_number_asc author_dsc )); + is_deeply( + \@sort_by, + [ + { field => 'callnum', direction => 'asc' }, + { field => 'author', direction => 'desc' } + ], + 'sort fields should have been split correctly' + ); + + # We could expect this to pass, but direction is undef instead of 'desc' + #@sort_by = $builder->_convert_sort_fields(qw( call_number_asc author_desc )); + #is_deeply( + # \@sort_by, + # [ + # { field => 'callnum', direction => 'asc' }, + # { field => 'author', direction => 'desc' } + # ], + # 'sort fields should have been split correctly' + #); +}; -- 2.1.4