Added Unit Test for Actions Block. Use FQCN. Reduce Code-Smell

Author: DavidLambauer

app/code/Magento/AdminNotification/Block/Grid/Renderer/Actions.php

@@ -1,4 +1,6 @@
 <?php
+declare(strict_types=1);
+
 /**
  * Adminhtml AdminNotification Severity Renderer
  *
@@ -8,12 +10,17 @@
 
 namespace Magento\AdminNotification\Block\Grid\Renderer;
 
+use Magento\Backend\Block\Context;
+use Magento\Backend\Block\Widget\Grid\Column\Renderer\AbstractRenderer;
+use Magento\Framework\App\ActionInterface;
+use Magento\Framework\DataObject;
+use Magento\Framework\Url\Helper\Data;
+
 /**
  * Renderer class for action in the admin notifications grid
- *
  * @package Magento\AdminNotification\Block\Grid\Renderer
  */
-class Actions extends \Magento\Backend\Block\Widget\Grid\Column\Renderer\AbstractRenderer
+class Actions extends AbstractRenderer
 {
     /**
      * @var \Magento\Framework\Url\Helper\Data
@@ -25,11 +32,8 @@ class Actions extends \Magento\Backend\Block\Widget\Grid\Column\Renderer\Abstrac
      * @param \Magento\Framework\Url\Helper\Data $urlHelper
      * @param array $data
      */
-    public function __construct(
-        \Magento\Backend\Block\Context $context,
-        \Magento\Framework\Url\Helper\Data $urlHelper,
-        array $data = []
-    ) {
+    public function __construct(Context $context, Data $urlHelper, array $data = [])
+    {
         $this->_urlHelper = $urlHelper;
         parent::__construct($context, $data);
     }
@@ -40,16 +44,16 @@ public function __construct(
      * @param   \Magento\Framework\DataObject $row
      * @return  string
      */
-    public function render(\Magento\Framework\DataObject $row)
+    public function render(DataObject $row) : string
     {
-        $readDetailsHtml = $row->getUrl() ? '<a class="action-details" target="_blank" href="' .
-            $this->escapeUrl($row->getUrl())
+        $readDetailsHtml = $row->getData('url') ? '<a class="action-details" target="_blank" href="' .
+            $this->escapeUrl($row->getData('url'))
             . '">' .
             __('Read Details') . '</a>' : '';
 
-        $markAsReadHtml = !$row->getIsRead() ? '<a class="action-mark" href="' . $this->getUrl(
+        $markAsReadHtml = !$row->getData('is_read') ? '<a class="action-mark" href="' . $this->getUrl(
             '*/*/markAsRead/',
-            ['_current' => true, 'id' => $row->getId()]
+            ['_current' => true, 'id' => $row->getData('id')]
         ) . '">' . __(
             'Mark as Read'
         ) . '</a>' : '';
@@ -63,8 +67,8 @@ public function render(\Magento\Framework\DataObject $row)
                 '*/*/remove/',
                 [
                     '_current' => true,
-                    'id' => $row->getId(),
-                    \Magento\Framework\App\ActionInterface::PARAM_NAME_URL_ENCODED => $encodedUrl
+                    'id' => $row->getData('id'),
+                    ActionInterface::PARAM_NAME_URL_ENCODED => $encodedUrl
                 ]
             ),
             __('Are you sure?'),

app/code/Magento/AdminNotification/Test/Unit/Block/Grid/Renderer/ActionsTest.php

@@ -0,0 +1,70 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+/**
+ * Test class for \Magento\AdminNotification\Block\Grid\Renderer\Actions
+ */
+namespace Magento\AdminNotification\Test\Unit\Block\Grid\Renderer;
+
+use Magento\AdminNotification\Block\Grid\Renderer\Actions;
+use Magento\Backend\Block\Context;
+use Magento\Framework\DataObject;
+use Magento\Framework\Escaper;
+use Magento\Framework\Url\Helper\Data;
+use Magento\Framework\UrlInterface;
+use PHPUnit\Framework\TestCase;
+
+class ActionsTest extends TestCase
+{
+    /**
+     * System under Test
+     *
+     * @var Actions
+     */
+    private $sut;
+
+    protected function setUp() : void
+    {
+        parent::setUp();
+
+        /** @var Escaper | \PHPUnit_Framework_MockObject_MockObject $escaperMock */
+        $escaperMock = $this->getMockBuilder(Escaper::class)->disableOriginalConstructor()->getMock();
+        $escaperMock->expects($this->once())->method('escapeUrl')->willReturn('https://magento.com');
+
+        /** @var UrlInterface | \PHPUnit_Framework_MockObject_MockObject $urlBuilder */
+        $urlBuilder = $this->getMockBuilder(UrlInterface::class)->getMock();
+        $urlBuilder->expects($this->once())->method('getUrl')->willReturn('http://magento.com');
+
+        /** @var Context | \PHPUnit_Framework_MockObject_MockObject $contextMock */
+        $contextMock = $this->getMockBuilder(Context::class)->disableOriginalConstructor()->getMock();
+        $contextMock->expects($this->once())->method('getEscaper')->willReturn($escaperMock);
+        $contextMock->expects($this->once())->method('getUrlBuilder')->willReturn($urlBuilder);
+
+        /** @var Data | \PHPUnit_Framework_MockObject_MockObject $urlHelperMock */
+        $urlHelperMock = $this->getMockBuilder(Data::class)->disableOriginalConstructor()->getMock();
+        $urlHelperMock->expects($this->once())->method('getEncodedUrl')->willReturn('http://magento.com');
+
+        $this->sut = new Actions($contextMock, $urlHelperMock);
+
+    }
+
+    public function test_should_render_message_when_urlIsGiven() : void
+    {
+        $dataObject = new DataObject();
+        $dataObject->setdata('url', 'https://magento.com');
+        $dataObject->setdata('is_read', true);
+        $dataObject->setdata('id', 1);
+
+        $actual = $this->sut->render($dataObject);
+        $expected = <<<HTML
+<a class="action-details" target="_blank" href="https://magento.com">Read Details</a><a class="action-delete" href="http://magento.com" onClick="deleteConfirm('Are you sure?', this.href); return false;">Remove</a>
+HTML;
+
+        $this->assertEquals($actual, $expected);
+    }
+}