[FIX] Actions: ensure the sequence is applied on action children

Currently, the helper 'createActions' pre-sorts a list of menuRegistry
entries but this sort is not applied to the children (which can be
generated dynamically). It never showed up as we rarely add children
entries from different sources and their sequence often matches the
order of insertion in the registry.

Task: 5452669
X-original-commit: 7c8456d

Author: rrahir

src/actions/action.ts

@@ -113,7 +113,8 @@ export function createAction(item: ActionSpec): Action {
           return children
             .map((child) => (typeof child === "function" ? child(env) : child))
             .flat()
-            .map(createAction);
+            .map(createAction)
+            .sort((a, b) => a.sequence - b.sequence);
         }
       : () => [],
     isReadonlyAllowed: item.isReadonlyAllowed || false,

tests/menus/menu_items_registry.test.ts

@@ -50,6 +50,7 @@ import {
 
 import { Currency, Model } from "../../src";
 
+import { createActions } from "../../src/actions/action";
 import { CellComposerStore } from "../../src/components/composer/composer/cell_composer_store";
 import { FONT_SIZES } from "../../src/constants";
 import { functionRegistry } from "../../src/functions";
@@ -116,6 +117,7 @@ describe("Top Bar Menu Item Registry", () => {
         id: name,
         name: name,
         execute: () => {},
+        sequence: 1,
       }));
     });
     const env = makeTestEnv();
@@ -1978,3 +1980,32 @@ describe("Menu Item actions", () => {
     });
   });
 });
+
+test("Menu children are sorted by sequence", async () => {
+  const env = makeTestEnv();
+  const menuItems = createActions([
+    {
+      id: "menu_1",
+      name: "Menu 1",
+      sequence: 20,
+      children: [
+        {
+          id: "secondItem",
+          name: "bigger sequence Item",
+          sequence: 30,
+          execute: () => {},
+        },
+        {
+          id: "firstItem",
+          name: "lower sequence Item",
+          sequence: 10,
+          execute: () => {},
+        },
+      ],
+    },
+  ]);
+
+  const children = menuItems[0].children(env);
+  expect(children[0].id).toBe("firstItem");
+  expect(children[1].id).toBe("secondItem");
+});