test: make hierarchical assertions order-insensitive

Author: twlite

packages/commandkit/src/app/handlers/AppCommandHandler.hierarchical.test.ts

@@ -184,11 +184,20 @@ export async function message() {}
         handler.getCommandsArray().map((command) => command.command.name),
       ).toEqual(['ping']);
 
-      expect(
-        handler.getRuntimeCommandsArray().map((command) => {
+      const runtimeRouteKeys = handler
+        .getRuntimeCommandsArray()
+        .map((command) => {
           return (command.data.command as Record<string, any>).__routeKey;
-        }),
-      ).toEqual(['ping', 'admin.moderation.ban', 'admin.moderation.kick']);
+        });
+
+      expect(runtimeRouteKeys).toHaveLength(3);
+      expect(runtimeRouteKeys).toEqual(
+        expect.arrayContaining([
+          'ping',
+          'admin.moderation.ban',
+          'admin.moderation.kick',
+        ]),
+      );
 
       const preparedFlat = await handler.prepareCommandRun(
         createChatInputInteraction('ping'),

packages/commandkit/src/app/register/CommandRegistrar.hierarchical.test.ts

@@ -116,34 +116,56 @@ describe('Hierarchical command registration', () => {
       const admin = registrationCommands.find(
         (entry) => entry.name === 'admin',
       );
+      const moderationGroup = admin?.options?.[0] as
+        | {
+            name: string;
+            options?: Array<{
+              name: string;
+              description: string;
+              options: Array<{
+                name: string;
+                type: ApplicationCommandOptionType;
+              }>;
+              type: ApplicationCommandOptionType;
+            }>;
+            type: ApplicationCommandOptionType;
+          }
+        | undefined;
 
       expect(ping?.type).toBe(ApplicationCommandType.ChatInput);
       expect(admin?.type).toBe(ApplicationCommandType.ChatInput);
       expect(admin?.description).toBe('Admin');
-      expect(admin?.options).toEqual([
+      expect(moderationGroup?.name).toBe('moderation');
+      expect(moderationGroup?.type).toBe(
+        ApplicationCommandOptionType.SubcommandGroup,
+      );
+
+      const normalizedSubcommands = [...(moderationGroup?.options ?? [])]
+        .map((option) => ({
+          ...option,
+          options: [...(option.options ?? [])].sort((a, b) =>
+            a.name.localeCompare(b.name),
+          ),
+        }))
+        .sort((a, b) => a.name.localeCompare(b.name));
+
+      expect(normalizedSubcommands).toEqual([
         {
-          description: 'Moderation',
-          name: 'moderation',
+          description: 'Ban',
+          name: 'ban',
           options: [
             {
-              description: 'Ban',
-              name: 'ban',
-              options: [
-                {
-                  name: 'reason',
-                  type: ApplicationCommandOptionType.String,
-                },
-              ],
-              type: ApplicationCommandOptionType.Subcommand,
-            },
-            {
-              description: 'Kick',
-              name: 'kick',
-              options: [],
-              type: ApplicationCommandOptionType.Subcommand,
+              name: 'reason',
+              type: ApplicationCommandOptionType.String,
             },
           ],
-          type: ApplicationCommandOptionType.SubcommandGroup,
+          type: ApplicationCommandOptionType.Subcommand,
+        },
+        {
+          description: 'Kick',
+          name: 'kick',
+          options: [],
+          type: ApplicationCommandOptionType.Subcommand,
         },
       ]);