From fb35b52da5384ea98c2952aa0fe83d6e8fd9c13e Mon Sep 17 00:00:00 2001
From: Pieter-Jan Briers <pieterjan.briers+git@gmail.com>
Date: Wed, 10 Sep 2025 22:08:16 +0200
Subject: [PATCH] Ignore non-content commands in AllCommandsHavePermissions
 (#39336)

Causing a test failure every time a Toolshed command gets added to engine is ridiculous.
---
 Content.IntegrationTests/Tests/Toolshed/AdminTest.cs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Content.IntegrationTests/Tests/Toolshed/AdminTest.cs b/Content.IntegrationTests/Tests/Toolshed/AdminTest.cs
index ecb11fc1ba..ca70120ee9 100644
--- a/Content.IntegrationTests/Tests/Toolshed/AdminTest.cs
+++ b/Content.IntegrationTests/Tests/Toolshed/AdminTest.cs
@@ -25,6 +25,11 @@ public sealed class AdminTest : ToolshedTest
                     if (ignored.Contains(cmd.Cmd.GetType().Assembly))
                         continue;
 
+                    // Only care about content commands.
+                    var assemblyName = cmd.Cmd.GetType().Assembly.FullName;
+                    if (assemblyName == null || !assemblyName.StartsWith("Content."))
+                        continue;
+
                     Assert.That(admin.TryGetCommandFlags(cmd, out _), $"Command does not have admin permissions set up: {cmd.FullName()}");
                 }
             });