From 3a7a404918d20dfe58842d60b062967b59bfeb39 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
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 <m.de.rooy@rijksmuseum.nl>

Sponsored-by: Rijksmuseum, Netherlands

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 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 d00cdfda53e..7c995d496a6 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.30.2