Merge remote-tracking branch '38078/fix-for-issue-37812' into comm_prs_247beta3_dec

Author: Indraniks

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

@@ -18,7 +18,7 @@
  */
 class Menu extends \Magento\Backend\Block\Template
 {
-    const CACHE_TAGS = 'BACKEND_MAINMENU';
+    public const CACHE_TAGS = 'BACKEND_MAINMENU';
 
     /**
      * @var string
@@ -347,6 +347,11 @@ protected function _columnBrake($items, $limit)
             }
             $result[] = ['place' => $place, 'colbrake' => $colbrake];
         }
+
+        if (isset($result[1]) && $result[1]['colbrake'] === true && isset($result[2])) {
+            $result[2]['colbrake'] = true;
+        }
+
         return $result;
     }
 

dev/tests/integration/testsuite/Magento/Backend/Block/MenuTest.php

@@ -59,7 +59,7 @@ protected function setUp(): void
      */
     public function testRenderNavigation()
     {
-        $menuHtml = $this->blockMenu->renderNavigation($this->menuConfig->getMenu());
+        $menuHtml = $this->blockMenu->renderNavigation($this->menuConfig->getMenu(), 0, 12);
         $menu = new \SimpleXMLElement($menuHtml);
 
         $item = $menu->xpath('/ul/li/a/span')[0];
@@ -74,12 +74,17 @@ public function testRenderNavigation()
             'Invited Customers',
         ];
         foreach ($menu->xpath('/ul//ul//ul/li/a/span') as $sortOrder => $item) {
+            if ($sortOrder>2) {
+                break;
+            }
             $this->assertEquals(
                 $liTitles[$sortOrder],
                 (string)$item,
                 '"' . $liTitles[$sortOrder] . '" item is absent or located on wrong menu level.'
             );
         }
+        // test column break if submenu contain more than 12 node
+        $this->assertStringContainsString('<li class="column">', $menuHtml);
     }
 
     /**

dev/tests/integration/testsuite/Magento/Backend/Block/_files/menu/Magento/Backend/etc/adminhtml/menu.xml

@@ -13,5 +13,41 @@
         <add id="Magento_Backend::system_report_magento_invite_general" title="Invite" module="Magento_Backend" sortOrder="20" parent="Magento_Backend::system_report" resource="Magento_Backend::report_magento_invite_general"/>
         <add id="Magento_Backend::system_report_private_sales" title="Private Sales" module="Magento_Backend" sortOrder="10" parent="Magento_Backend::system_report" resource="Magento_Backend::report_private_sales" />
         <add id="Magento_Backend::system_report_magento_invite_customer" title="Invited Customers" module="Magento_Backend" sortOrder="30" parent="Magento_Backend::system_report" resource="Magento_Backend::report_magento_invite_customer"/>
+        <add id="Magento_Cms::test1" title="Test 1"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test2" title="Test 2"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test3" title="Test 3"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test4" title="Test 4"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test5" title="Test 5"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test6" title="Test 6"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test7" title="Test 7"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test8" title="Test 8"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
+        <add id="Magento_Cms::test9" title="Test 9"
+             translate="title" module="Magento_Cms" sortOrder="100"
+             action="test/test/test"
+             parent="Magento_Backend::system" resource="Magento_Cms::page"/>
     </menu>
 </config>