Attachment 73386 Details for Bug 19564 – Bug 19564: Add tests for _convert_sort_fields

[patch] Bug 19564: Add tests for _convert_sort_fields

Bug-19564-Add-tests-for-convertsortfields.patch (text/plain), 1.78 KB, created by Nick Clemens (kidclamp) on 2018-03-28 12:47:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2018-03-28 12:47:17 UTC

Size: 1.78 KB

patch

obsolete

>From 6b0e46533b5fb9480d5c3e274f2a580c2b42d1c2 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Fri, 22 Dec 2017 15:21:51 -0300
>Subject: [PATCH] Bug 19564: Add tests for _convert_sort_fields
>
>Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr>
>
>Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
>---
> .../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

Actions: View | Diff | Splinter Review

Attachments on bug 19564: 68892 | 68998 | 70137 | 73022 | 73023 | 73385 | 73386 | 73387