Merge pull request #131 from artengin/update-routes

task: Update routes

Author: DenTray

routes/api.php

@@ -14,9 +14,9 @@
 
                 Route::controller(UserController::class)->group(function () {
                     Route::post('users', 'create');
-                    Route::put('users/{id}', 'update');
-                    Route::delete('users/{id}', 'delete');
-                    Route::get('users/{id}', 'get');
+                    Route::put('users/{id}', 'update')->whereNumber('id');
+                    Route::delete('users/{id}', 'delete')->whereNumber('id');
+                    Route::get('users/{id}', 'get')->whereNumber('id');
                     Route::get('users', 'search');
                     Route::get('profile', 'profile');
                     Route::put('profile', 'updateProfile');

tests/UserTest.php

@@ -260,6 +260,27 @@ public function testGet()
         $this->assertEqualsFixture('get_user', $response->json());
     }
 
+    public function testGetIdParamAsString()
+    {
+        $response = $this->actingAs(self::$admin)->json('get', '/users/test');
+
+        $response->assertNotFound();
+    }
+
+    public function testPutIdParamAsString()
+    {
+        $response = $this->actingAs(self::$admin)->json('put', '/users/test');
+
+        $response->assertNotFound();
+    }
+
+    public function testDeleteIdParamAsString()
+    {
+        $response = $this->actingAs(self::$admin)->json('delete', '/users/test');
+
+        $response->assertNotFound();
+    }
+
     public function testGetNotExists()
     {
         $response = $this->actingAs(self::$admin)->json('get', '/users/0');