implementation of user limits and refactoring

tests/Domain/Sharing/UserShareTest.php

@@ -16,10 +16,10 @@ class UserShareTest extends TestCase
      */
     public function it_share_single_file_without_password()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
-        $file = File::factory(File::class)
+        $file = File::factory()
             ->create(['user_id' => $user->id]);
 
         $this
@@ -49,7 +49,7 @@ class UserShareTest extends TestCase
      */
     public function it_share_folder_without_password()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)
@@ -82,7 +82,7 @@ class UserShareTest extends TestCase
      */
     public function it_share_folder_with_password()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)
@@ -121,7 +121,7 @@ class UserShareTest extends TestCase
      */
     public function it_share_folder_with_expiration_time()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)
@@ -148,7 +148,7 @@ class UserShareTest extends TestCase
      */
     public function it_share_folder_and_send_link_for_multiple_email()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)
@@ -175,7 +175,7 @@ class UserShareTest extends TestCase
      */
     public function it_send_existing_shared_folder_for_multiple_email_once_again()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)
@@ -205,7 +205,7 @@ class UserShareTest extends TestCase
      */
     public function it_revoke_single_share_record()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
         $folder = Folder::factory(Folder::class)

tests/Domain/Sharing/VisitorAccessToItemsTest.php

@@ -18,7 +18,7 @@ class VisitorAccessToItemsTest extends TestCase
     {
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $document = UploadedFile::fake()
@@ -26,7 +26,7 @@ class VisitorAccessToItemsTest extends TestCase
 
                 Storage::putFileAs("files/$user->id", $document, $document->name);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'filesize' => $document->getSize(),
                         'user_id'  => $user->id,
@@ -66,10 +66,10 @@ class VisitorAccessToItemsTest extends TestCase
      */
     public function it_try_to_get_protected_file_record()
     {
-        $user = User::factory(User::class)
+        $user = User::factory()
             ->create();
 
-        $file = File::factory(File::class)
+        $file = File::factory()
             ->create([
                 'user_id' => $user->id,
                 'name'    => 'fake-thumbnail.jpg',
@@ -95,7 +95,7 @@ class VisitorAccessToItemsTest extends TestCase
     {
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $fileName = Str::random() . '-fake-image.jpg';
@@ -105,7 +105,7 @@ class VisitorAccessToItemsTest extends TestCase
 
                 Storage::putFileAs("files/$user->id", $thumbnail, $fileName);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'user_id'   => $user->id,
                         'basename'  => $fileName,
@@ -149,7 +149,7 @@ class VisitorAccessToItemsTest extends TestCase
     {
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $fileName = Str::random() . '-fake-thumbnail.jpg';
@@ -159,7 +159,7 @@ class VisitorAccessToItemsTest extends TestCase
 
                 Storage::putFileAs("files/$user->id", $thumbnail, $fileName);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'user_id'  => $user->id,
                         'name'     => $fileName,

tests/Domain/Sharing/VisitorBrowseTest.php

@@ -77,7 +77,7 @@ class VisitorBrowseTest extends TestCase
      */
     public function it_authenticate_protected_file_with_correct_password()
     {
-        $file = File::factory(File::class)
+        $file = File::factory()
             ->create();
 
         $share = Share::factory(Share::class)
@@ -104,7 +104,7 @@ class VisitorBrowseTest extends TestCase
      */
     public function it_authenticate_protected_file_with_incorrect_password()
     {
-        $file = File::factory(File::class)
+        $file = File::factory()
             ->create();
 
         $share = Share::factory(Share::class)
@@ -131,7 +131,7 @@ class VisitorBrowseTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $root = Folder::factory(Folder::class)
@@ -157,7 +157,7 @@ class VisitorBrowseTest extends TestCase
                         'user_id'   => $user->id,
                     ]);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'parent_id' => $root->id,
                         'name'      => 'Document',
@@ -209,7 +209,7 @@ class VisitorBrowseTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $folder_level_1 = Folder::factory(Folder::class)
@@ -332,7 +332,7 @@ class VisitorBrowseTest extends TestCase
                         'password'     => bcrypt('secret'),
                     ]);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'name'      => 'Document',
                         'parent_id' => $folder->id,
@@ -386,7 +386,7 @@ class VisitorBrowseTest extends TestCase
                         'password'     => bcrypt('secret'),
                     ]);
 
-                File::factory(File::class)
+                File::factory()
                     ->create([
                         'name'    => 'Document',
                         'user_id' => $folder->user_id,
@@ -422,7 +422,7 @@ class VisitorBrowseTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'name' => 'Document',
                     ]);

tests/Domain/Sharing/VisitorManipulatingTest.php

@@ -19,7 +19,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $folder = Folder::factory(Folder::class)
@@ -27,7 +27,7 @@ class VisitorManipulatingTest extends TestCase
                         'user_id' => $user->id,
                     ]);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'parent_id' => $folder->id,
                         'user_id'   => $user->id,
@@ -88,7 +88,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $root = Folder::factory(Folder::class)
@@ -157,7 +157,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $folder = Folder::factory(Folder::class)
@@ -221,7 +221,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $folder = Folder::factory(Folder::class)
@@ -238,7 +238,7 @@ class VisitorManipulatingTest extends TestCase
                         'permission'   => 'editor',
                     ]);
 
-                $files = File::factory(File::class)
+                $files = File::factory()
                     ->count(2)
                     ->create([
                         'user_id'   => $user->id,
@@ -296,7 +296,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $folder = Folder::factory(Folder::class)
@@ -365,7 +365,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $root = Folder::factory(Folder::class)
@@ -379,7 +379,7 @@ class VisitorManipulatingTest extends TestCase
                         'parent_id' => $root->id,
                     ]);
 
-                $file = File::factory(File::class)
+                $file = File::factory()
                     ->create([
                         'user_id'   => $user->id,
                         'parent_id' => $root->id,
@@ -438,7 +438,7 @@ class VisitorManipulatingTest extends TestCase
         // check private or public share record
         collect([true, false])
             ->each(function ($is_protected) {
-                $user = User::factory(User::class)
+                $user = User::factory()
                     ->create();
 
                 $root = Folder::factory(Folder::class)