From 2cba1e4146221f28ac195f62fbbb6cd9dbb555fd Mon Sep 17 00:00:00 2001
From: Marco Nassabain <marco.nassabain@hotmail.com>
Date: Fri, 5 Mar 2021 22:14:56 +0100
Subject: =?UTF-8?q?=F0=9F=A7=B9=20Cleanup=20code,=20remove=20spaces,=20ren?=
 =?UTF-8?q?ame=20variables?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Marco Nassabain <marco.nassabain@hotmail.com>
---
 tests/Unit/Db/ItemMapperPaginatedTest.php | 1 +
 tests/Unit/Db/ItemMapperTest.php          | 6 +-----
 tests/Unit/Service/ItemServiceTest.php    | 3 +--
 tests/Unit/Service/ShareServiceTest.php   | 4 +---
 4 files changed, 4 insertions(+), 10 deletions(-)

(limited to 'tests/Unit')

diff --git a/tests/Unit/Db/ItemMapperPaginatedTest.php b/tests/Unit/Db/ItemMapperPaginatedTest.php
index e0625aa0c..330eaff51 100644
--- a/tests/Unit/Db/ItemMapperPaginatedTest.php
+++ b/tests/Unit/Db/ItemMapperPaginatedTest.php
@@ -1286,4 +1286,5 @@ class ItemMapperPaginatedTest extends MapperTestUtility
         $result = $this->class->findAllFolder('jack', 2, 10, 10, false, false, ['key', 'word']);
         $this->assertEquals([Item::fromRow(['id' => 4])], $result);
     }
+
 }
diff --git a/tests/Unit/Db/ItemMapperTest.php b/tests/Unit/Db/ItemMapperTest.php
index ba73d93b8..5cb42c3fa 100644
--- a/tests/Unit/Db/ItemMapperTest.php
+++ b/tests/Unit/Db/ItemMapperTest.php
@@ -388,11 +388,7 @@ class ItemMapperTest extends MapperTestUtility
 
         $this->builder->expects($this->exactly(3))
             ->method('andWhere')
-            ->withConsecutive(
-                ['feeds.user_id = :user_id'],
-                ['feeds.id = :feed_id'],
-                ['items.guid_hash = :guid_hash']
-            )
+            ->withConsecutive(['feeds.user_id = :user_id'], ['feeds.id = :feed_id'], ['items.guid_hash = :guid_hash'])
             ->will($this->returnSelf());
 
         $this->builder->expects($this->exactly(3))
diff --git a/tests/Unit/Service/ItemServiceTest.php b/tests/Unit/Service/ItemServiceTest.php
index b1cbe6d8d..f91692cda 100644
--- a/tests/Unit/Service/ItemServiceTest.php
+++ b/tests/Unit/Service/ItemServiceTest.php
@@ -41,7 +41,6 @@ class ItemServiceTest extends TestCase
      * @var MockObject|ItemMapperV2
      */
     private $mapper;
-
     /**
      * @var  ItemServiceV2
      */
@@ -71,7 +70,6 @@ class ItemServiceTest extends TestCase
         $this->mapper = $this->getMockBuilder(ItemMapperV2::class)
             ->disableOriginalConstructor()
             ->getMock();
-
         $this->config = $this->getMockBuilder(IConfig::class)
             ->disableOriginalConstructor()
             ->getMock();
@@ -631,4 +629,5 @@ class ItemServiceTest extends TestCase
 
         $this->class->purgeOverThreshold(5);
     }
+
 }
diff --git a/tests/Unit/Service/ShareServiceTest.php b/tests/Unit/Service/ShareServiceTest.php
index 4c74e39a5..06359d572 100644
--- a/tests/Unit/Service/ShareServiceTest.php
+++ b/tests/Unit/Service/ShareServiceTest.php
@@ -107,8 +107,7 @@ class ShareServiceTest extends TestCase
 
         // Shared item
         $sharedItem = clone $item;
-        $sharedItem
-            ->setUnread(1)              // A newly shared item is unread, ...
+        $sharedItem->setUnread(1)       // A newly shared item is unread, ...
             ->setStarred(0)             // ... not starred, ...
             ->setFeedId(100)            // ... placed in the 'Shared with me' feed, ...
             ->setSharedBy($this->uid);  // ... and contains the senders user ID
@@ -190,7 +189,6 @@ class ShareServiceTest extends TestCase
             ->with($this->recipient, $feedUrl)
             ->will($this->returnValue(null));
 
-        // TODO: Test if the feed is properly created
         $this->feedService->expects($this->once())
             ->method('insert')
             ->will($this->returnValue($feed));
-- 
cgit v1.2.3