Merge remote-tracking branch 'mainline/2.4-develop' into Chaika-PR24-2020-01-22

Author: Oleksandr Dubovyk

.github/CODEOWNERS

@@ -1,6 +1,7 @@
 ---
 name: Developer experience issue
 about: Issues related to customization, extensibility, modularity
+labels: 'Triage: Dev.Experience'
 
 ---
 

.github/ISSUE_TEMPLATE/developer-experience-issue.md

@@ -1,6 +1,7 @@
 ---
 name: Feature request
 about: Please consider reporting directly to https://github.com/magento/community-features
+labels: 'feature request'
 
 ---
 

.github/ISSUE_TEMPLATE/feature_request.md

@@ -15,6 +15,9 @@
     Letting us know what has changed and why it needed changing will help us validate this pull request.
 -->
 
+### Related Pull Requests
+<!-- related pull request placeholder -->
+
 ### Fixed Issues (if relevant)
 <!---
     If relevant, please provide a list of fixed issues in the format magento/magento2#<issue_number>.

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,134 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\AdminNotification\Test\Unit\Observer;
+
+use Magento\AdminNotification\Model\Feed;
+use Magento\AdminNotification\Model\FeedFactory;
+use Magento\AdminNotification\Observer\PredispatchAdminActionControllerObserver;
+use Magento\Backend\Model\Auth\Session;
+use Magento\Framework\Event\Observer;
+use Magento\Framework\TestFramework\Unit\Helper\ObjectManager;
+use PHPUnit\Framework\MockObject\MockObject;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * Test class for \Magento\AdminNotification\Observer\PredispatchAdminActionControllerObserver
+ */
+class PredispatchAdminActionControllerObserverTest extends TestCase
+{
+    private const STATUS_ADMIN_LOGGED_IN = true;
+    private const STATUS_ADMIN_IS_NOT_LOGGED = false;
+
+    /**
+     * @var Session|MockObject
+     */
+    private $backendAuthSessionMock;
+
+    /**
+     * @var Feed|MockObject
+     */
+    private $feedMock;
+
+    /**
+     * @var FeedFactory|MockObject
+     */
+    private $feedFactoryMock;
+
+    /**
+     * Object Manager Instance
+     *
+     * @var ObjectManager
+     */
+    private $objectManager;
+
+    /**
+     * Testable Object
+     *
+     * @var PredispatchAdminActionControllerObserver
+     */
+    private $observer;
+
+    /**
+     * @var Observer|MockObject
+     */
+    private $observerMock;
+
+    /**
+     * @inheritdoc
+     */
+    protected function setUp() : void
+    {
+        $this->objectManager = new ObjectManager($this);
+        $this->observerMock = $this->createMock(Observer::class);
+
+        $this->backendAuthSessionMock = $this->getMockBuilder(Session::class)
+            ->disableOriginalConstructor()
+            ->setMethods(['isLoggedIn'])
+            ->getMock();
+
+        $this->feedMock = $this->getMockBuilder(Feed::class)
+            ->disableOriginalConstructor()
+            ->setMethods(['checkUpdate'])
+            ->getMock();
+
+        $this->feedFactoryMock = $this->getMockBuilder(FeedFactory::class)
+            ->disableOriginalConstructor()
+            ->setMethods(['create'])
+            ->getMock();
+
+        $this->observer = $this->objectManager->getObject(
+            PredispatchAdminActionControllerObserver::class,
+            [
+                '_feedFactory' => $this->feedFactoryMock,
+                '_backendAuthSession' => $this->backendAuthSessionMock,
+            ]
+        );
+    }
+
+    /**
+     * Test observer when admin user is logged in
+     */
+    public function testPredispatchObserverWhenAdminLoggedIn()
+    {
+        $this->backendAuthSessionMock
+            ->expects($this->once())
+            ->method('isLoggedIn')
+            ->willReturn(self::STATUS_ADMIN_LOGGED_IN);
+
+        $this->feedFactoryMock
+            ->expects($this->once())
+            ->method('create')
+            ->willReturn($this->feedMock);
+
+        $this->feedMock
+            ->expects($this->once())
+            ->method('checkUpdate')
+            ->willReturn($this->feedMock);
+
+        $this->observer->execute($this->observerMock);
+    }
+
+    /**
+     * Test observer when admin user is not logged in
+     */
+    public function testPredispatchObserverWhenAdminIsNotLoggedIn()
+    {
+        $this->backendAuthSessionMock
+            ->expects($this->once())
+            ->method('isLoggedIn')
+            ->willReturn(self::STATUS_ADMIN_IS_NOT_LOGGED);
+
+        $this->feedFactoryMock
+            ->expects($this->never())
+            ->method('create');
+
+        $this->feedMock
+            ->expects($this->never())
+            ->method('checkUpdate');
+
+        $this->observer->execute($this->observerMock);
+    }
+}

app/code/Magento/AdminNotification/Test/Unit/Observer/PredispatchAdminActionControllerObserverTest.php

@@ -87,7 +87,7 @@ protected function _construct()
 
         $this->setId($this->getId() . '_Uploader');
 
-        $uploadUrl = $this->_urlBuilder->addSessionParam()->getUrl('adminhtml/*/upload');
+        $uploadUrl = $this->_urlBuilder->getUrl('adminhtml/*/upload');
         $this->getConfig()->setUrl($uploadUrl);
         $this->getConfig()->setParams(['form_key' => $this->getFormKey()]);
         $this->getConfig()->setFileField('file');

app/code/Magento/Backend/Block/Media/Uploader.php

@@ -95,7 +95,7 @@ protected function _prepareLayout()
         );
 
         $this->getUploader()->getConfig()->setUrl(
-            $this->_urlBuilder->addSessionParam()->getUrl('catalog/product_gallery/upload')
+            $this->_urlBuilder->getUrl('catalog/product_gallery/upload')
         )->setFileField(
             'image'
         )->setFilters(