Add base64 image support

[bookstack] / tests / SharedTestHelpers.php

diff --git a/tests/SharedTestHelpers.php b/tests/SharedTestHelpers.php
index 02f7caae1cb859b4c14d105eab864a78eb4ec0fc..78c1f3b1825121a9c2a8ece027e29fe2fd087f3d 100644 (file)
--- a/tests/SharedTestHelpers.php
+++ b/tests/SharedTestHelpers.php
@@ -15,13 +15,11 @@ use BookStack\Auth\Permissions\PermissionService;
 use BookStack\Entities\Repos\PageRepo;
 use BookStack\Settings\SettingService;
 use BookStack\Uploads\HttpFetcher;
 use BookStack\Entities\Repos\PageRepo;
 use BookStack\Settings\SettingService;
 use BookStack\Uploads\HttpFetcher;

-use Illuminate\Http\Response;

 use Illuminate\Support\Env;
 use Illuminate\Support\Facades\Log;
 use Mockery;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;
 use Illuminate\Support\Env;
 use Illuminate\Support\Facades\Log;
 use Mockery;
 use Monolog\Handler\TestHandler;
 use Monolog\Logger;

-use Throwable;

 use Illuminate\Foundation\Testing\Assert as PHPUnit;
 
 trait SharedTestHelpers
 use Illuminate\Foundation\Testing\Assert as PHPUnit;
 
 trait SharedTestHelpers


@@ -32,7 +30,6 @@ trait SharedTestHelpers
 
     /**
      * Set the current user context to be an admin.
 
     /**
      * Set the current user context to be an admin.

-     * @return $this

      */
     public function asAdmin()
     {
      */
     public function asAdmin()
     {


@@ -41,19 +38,19 @@ trait SharedTestHelpers
 
     /**
      * Get the current admin user.
 
     /**
      * Get the current admin user.

-     * @return mixed

      */
      */

-    public function getAdmin() {
-        if($this->admin === null) {
+    public function getAdmin(): User
+    {
+        if (is_null($this->admin)) {

             $adminRole = Role::getSystemRole('admin');
             $this->admin = $adminRole->users->first();
         }
             $adminRole = Role::getSystemRole('admin');
             $this->admin = $adminRole->users->first();
         }

+

         return $this->admin;
     }
 
     /**
      * Set the current user context to be an editor.
         return $this->admin;
     }
 
     /**
      * Set the current user context to be an editor.

-     * @return $this

      */
     public function asEditor()
     {
      */
     public function asEditor()
     {


@@ -63,10 +60,10 @@ trait SharedTestHelpers
 
     /**
      * Get a editor user.
 
     /**
      * Get a editor user.

-     * @return mixed

      */
      */

-    protected function getEditor() {
-        if($this->editor === null) {
+    protected function getEditor(): User
+    {
+        if ($this->editor === null) {

             $editorRole = Role::getRole('editor');
             $this->editor = $editorRole->users->first();
         }
             $editorRole = Role::getRole('editor');
             $this->editor = $editorRole->users->first();
         }


@@ -87,10 +84,8 @@ trait SharedTestHelpers
 
     /**
      * Regenerate the permission for an entity.
 
     /**
      * Regenerate the permission for an entity.

-     * @param Entity $entity
-     * @throws Throwable

      */
      */

-    protected function regenEntityPermissions(Entity $entity)
+    protected function regenEntityPermissions(Entity $entity): void

     {
         $entity->rebuildPermissions();
         $entity->load('jointPermissions');
     {
         $entity->rebuildPermissions();
         $entity->load('jointPermissions');


@@ -98,40 +93,34 @@ trait SharedTestHelpers
 
     /**
      * Create and return a new bookshelf.
 
     /**
      * Create and return a new bookshelf.

-     * @param array $input
-     * @return Bookshelf

      */
      */

-    public function newShelf($input = ['name' => 'test shelf', 'description' => 'My new test shelf']) {
+    public function newShelf(array $input = ['name' => 'test shelf', 'description' => 'My new test shelf']): Bookshelf
+    {

         return app(BookshelfRepo::class)->create($input, []);
     }
 
     /**
      * Create and return a new book.
         return app(BookshelfRepo::class)->create($input, []);
     }
 
     /**
      * Create and return a new book.

-     * @param array $input
-     * @return Book

      */
      */

-    public function newBook($input = ['name' => 'test book', 'description' => 'My new test book']) {
+    public function newBook(array $input = ['name' => 'test book', 'description' => 'My new test book']): Book
+    {

         return app(BookRepo::class)->create($input);
     }
 
     /**
      * Create and return a new test chapter
         return app(BookRepo::class)->create($input);
     }
 
     /**
      * Create and return a new test chapter

-     * @param array $input
-     * @param Book $book
-     * @return Chapter

      */
      */

-    public function newChapter($input = ['name' => 'test chapter', 'description' => 'My new test chapter'], Book $book) {
+    public function newChapter(array $input = ['name' => 'test chapter', 'description' => 'My new test chapter'], Book $book): Chapter
+    {

         return app(ChapterRepo::class)->create($input, $book);
     }
 
     /**
      * Create and return a new test page
         return app(ChapterRepo::class)->create($input, $book);
     }
 
     /**
      * Create and return a new test page

-     * @param array $input
-     * @return Page
-     * @throws Throwable

      */
      */

-    public function newPage($input = ['name' => 'test page', 'html' => 'My new test page']) {
-        $book = Book::first();
+    public function newPage(array $input = ['name' => 'test page', 'html' => 'My new test page']): Page
+    {
+        $book = Book::query()->first();

         $pageRepo = app(PageRepo::class);
         $draftPage = $pageRepo->getNewDraftPage($book);
         return $pageRepo->publishDraft($draftPage, $input);
         $pageRepo = app(PageRepo::class);
         $draftPage = $pageRepo->getNewDraftPage($book);
         return $pageRepo->publishDraft($draftPage, $input);


@@ -139,9 +128,8 @@ trait SharedTestHelpers
 
     /**
      * Quickly sets an array of settings.
 
     /**
      * Quickly sets an array of settings.

-     * @param $settingsArray

      */
      */

-    protected function setSettings($settingsArray)
+    protected function setSettings(array $settingsArray): void

     {
         $settings = app(SettingService::class);
         foreach ($settingsArray as $key => $value) {
     {
         $settings = app(SettingService::class);
         foreach ($settingsArray as $key => $value) {


@@ -151,11 +139,8 @@ trait SharedTestHelpers
 
     /**
      * Manually set some permissions on an entity.
 
     /**
      * Manually set some permissions on an entity.

-     * @param Entity $entity
-     * @param array $actions
-     * @param array $roles

      */
      */

-    protected function setEntityRestrictions(Entity $entity, $actions = [], $roles = [])
+    protected function setEntityRestrictions(Entity $entity, array $actions = [], array $roles = []): void

     {
         $entity->restricted = true;
         $entity->permissions()->delete();
     {
         $entity->restricted = true;
         $entity->permissions()->delete();


@@ -180,7 +165,7 @@ trait SharedTestHelpers
     /**
      * Give the given user some permissions.
      */
     /**
      * Give the given user some permissions.
      */

-    protected function giveUserPermissions(User $user, array $permissions = [])
+    protected function giveUserPermissions(User $user, array $permissions = []): void

     {
         $newRole = $this->createNewRole($permissions);
         $user->attachRole($newRole);
     {
         $newRole = $this->createNewRole($permissions);
         $user->attachRole($newRole);


@@ -190,10 +175,8 @@ trait SharedTestHelpers
 
     /**
      * Create a new basic role for testing purposes.
 
     /**
      * Create a new basic role for testing purposes.

-     * @param array $permissions
-     * @return Role

      */
      */

-    protected function createNewRole($permissions = [])
+    protected function createNewRole(array $permissions = []): Role

     {
         $permissionRepo = app(PermissionsRepo::class);
         $roleData = factory(Role::class)->make()->toArray();
     {
         $permissionRepo = app(PermissionsRepo::class);
         $roleData = factory(Role::class)->make()->toArray();


@@ -203,8 +186,6 @@ trait SharedTestHelpers
 
     /**
      * Mock the HttpFetcher service and return the given data on fetch.
 
     /**
      * Mock the HttpFetcher service and return the given data on fetch.

-     * @param $returnData
-     * @param int $times

      */
     protected function mockHttpFetch($returnData, int $times = 1)
     {
      */
     protected function mockHttpFetch($returnData, int $times = 1)
     {


@@ -218,9 +199,6 @@ trait SharedTestHelpers
     /**
      * Run a set test with the given env variable.
      * Remembers the original and resets the value after test.
     /**
      * Run a set test with the given env variable.
      * Remembers the original and resets the value after test.

-     * @param string $name
-     * @param $value
-     * @param callable $callback

      */
     protected function runWithEnv(string $name, $value, callable $callback)
     {
      */
     protected function runWithEnv(string $name, $value, callable $callback)
     {


@@ -246,11 +224,8 @@ trait SharedTestHelpers
     /**
      * Check the keys and properties in the given map to include
      * exist, albeit not exclusively, within the map to check.
     /**
      * Check the keys and properties in the given map to include
      * exist, albeit not exclusively, within the map to check.

-     * @param array $mapToInclude
-     * @param array $mapToCheck
-     * @param string $message

      */
      */

-    protected function assertArrayMapIncludes(array $mapToInclude, array $mapToCheck, string $message = '') : void
+    protected function assertArrayMapIncludes(array $mapToInclude, array $mapToCheck, string $message = ''): void

     {
         $passed = true;
 
     {
         $passed = true;
 


@@ -301,7 +276,7 @@ trait SharedTestHelpers
         $testHandler = new TestHandler();
         $monolog->pushHandler($testHandler);
 
         $testHandler = new TestHandler();
         $monolog->pushHandler($testHandler);
 

-        Log::extend('testing', function() use ($monolog) {
+        Log::extend('testing', function () use ($monolog) {

             return $monolog;
         });
         Log::setDefaultDriver('testing');
             return $monolog;
         });
         Log::setDefaultDriver('testing');