From 7bfc034cef303a922c7843385bfe29ec7a0ead49 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Wed, 1 Oct 2025 12:07:24 +0100 Subject: [PATCH] Bug 20253: (follow-up) Update tests for Bug 38489 rebase With the move to Koha::File::Transports we need to update the tests introduced in this patchset to match the others by generating a 'local' transport and using that. --- t/db_dependent/Koha/EDI.t | 110 ++++++++++++++++++++++++++++---------- 1 file changed, 81 insertions(+), 29 deletions(-) diff --git a/t/db_dependent/Koha/EDI.t b/t/db_dependent/Koha/EDI.t index d172af55e1e..c6151bb2069 100755 --- a/t/db_dependent/Koha/EDI.t +++ b/t/db_dependent/Koha/EDI.t @@ -710,18 +710,31 @@ subtest 'process_quote' => sub { $schema->storage->txn_begin; + # Create file transport for local testing + my $file_transport = $builder->build( + { + source => 'FileTransport', + value => { + name => 'Test Multiple Message Transport', + transport => 'local', + download_directory => $dirname, + upload_directory => $dirname, + } + } + ); + # Create vendor EDI account with po_is_basketname set to false (default) my $account_filename = $builder->build( { source => 'VendorEdiAccount', value => { - description => 'test vendor filename', - transport => 'FILE', - plugin => '', - san => $test_san, - quotes_enabled => 1, - auto_orders => 0, - po_is_basketname => 0, + description => 'test vendor filename', + file_transport_id => $file_transport->{file_transport_id}, + plugin => '', + san => $test_san, + quotes_enabled => 1, + auto_orders => 0, + po_is_basketname => 0, } } ); @@ -731,13 +744,13 @@ subtest 'process_quote' => sub { { source => 'VendorEdiAccount', value => { - description => 'test vendor purchase order', - transport => 'FILE', - plugin => '', - san => $test_san, - quotes_enabled => 1, - auto_orders => 0, - po_is_basketname => 1, + description => 'test vendor purchase order', + file_transport_id => $file_transport->{file_transport_id}, + plugin => '', + san => $test_san, + quotes_enabled => 1, + auto_orders => 0, + po_is_basketname => 1, } } ); @@ -814,14 +827,27 @@ subtest 'process_quote' => sub { $schema->storage->txn_begin; + # Create file transport for local testing + my $file_transport = $builder->build( + { + source => 'FileTransport', + value => { + name => 'Test Multiple Message Transport', + transport => 'local', + download_directory => $dirname, + upload_directory => $dirname, + } + } + ); + # Create vendor EDI account with po_is_basketname set to true my $account = $builder->build( { source => 'VendorEdiAccount', value => { - description => 'test vendor duplicate po', - transport => 'FILE', - po_is_basketname => 1, + description => 'test vendor duplicate po', + file_transport_id => $file_transport->{file_transport_id}, + po_is_basketname => 1, }, } ); @@ -1169,17 +1195,30 @@ subtest 'duplicate_po_number_blocking' => sub { $schema->storage->txn_begin; + # Create file transport for local testing + my $file_transport = $builder->build( + { + source => 'FileTransport', + value => { + name => 'Test Multiple Message Transport', + transport => 'local', + download_directory => $dirname, + upload_directory => $dirname, + } + } + ); + # Create vendor EDI account with auto_orders and po_is_basketname enabled my $account = $builder->build( { source => 'VendorEdiAccount', value => { - description => 'auto_orders duplicate test vendor', - transport => 'FILE', - plugin => '', - san => $test_san, - auto_orders => 1, - po_is_basketname => 1, + description => 'auto_orders duplicate test vendor', + file_transport_id => $file_transport->{file_transport_id}, + plugin => '', + san => $test_san, + auto_orders => 1, + po_is_basketname => 1, } } ); @@ -1288,17 +1327,30 @@ subtest 'duplicate_po_number_blocking' => sub { $schema->storage->txn_begin; + # Create file transport for local testing + my $file_transport = $builder->build( + { + source => 'FileTransport', + value => { + name => 'Test Multiple Message Transport', + transport => 'local', + download_directory => $dirname, + upload_directory => $dirname, + } + } + ); + # Create vendor EDI account with po_is_basketname DISABLED my $account = $builder->build( { source => 'VendorEdiAccount', value => { - description => 'feature disabled test vendor', - transport => 'FILE', - plugin => '', - san => $test_san, - auto_orders => 1, - po_is_basketname => 0, # Feature disabled + description => 'feature disabled test vendor', + file_transport_id => $file_transport->{file_transport_id}, + plugin => '', + san => $test_san, + auto_orders => 1, + po_is_basketname => 0, # Feature disabled } } ); -- 2.51.0