Bugzilla – Attachment 133623 Details for
Bug 28998
Encrypt borrowers.secret
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 28998: [TO_BE_SQUASHED] Adjust secret in Selenium test
Bug-28998-TOBESQUASHED-Adjust-secret-in-Selenium-t.patch (text/plain), 1.82 KB, created by
Marcel de Rooy
on 2022-04-22 07:36:47 UTC
(
hide
)
Description:
Bug 28998: [TO_BE_SQUASHED] Adjust secret in Selenium test
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2022-04-22 07:36:47 UTC
Size:
1.82 KB
patch
obsolete
>From 961eecd9b709e2b4404a61fedfbc3657f5a5e11e Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Thu, 14 Apr 2022 13:50:52 +0000 >Subject: [PATCH] Bug 28998: [TO_BE_SQUASHED] Adjust secret in Selenium test >Content-Type: text/plain; charset=utf-8 > >This patch can be squashed with the previous one, if all is ok. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> >--- > t/db_dependent/selenium/authentication_2fa.t | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > >diff --git a/t/db_dependent/selenium/authentication_2fa.t b/t/db_dependent/selenium/authentication_2fa.t >index 6fc42228e7..25854efd0f 100755 >--- a/t/db_dependent/selenium/authentication_2fa.t >+++ b/t/db_dependent/selenium/authentication_2fa.t >@@ -21,7 +21,6 @@ use Test::More tests => 2; > use C4::Context; > use Koha::AuthUtils; > use Koha::Auth::TwoFactorAuth; >-use Koha::Encryption; > use t::lib::Mocks; > use t::lib::Selenium; > use t::lib::TestBuilder; >@@ -81,7 +80,7 @@ SKIP: { > $s->submit_form; > is( $driver->find_element('//div[@class="two-factor-status"]')->get_text(), 'Status: Enabled', '2FA is enabled' ); > $patron = $patron->get_from_storage; >- is( Koha::Encryption->new->decrypt_hex($patron->secret), $secret32, 'encrypted secret is set in DB' ); >+ is( $patron->decoded_secret, $secret32, 'encrypted secret is set in DB' ); > > }; > >@@ -90,7 +89,7 @@ SKIP: { > > my $mainpage = $s->base_url . q|mainpage.pl|; > >- my $secret32 = Koha::Encryption->new->decrypt_hex($patron->secret); >+ my $secret32 = $patron->decoded_secret; > { # ok first try > $driver->get($mainpage . q|?logout.x=1|); > $driver->get($s->base_url . q|circ/circulation.pl?borrowernumber=|.$patron->borrowernumber); >-- >2.20.1
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 28998
:
124772
|
124773
|
124774
|
133291
|
133317
|
133318
|
133319
|
133320
|
133321
|
133322
|
133323
|
133523
|
133524
|
133525
|
133526
|
133527
|
133528
|
133529
|
133580
|
133581
|
133582
|
133583
|
133584
|
133585
|
133586
|
133588
|
133619
|
133620
|
133621
|
133622
|
133623
|
133624
|
133625
|
133626
|
133627
|
133628
|
133629
|
133631
|
133632
|
133633
|
133634
|
133635
|
133636
|
133637
|
133638
|
133639
|
133640
|
133641
|
133642
|
133645
|
133646
|
133647
|
133648
|
133649
|
133650
|
133651
|
133652
|
133653
|
133654
|
133655
|
134636
|
134637
|
134671