Attachment 52262 Details for Bug 16502 – Bug 16502: Adjust test for GetPlugins

[patch] Bug 16502: Adjust test for GetPlugins

Bug-16502-Adjust-test-for-GetPlugins.patch (text/plain), 1.42 KB, created by Jonathan Druart on 2016-06-11 15:13:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-06-11 15:13:51 UTC

Size: 1.42 KB

patch

obsolete

>From cb954a0e099c1c21d0a3183f135442dacd2d06c1 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Mon, 23 May 2016 08:58:02 +0200
>Subject: [PATCH] Bug 16502: Adjust test for GetPlugins
>
>The current test assumes that GetPlugins will return the test plugin
>as the first one in the array. This is not correct.
>This patch adjusts the test to a grep.
>
>Test plan:
>Run the test.
>Bonus: Add additional plugins. Run the test again.
>
>Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> t/db_dependent/Plugins.t | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Plugins.t b/t/db_dependent/Plugins.t
>index 308e215..73ce65b 100755
>--- a/t/db_dependent/Plugins.t
>+++ b/t/db_dependent/Plugins.t
>@@ -43,7 +43,8 @@ ok( $plugin->get_qualified_table_name('mytable') eq 'koha_plugin_test_mytable',
> ok( $plugin->get_plugin_http_path() eq '/plugin/Koha/Plugin/Test', 'Test $plugin->get_plugin_http_path()' );
> 
> my @plugins = Koha::Plugins->new({ enable_plugins => 1 })->GetPlugins( 'report' );
>-ok( $plugins[0]->get_metadata()->{'name'} eq 'Test Plugin', "Koha::Plugins::GetPlugins functions correctly" );
>+my @names = map { $_->get_metadata()->{'name'} } @plugins;
>+is( scalar grep( /^Test Plugin$/, @names), 1, "Koha::Plugins::GetPlugins functions correctly" );
> 
> SKIP: {
>     my $plugins_dir = C4::Context->config("pluginsdir");
>-- 
>2.8.1

Actions: View | Diff | Splinter Review

Attachments on bug 16502: 51544 | 51545 | 51713 | 51714 | 51804 | 51805 | 51806 | 51853 | 51854 | 51855 | 51856 | 52260 | 52261 | 52262 | 52263