fix type and test reset

app/controllers/shared/api.php

@@ -194,15 +194,15 @@ App::init()
             ->setParam('project.{scope}.network.outbound', 0);
 
         $smtp = $project->getAttribute('smtp', []);
-        if (!empty($smtp)) {
+        if (!empty($smtp) && ($smtp['enabled'] ?? false)) {
             $mails
                 ->setSmtpHost($smtp['host'] ?? '')
-                ->setSmtpHost($smtp['port'] ?? 25)
-                ->setSmtpHost($smtp['username'] ?? '')
-                ->setSmtpHost($smtp['password'] ?? '')
-                ->setSmtpHost($smtp['senderEmail'] ?? '')
-                ->setSmtpHost($smtp['senderName'] ?? '')
-                ->setSmtpPort($smtp['replyTo'] ?? '');
+                ->setSmtpPort($smtp['port'] ?? 25)
+                ->setSmtpUsername($smtp['username'] ?? '')
+                ->setSmtpPassword($smtp['password'] ?? '')
+                ->setSmtpSenderEmail($smtp['senderEmail'] ?? '')
+                ->setSmtpSenderName($smtp['senderName'] ?? '')
+                ->setSmtpReplyTo($smtp['replyTo'] ?? '');
         }
 
         $deletes->setProject($project);

tests/e2e/Services/Projects/ProjectsConsoleClientTest.php

@@ -474,6 +474,19 @@ class ProjectsConsoleClientTest extends Scope
         $this->assertEquals('emailuser', $response['body']['smtpUsername']);
         $this->assertEquals('securepassword', $response['body']['smtpPassword']);
         $this->assertEquals('', $response['body']['smtpSecure']);
+
+        /** Reset */
+        $response = $this->client->call(Client::METHOD_PATCH, '/projects/' . $id . '/smtp', array_merge([
+            'content-type' => 'application/json',
+            'x-appwrite-project' => $this->getProject()['$id'],
+        ], $this->getHeaders()), [
+            'enabled' => false,
+            'sender' => 'mailer@appwrite.io',
+            'host' => 'mail.appwrite.io',
+            'port' => 25,
+            'username' => 'emailuser',
+            'password' => 'securepassword',
+        ]);
         return $data;
     }