Merge branch '2.4-develop' into api-functional-graphql-removeItemFromCart

Author: atwixfirster

.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

@@ -6,6 +6,5 @@
  */
 -->
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-    <module name="Magento_AdvancedPricingImportExport" >
-    </module>
+    <module name="Magento_AdvancedPricingImportExport"/>
 </config>

app/code/Magento/AdvancedPricingImportExport/etc/module.xml

@@ -53,7 +53,7 @@ public function __construct(
     public function execute($commandName)
     {
         if (!array_key_exists($commandName, $this->commands)) {
-            throw new NotFoundException(__('Command was not found.'));
+            throw new NotFoundException(__('Command "%1" was not found.', $commandName));
         }
 
         /** @var \Magento\Analytics\Model\Connector\CommandInterface $command */

app/code/Magento/Analytics/Model/Connector.php

@@ -57,9 +57,13 @@ public function testExecute()
     }
 
     /**
+     * Executing non-existing command
+     *
      * @expectedException \Magento\Framework\Exception\NotFoundException
+     * @expectedExceptionMessage Command "register" was not found.
+     * @return void
      */
-    public function testExecuteCommandNotFound()
+    public function testExecuteCommandNotFound(): void
     {
         $commandName = 'register';
         $this->connector->execute($commandName);