Merge branch 'feature/fix-use-email-for-login' into 'develop'

Use email for authentication

See merge request !6

app/Http/Requests/LoginRequest.php

@@ -11,9 +11,10 @@
  *   @OA\JsonContent(
  *     required={"username","password"},
  *     @OA\Property(
- *       property="username",
+ *       property="email",
  *       type="string",
- *       description="Username of the user",
+ *       format="email",
+ *       description="Email of the user",
  *     ),
  *     @OA\Property(
  *       property="password",
@@ -43,7 +44,7 @@ public function authorize()
     public function rules()
     {
         return [
-            'username' => 'required|string',
+            'email' => 'required|string|email',
             'password' => 'required|string',
         ];
     }

tests/Feature/LoginTest.php

@@ -25,6 +25,21 @@ public function test_empty() {
         $response->assertStatus(422);
     }
 
+    public function test_invalid_email() {
+        
+        $user = User::create([
+            'username' => 'test',
+            'email' => 'test@localhost',
+            'password' => Hash::make('test'),
+        ]);
+
+        $response = $this->postJson("/api/auth/login", [
+            "email" => "testfdfgf",
+            "password" => "123"
+        ]);
+        $response->assertStatus(422);
+    }
+
     public function test_wrong_password() {
         
         $user = User::create([
@@ -34,7 +49,7 @@ public function test_wrong_password() {
         ]);
 
         $response = $this->postJson("/api/auth/login", [
-            "username" => "test",
+            "email" => "test@localhost",
             "password" => "123"
         ]);
         $response->assertStatus(401);
@@ -48,7 +63,7 @@ public function test_good() {
         ]);
 
         $response = $this->postJson("/api/auth/login", [
-            "username" => "test",
+            "email" => "test@localhost",
             "password" => "test",
         ]);