gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 39190: (follow-up) Two QA fixes 

This patch:

* Removes unused logger object instantiation
* Adds missing Test::NoWarnings to the API tests

Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Tomás Cohen Arazi 2025-09-19 08:27:00 -03:00 committed by Lucas Gass
parent a5f72d7362
commit 0c41c436c9
No known key found for this signature in database
GPG Key ID: 60B447398B0D36CA
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Koha/File/Transport/SFTP.pm
View File

@ -90,8 +90,6 @@ sub upload_file {
my ( $self, $local_file, $remote_file ) = @_;
my $operation = "upload";
my $logger = Koha::Logger->get_logger();
$self->{connection}->put( $local_file, $remote_file ) or return $self->_abort_operation($operation);
$self->add_message(

3
t/db_dependent/api/v1/file_transports.t
View File

@ -17,8 +17,9 @@
use Modern::Perl;
use Test::More tests => 5;
use Test::More tests => 6;
use Test::Mojo;
use Test::NoWarnings;
use t::lib::TestBuilder;
use t::lib::Mocks;