Bugzilla – Attachment 152632 Details for
Bug 30524
Add base framework for dealing with CSRF in Koha
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 30524: Attempt to mock
Bug-30524-Attempt-to-mock.patch (text/plain), 1.91 KB, created by
Martin Renvoize (ashimema)
on 2023-06-23 12:17:48 UTC
(
hide
)
Description:
Bug 30524: Attempt to mock
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2023-06-23 12:17:48 UTC
Size:
1.91 KB
patch
obsolete
>From acde5f72b17e02aae7c427f8a0a3a2e420348073 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Fri, 23 Jun 2023 13:16:44 +0100 >Subject: [PATCH] Bug 30524: Attempt to mock > >--- > t/Output.t | 25 +++++++++++++++++++++---- > 1 file changed, 21 insertions(+), 4 deletions(-) > >diff --git a/t/Output.t b/t/Output.t >index c9b202f3dc..79a7f212c3 100755 >--- a/t/Output.t >+++ b/t/Output.t >@@ -19,7 +19,7 @@ use Modern::Perl; > > use Test::More tests => 8; > use Test::Warn; >-use Test::Trap; >+use Test::MockModule; > > use File::Temp qw/tempfile/; > use CGI qw ( -utf8 ); >@@ -103,6 +103,18 @@ subtest 'output_and_exit_if_error() tests' => sub { > > plan tests => 2; > >+ my $exit; >+ my $output = Test::MockModule->new('C4::Output'); >+ $output->mock( >+ 'output_and_exit', >+ sub { >+ my ( $query, $cookie, $template, $error ) = @_; >+ $template->param( blocking_error => $error ); >+ output_html_with_http_headers( $query, $cookie, $template->output ); >+ $exit = 1; >+ } >+ ); >+ > t::lib::Mocks::mock_config( 'pluginsdir', [ C4::Context::temporary_directory ] ); > my ( $fh, $fn ) = tempfile( SUFFIX => '.tt', UNLINK => 1, DIR => C4::Context::temporary_directory ); > print $fh qq|[% blocking_error %]|; >@@ -119,7 +131,12 @@ subtest 'output_and_exit_if_error() tests' => sub { > } > ); > >- my @r = trap { output_and_exit_if_error($query, $cookie, $template, { check => 'csrf_token' }) }; >- is ( $trap->exit, 0, 'Expecting &output_and_exit_if_error to exit with 0' ); >- like ( $trap->stdout, qr/wrong_csrf_token/, 'Expecting wrong_csrf_token' ); >+ local *STDOUT; >+ my $stdout; >+ >+ open STDOUT, '>', \$stdout; >+ output_and_exit_if_error($query, $cookie, $template, { check => 'csrf_token' }); >+ is( $exit, 1, "Exit"); >+ like ( $stdout, qr/wrong_csrf_token/, 'Expecting wrong_csrf_token' ); >+ close STDOUT; > }; >-- >2.41.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 30524
:
133249
|
133250
|
133420
|
138055
|
141297
|
152629
|
152630
|
152631
|
152632
|
152634
|
152635
|
152637
|
152638
|
153964
|
153965
|
153975
|
153976
|
154000
|
154038