diff options
Diffstat (limited to 'tests/Unit/Service/ItemServiceTest.php')
-rw-r--r-- | tests/Unit/Service/ItemServiceTest.php | 29 |
1 files changed, 12 insertions, 17 deletions
diff --git a/tests/Unit/Service/ItemServiceTest.php b/tests/Unit/Service/ItemServiceTest.php index 0ef5ce366..9a89d2e1a 100644 --- a/tests/Unit/Service/ItemServiceTest.php +++ b/tests/Unit/Service/ItemServiceTest.php @@ -13,11 +13,16 @@ namespace OCA\News\Tests\Unit\Service; +use OCA\News\Config\Config; +use OCA\News\Db\ItemMapper; use OCA\News\Service\ItemService; +use OCA\News\Service\ServiceNotFoundException; +use OCA\News\Utility\Time; use \OCP\AppFramework\Db\DoesNotExistException; use \OCA\News\Db\Item; use \OCA\News\Db\FeedType; +use OCP\IConfig; use PHPUnit\Framework\TestCase; @@ -41,7 +46,7 @@ class ItemServiceTest extends TestCase protected function setUp() { $this->time = 222; - $this->timeFactory = $this->getMockBuilder('\OCA\News\Utility\Time') + $this->timeFactory = $this->getMockBuilder(Time::class) ->disableOriginalConstructor() ->getMock(); $this->timeFactory->expects($this->any()) @@ -50,17 +55,13 @@ class ItemServiceTest extends TestCase $this->timeFactory->expects($this->any()) ->method('getMicroTime') ->will($this->returnValue($this->time)); - $this->mapper = $this->getMockBuilder('\OCA\News\Db\ItemMapper') + $this->mapper = $this->getMockBuilder(ItemMapper::class) ->disableOriginalConstructor() ->getMock(); - $this->config = $this->getMockBuilder( - '\OCA\News\Config\Config' - ) + $this->config = $this->getMockBuilder(Config::class) ->disableOriginalConstructor() ->getMock(); - $this->systemConfig = $this->getMockBuilder( - 'OCP\IConfig' - ) + $this->systemConfig = $this->getMockBuilder(IConfig::class) ->disableOriginalConstructor() ->getMock(); $this->itemService = new ItemService( @@ -339,9 +340,7 @@ class ItemServiceTest extends TestCase public function testReadDoesNotExist() { - $this->expectException( - '\OCA\News\Service\ServiceNotFoundException' - ); + $this->expectException(ServiceNotFoundException::class); $this->mapper->expects($this->once()) ->method('readItem') ->will($this->throwException(new DoesNotExistException(''))); @@ -352,9 +351,7 @@ class ItemServiceTest extends TestCase public function testStarDoesNotExist() { - $this->expectException( - '\OCA\News\Service\ServiceNotFoundException' - ); + $this->expectException(ServiceNotFoundException::class); $this->mapper->expects($this->once()) ->method('findByGuidHash') ->will($this->throwException(new DoesNotExistException(''))); @@ -462,9 +459,7 @@ class ItemServiceTest extends TestCase ) ); - $this->expectException( - '\OCA\News\Service\ServiceNotFoundException' - ); + $this->expectException(ServiceNotFoundException::class); $this->itemService->getNewestItemId($this->user); } |