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

Bug 40429: Remove warnings from Koha/Patron/Modifications.t 

t/db_dependent/Koha/Patron/Modifications.t .. 1/7 DBIx::Class::Storage::DBI::insert(): Missing value for primary key column 'borrowernumber' on BorrowerModification - perhaps you forgot to set its 'is_auto_increme
nt' attribute during add_columns()? Treating 'borrowernumber' implicitly as an autoinc, and attempting value retrieval at /kohadevbox/koha/Koha/Object.pm line 174
t/db_dependent/Koha/Patron/Modifications.t .. 5/7 DBIx::Class::Storage::DBI::insert(): Missing value for primary key column 'verification_token' on BorrowerModification - perhaps you forgot to set its 'is_auto_inc
rement' attribute during add_columns()? Treating 'verification_token' implicitly as an autoinc, and attempting value retrieval at /kohadevbox/koha/Koha/Object.pm line 174

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit f851367cf4)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
Jonathan Druart 2025-07-17 09:51:02 +02:00 committed by Paul Derscheid
parent 8e93b7c900
commit 0bd9f325c8
No known key found for this signature in database
GPG Key ID: C25F9FD44CED595B
1 changed files with 23 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
t/db_dependent/Koha/Patron/Modifications.t
View File

@ -19,7 +19,8 @@ use Modern::Perl;
use utf8;
use Test::More tests => 7;
use Test::More tests => 8;
use Test::NoWarnings;
use Test::Exception;
use t::lib::TestBuilder;
@ -49,9 +50,12 @@ subtest 'new() tests' => sub {
Koha::Patron::Modifications->search->delete;
my $patron = $builder->build_object( { class => 'Koha::Patrons' } );
# Create new pending modification
Koha::Patron::Modification->new(
{
borrowernumber => $patron->borrowernumber,
verification_token => '1234567890',
changed_fields => 'surname,firstname',
surname => 'Hall',
@ -443,9 +447,10 @@ subtest 'dateofbirth tests' => sub {
{ class => 'Koha::Patrons', value => { dateofbirth => '1980-01-01', surname => 'a_surname' } } );
my $patron_modification = Koha::Patron::Modification->new(
{
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => undef
verification_token => '1234567890',
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => undef
}
)->store;
$patron_modification->approve;
@ -459,9 +464,10 @@ subtest 'dateofbirth tests' => sub {
# Adding a dateofbirth
$patron_modification = Koha::Patron::Modification->new(
{
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => '1980-02-02'
verification_token => '1234567890',
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => '1980-02-02'
}
)->store;
$patron_modification->approve;
@ -476,10 +482,11 @@ subtest 'dateofbirth tests' => sub {
# Modifying a dateofbirth
$patron_modification = Koha::Patron::Modification->new(
{
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => '1980-03-03',
surname => undef
verification_token => '1234567890',
changed_fields => 'borrowernumber,dateofbirth',
borrowernumber => $patron->borrowernumber,
dateofbirth => '1980-03-03',
surname => undef
}
)->store;
$patron_modification->approve;
@ -494,10 +501,11 @@ subtest 'dateofbirth tests' => sub {
# Modifying something else
$patron_modification = Koha::Patron::Modification->new(
{
changed_fields => 'borrowernumber,surname',
borrowernumber => $patron->borrowernumber,
surname => 'another_surname',
dateofbirth => undef
verification_token => '1234567890',
changed_fields => 'borrowernumber,surname',
borrowernumber => $patron->borrowernumber,
surname => 'another_surname',
dateofbirth => undef
}
)->store;
$patron_modification->approve;