Kyle M Hall
323b1553b0
While PayPal is fairly universal, there is a plethora of online payment system that are far more localized, servicing a single country ( e.g. Bug 18968 ) or even a single city! Instead of adding support for each and every one of these payment options directly into Koha, it makes more sense to add the ability to create online payment plugins. Test Plan: 1) Apply this patch 2) Download and install the Kitchen Sink plugin version 2.1.1 or later https://github.com/bywatersolutions/koha-plugin-kitchen-sink/releases 3) In the plugin options, enable the opac payments option 4) Create a patron with one or more fines 5) Log into the opac as that patron, note you now have the option to pay online via KitchenSink ImaginaryPay 6) Make an online payment 7) Note the payment was processed correctly Sponsored-by: Washoe County Library System Signed-off-by: Kyle M Hall <kyle@gmail.com> Signed-off-by: Magnus Enger <magnus@libriotech.no> Awesome enhancement! I know we want to add at least one Norwegian payment service at some point. I followed the test plan and everything works as advertised. Turning off the "opac payments option" makes the option dissappear cleanly from the OPAC. I have *not* looked at the code or done any considerations about security. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
102 lines
4.6 KiB
Perl
Executable file
102 lines
4.6 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
use Modern::Perl;
|
|
|
|
use Test::More tests => 31;
|
|
use File::Basename;
|
|
use File::Temp qw( tempdir );
|
|
use FindBin qw($Bin);
|
|
use Archive::Extract;
|
|
use Module::Load::Conditional qw(can_load);
|
|
|
|
use C4::Context;
|
|
use t::lib::Mocks;
|
|
|
|
BEGIN {
|
|
push( @INC, dirname(__FILE__) . '/..' );
|
|
|
|
use_ok('Koha::Plugins');
|
|
use_ok('Koha::Plugins::Handler');
|
|
use_ok('Koha::Plugins::Base');
|
|
use_ok('Koha::Plugin::Test');
|
|
}
|
|
|
|
ok( can_load( modules => { "Koha::Plugin::Test" => undef } ), 'Test can_load' );
|
|
|
|
my $plugin = Koha::Plugin::Test->new({ enable_plugins => 1});
|
|
|
|
isa_ok( $plugin, "Koha::Plugin::Test", 'Test plugin class' );
|
|
isa_ok( $plugin, "Koha::Plugins::Base", 'Test plugin parent class' );
|
|
|
|
ok( $plugin->can('report'), 'Test plugin can report' );
|
|
ok( $plugin->can('tool'), 'Test plugin can tool' );
|
|
ok( $plugin->can('to_marc'), 'Test plugin can to_marc' );
|
|
ok( $plugin->can('opac_online_payment'), 'Test plugin can opac_online_payment' );
|
|
ok( $plugin->can('opac_online_payment_begin'), 'Test plugin can opac_online_payment_begin' );
|
|
ok( $plugin->can('opac_online_payment_end'), 'Test plugin can opac_online_payment_end' );
|
|
ok( $plugin->can('configure'), 'Test plugin can configure' );
|
|
ok( $plugin->can('install'), 'Test plugin can install' );
|
|
ok( $plugin->can('uninstall'), 'Test plugin can install' );
|
|
|
|
is( Koha::Plugins::Handler->run({ class => "Koha::Plugin::Test", method => 'report', enable_plugins => 1 }), "Koha::Plugin::Test::report", 'Test run plugin report method' );
|
|
|
|
my $metadata = $plugin->get_metadata();
|
|
is( $metadata->{'name'}, 'Test Plugin', 'Test $plugin->get_metadata()' );
|
|
|
|
is( $plugin->get_qualified_table_name('mytable'), 'koha_plugin_test_mytable', 'Test $plugin->get_qualified_table_name()' );
|
|
is( $plugin->get_plugin_http_path(), '/plugin/Koha/Plugin/Test', 'Test $plugin->get_plugin_http_path()' );
|
|
|
|
# testing GetPlugins
|
|
my @plugins = Koha::Plugins->new({ enable_plugins => 1 })->GetPlugins({
|
|
method => 'report'
|
|
});
|
|
my @names = map { $_->get_metadata()->{'name'} } @plugins;
|
|
is( scalar grep( /^Test Plugin$/, @names), 1, "Koha::Plugins::GetPlugins functions correctly" );
|
|
@plugins = Koha::Plugins->new({ enable_plugins => 1 })->GetPlugins({
|
|
metadata => { my_example_tag => 'find_me' },
|
|
});
|
|
@names = map { $_->get_metadata()->{'name'} } @plugins;
|
|
is( scalar grep( /^Test Plugin$/, @names), 1, "GetPlugins also found Test Plugin via a metadata tag" );
|
|
# Test two metadata conditions; one does not exist for Test.pm
|
|
# Since it is a required key, we should not find the same results
|
|
my @plugins2 = Koha::Plugins->new({ enable_plugins => 1 })->GetPlugins({
|
|
metadata => { my_example_tag => 'find_me', not_there => '1' },
|
|
});
|
|
isnt( scalar @plugins2, scalar @plugins, 'GetPlugins with two metadata conditions' );
|
|
|
|
for my $pass ( 1 .. 2 ) {
|
|
my $plugins_dir;
|
|
my $module_name = 'Koha::Plugin::Com::ByWaterSolutions::KitchenSink';
|
|
my $pm_path = 'Koha/Plugin/Com/ByWaterSolutions/KitchenSink.pm';
|
|
if ( $pass == 1 ) {
|
|
my $plugins_dir1 = tempdir( CLEANUP => 1 );
|
|
t::lib::Mocks::mock_config('pluginsdir', $plugins_dir1);
|
|
$plugins_dir = $plugins_dir1;
|
|
push @INC, $plugins_dir1;
|
|
} else {
|
|
my $plugins_dir1 = tempdir( CLEANUP => 1 );
|
|
my $plugins_dir2 = tempdir( CLEANUP => 1 );
|
|
t::lib::Mocks::mock_config('pluginsdir', [ $plugins_dir2, $plugins_dir1 ]);
|
|
$plugins_dir = $plugins_dir2;
|
|
pop @INC;
|
|
push @INC, $plugins_dir2;
|
|
push @INC, $plugins_dir1;
|
|
}
|
|
my $full_pm_path = $plugins_dir . '/' . $pm_path;
|
|
|
|
my $ae = Archive::Extract->new( archive => "$Bin/KitchenSinkPlugin.kpz", type => 'zip' );
|
|
unless ( $ae->extract( to => $plugins_dir ) ) {
|
|
warn "ERROR: " . $ae->error;
|
|
}
|
|
use_ok('Koha::Plugin::Com::ByWaterSolutions::KitchenSink');
|
|
$plugin = Koha::Plugin::Com::ByWaterSolutions::KitchenSink->new({ enable_plugins => 1});
|
|
my $table = $plugin->get_qualified_table_name( 'mytable' );
|
|
|
|
ok( -f $plugins_dir . "/Koha/Plugin/Com/ByWaterSolutions/KitchenSink.pm", "KitchenSink plugin installed successfully" );
|
|
$INC{$pm_path} = $full_pm_path; # FIXME I do not really know why, but if this is moved before the $plugin constructor, it will fail with Can't locate object method "new" via package "Koha::Plugin::Com::ByWaterSolutions::KitchenSink"
|
|
Koha::Plugins::Handler->delete({ class => "Koha::Plugin::Com::ByWaterSolutions::KitchenSink", enable_plugins => 1 });
|
|
my $sth = C4::Context->dbh->table_info( undef, undef, $table, 'TABLE' );
|
|
my $info = $sth->fetchall_arrayref;
|
|
is( @$info, 0, "Table $table does no longer exist" );
|
|
ok( !( -f $full_pm_path ), "Koha::Plugins::Handler::delete works correctly." );
|
|
}
|