Ignore non-content commands in AllCommandsHavePermissions (#39336)

Causing a test failure every time a Toolshed command gets added to engine is ridiculous.
This commit is contained in:
Pieter-Jan Briers
2025-09-10 22:08:16 +02:00
committed by GitHub
parent 3896c5be8e
commit fb35b52da5

View File

@@ -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()}");
}
});