MAGETWO-95048: Sidebar navigation is broken for Stores > Configuration, All Stores.

- Pass frontname to getSecretKey if route not found
- Add MFTF test to cover this fix

Author: tjwiebell

app/code/Magento/Backend/Block/Menu.php

@@ -75,7 +75,7 @@ class Menu extends \Magento\Backend\Block\Template
     private $anchorRenderer;
 
     /**
-     * @var ConfigInterface
+     * @var \Magento\Framework\App\Route\ConfigInterface
      */
     private $routeConfig;
 
@@ -216,7 +216,7 @@ protected function _callbackSecretKey($match)
     {
         $routeId = $this->routeConfig->getRouteByFrontName($match[1]);
         return \Magento\Backend\Model\UrlInterface::SECRET_KEY_PARAM_NAME . '/' . $this->_url->getSecretKey(
-            $routeId,
+            $routeId ?: $match[1],
             $match[2],
             $match[3]
         );

app/code/Magento/Backend/Test/Mftf/Section/AdminMenuSection.xml

@@ -7,9 +7,10 @@
 -->
 
 <sections xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-          xsi:noNamespaceSchemaLocation="../../../../../../../dev/tests/acceptance/vendor/magento/magento2-functional-testing-framework/src/Magento/FunctionalTestingFramework/Page/etc/SectionObject.xsd">
+          xsi:noNamespaceSchemaLocation="urn:magento:mftf:Page/etc/SectionObject.xsd">
     <section name="AdminMenuSection">
         <element name="catalog" type="button" selector="#menu-magento-catalog-catalog"/>
+        <element name="catalogProducts" type="button" selector="#nav li[data-ui-id='menu-magento-catalog-catalog-products']"/>
         <element name="customers" type="button" selector="#menu-magento-customer-customer"/>
         <element name="content" type="button" selector="#menu-magento-backend-content"/>
         <element name="widgets" type="button" selector="#nav li[data-ui-id='menu-magento-widget-cms-widget-instance']"/>

app/code/Magento/Backend/Test/Mftf/Test/AdminMenuNavigationWithSecretKeysTest.xml

@@ -0,0 +1,50 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+ /**
+  * Copyright © Magento, Inc. All rights reserved.
+  * See COPYING.txt for license details.
+  */
+-->
+
+<tests xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+        xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/testSchema.xsd">
+    <test name="AdminMenuNavigationWithSecretKeysTest">
+        <annotations>
+            <features value="Backend"/>
+            <stories value="Menu Navigation"/>
+            <title value="Admin should be able to navigate between menu options with secret url keys enabled"/>
+            <description value="Admin should be able to navigate between menu options with secret url keys enabled"/>
+            <severity value="CRITICAL"/>
+            <testCaseId value="MAGETWO-95349"/>
+            <group value="menu"/>
+        </annotations>
+        <before>
+            <magentoCLI command="config:set admin/security/use_form_key 1" stepKey="enableUrlSecretKeys"/>
+            <magentoCLI command="cache:clean config full_page" stepKey="cleanInvalidatedCaches1"/>
+            <actionGroup ref="LoginAsAdmin" stepKey="loginAsAdmin"/>
+        </before>
+        <after>
+            <magentoCLI command="config:set admin/security/use_form_key 0" stepKey="disableUrlSecretKeys"/>
+            <magentoCLI command="cache:clean config full_page" stepKey="cleanInvalidatedCaches2"/>
+            <actionGroup ref="logout" stepKey="logout"/>
+        </after>
+
+        <click selector="{{AdminMenuSection.stores}}" stepKey="clickStoresMenuOption1"/>
+        <waitForLoadingMaskToDisappear stepKey="waitForStoresMenu1" />
+        <click selector="{{AdminMenuSection.configuration}}" stepKey="clickStoresConfigurationMenuOption1"/>
+        <waitForPageLoad stepKey="waitForConfigurationPageLoad1"/>
+        <seeCurrentUrlMatches regex="~\/admin\/system_config\/~" stepKey="seeCurrentUrlMatchesConfigPath1"/>
+
+        <click selector="{{AdminMenuSection.catalog}}" stepKey="clickCatalogMenuOption"/>
+        <waitForLoadingMaskToDisappear stepKey="waitForCatalogMenu1" />
+        <click selector="{{AdminMenuSection.catalogProducts}}" stepKey="clickCatalogProductsMenuOption"/>
+        <waitForPageLoad stepKey="waitForProductsPageLoad"/>
+        <seeCurrentUrlMatches regex="~\/catalog\/product\/~" stepKey="seeCurrentUrlMatchesProductsPath"/>
+
+        <click selector="{{AdminMenuSection.stores}}" stepKey="clickStoresMenuOption2"/>
+        <waitForLoadingMaskToDisappear stepKey="waitForStoresMenu2" />
+        <click selector="{{AdminMenuSection.configuration}}" stepKey="clickStoresConfigurationMenuOption2"/>
+        <waitForPageLoad stepKey="waitForConfigurationPageLoad2"/>
+        <seeCurrentUrlMatches regex="~\/admin\/system_config\/~" stepKey="seeCurrentUrlMatchesConfigPath2"/>
+    </test>
+</tests>