From 2568234cdade350b7db83325b8375e1acdb06208 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 8 Jul 2022 09:08:40 +0200 Subject: [PATCH] Bug 28787: Comment selenium test that will fail if no SMTP server is defined This is a deadlock I think. Any ideas how we could continue to test that in any conditions? Signed-off-by: Marcel de Rooy Sponsored-by: Rijksmuseum, Netherlands Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi --- t/db_dependent/selenium/authentication_2fa.t | 26 +++++++++++--------- 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/t/db_dependent/selenium/authentication_2fa.t b/t/db_dependent/selenium/authentication_2fa.t index d00cdfda53..7c995d496a 100755 --- a/t/db_dependent/selenium/authentication_2fa.t +++ b/t/db_dependent/selenium/authentication_2fa.t @@ -150,7 +150,7 @@ SKIP: { }; subtest "Send OTP code" => sub { - plan tests => 4; + plan tests => 3; # Make sure the send won't fail because of invalid email addresses $patron->library->set( @@ -186,17 +186,19 @@ SKIP: { 'Email not sent will display an error' ); - $patron->email('test@example.org'); - $patron->store; - $driver->find_element('//a[@id="send_otp"]')->click; - $s->wait_for_ajax; - my $message = - $driver->find_element('//div[@id="email_success"]')->get_text; - is( - $message, - "The code has been sent by email, please check your inbox.", - 'The email must have been sent correctly' - ); + # This test will only pass if an SMTP server is defined + # It cannot be mocked from selenium tests + #$patron->email('test@example.org'); + #$patron->store; + #$driver->find_element('//a[@id="send_otp"]')->click; + #$s->wait_for_ajax; + #my $message = + # $driver->find_element('//div[@id="email_success"]')->get_text; + #is( + # $message, + # "The code has been sent by email, please check your inbox.", + # 'The email must have been sent correctly' + #); }; subtest "Disable" => sub { -- 2.39.5