Repository: appwrite/appwrite
File: tests/e2e/Services/Projects/ProjectsConsoleClientTest.php (PHP)

Code before:
        /**
         * Test for FAILURE
         */

        $response = $this->client->call(Client::METHOD_POST, '/projects', array_merge([
            'content-type' => 'application/json',
            'x-appwrite-project' => $this->getProject()['$id'],
        ], $this->getHeaders()), [
            'projectId' => ID::unique(),
            'name' => '',
        ]);

        $this->assertEquals(401, $response['headers']['status-code']);

        return ['projectId' => $projectId];
    }

    /**
     * @group smtpAndTemplates
     * @depends testCreateProject
     */
    public function testUpdateProjectSMTP($data): array
    {
        $id = $data['projectId'];
        $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' => true,
            'senderEmail' => 'mailer@appwrite.io',
            'senderName' => 'Mailer',
            'host' => 'maildev',
            'port' => 1025,
            'username' => 'user',
            'password' => 'password',
        ]);

        $this->assertEquals(200, $response['headers']['status-code']);
        $this->assertTrue($response['body']['smtpEnabled']);
        $this->assertEquals('mailer@appwrite.io', $response['body']['smtpSenderEmail']);
        $this->assertEquals('Mailer', $response['body']['smtpSenderName']);
        $this->assertEquals('maildev', $response['body']['smtpHost']);
        $this->assertEquals(1025, $response['body']['smtpPort']);
        $this->assertEquals('user', $response['body']['smtpUsername']);
        $this->assertEquals('password', $response['body']['smtpPassword']);
        $this->assertEquals('', $response['body']['smtpSecure']);

        /** Test Reading Project */
        $response = $this->client->call(Client::METHOD_GET, '/projects/' . $id, array_merge([
            'content-type' => 'application/json',
            'x-appwrite-project' => $this->getProject()['$id'],

Review comment (security):
Please format the comment like the other comments


```suggestion
        /**
         * Test for SUCCESS: Valid Credentials
         */
```

Code after:
        /**
         * Test for FAILURE
         */

        $response = $this->client->call(Client::METHOD_POST, '/projects', array_merge([
            'content-type' => 'application/json',
            'x-appwrite-project' => $this->getProject()['$id'],
        ], $this->getHeaders()), [
            'projectId' => ID::unique(),
            'name' => '',
        ]);

        $this->assertEquals(401, $response['headers']['status-code']);

        return ['projectId' => $projectId];
    }

    /**
     * @group smtpAndTemplates
     * @depends testCreateProject
     */
    public function testUpdateProjectSMTP($data): array
    {
        $id = $data['projectId'];
        
        /**Test for SUCCESS: Valid Credentials*/
        $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' => true,
            'senderEmail' => 'mailer@appwrite.io',
            'senderName' => 'Mailer',
            'host' => 'maildev',
            'port' => 1025,
            'username' => 'user',
            'password' => 'password',
        ]);

        $this->assertEquals(200, $response['headers']['status-code']);
        $this->assertTrue($response['body']['smtpEnabled']);
        $this->assertEquals('mailer@appwrite.io', $response['body']['smtpSenderEmail']);
        $this->assertEquals('Mailer', $response['body']['smtpSenderName']);
        $this->assertEquals('maildev', $response['body']['smtpHost']);
        $this->assertEquals(1025, $response['body']['smtpPort']);
        $this->assertEquals('user', $response['body']['smtpUsername']);
        $this->assertEquals('password', $response['body']['smtpPassword']);
        $this->assertEquals('', $response['body']['smtpSecure']);

        /** Test Reading Project */
        $response = $this->client->call(Client::METHOD_GET, '/projects/' . $id, array_merge([