using Admin.NET.Plugin.AiDOP.Controllers.S8;
using Admin.NET.Plugin.AiDOP.Infrastructure;
using Admin.NET.Plugin.AiDOP.Service.S8;
using System.Reflection;
using Xunit;
namespace Admin.NET.Plugin.AiDOP.Tests.S8;
///
/// S8-P0-3-MASTERDATA-TRUSTED-SCOPE-1:S8 主数据 / 操作员绑定的跨租户越权守卫。
///
/// 背景(2026-09-02 以普通租户账号 UATExceptionA 实测,HTTP 200):
///
/// - GET /api/aidop/s8/config/operator-bindings(不带任何参数)→ 500 行,
/// 按工厂分布 {1000: 498, 1329900200002: 2},**全部属于其它租户,本租户 0 行**;
/// - GET /api/aidop/s8/config/operator-bindings?factoryRefId=1000 → 500 条外租户员工;
/// - GET /api/aidop/s8/master-data/employees?factoryRefId=1000 → 500 条外租户员工工号 + 真实姓名。
///
///
/// 根因:两个控制器都未接入 (KNOWN-ISSUES I-001 的 6 个之二),
/// 服务层用 ClearFilter() + **客户端传入的** factoryRefId 作边界,
/// 且 ListAsync 的该谓词还是 WhereIF(factoryRefId.HasValue, …) —— 可选,不传即无边界。
///
/// 本测试锁住修复后的口径:作用域只能来自服务端认证身份;客户端 factoryRefId / tenantId
/// 不得出现在任何 Action 或 Service 签名上,杜绝「以后有人又把它接回去」。
///
public class S8MasterDataScopeGuardTests
{
private static readonly string PluginRoot =
Path.GetFullPath(Path.Combine(AppContext.BaseDirectory, "../../../../Admin.NET.Plugin.AiDOP"));
private const string MasterDataSrc = "Service/S8/S8MasterDataAdapter.cs";
private static string ReadSource(string relativePath)
{
var full = Path.Combine(PluginRoot, relativePath);
Assert.True(File.Exists(full), $"源码文件不存在,路径需同步更新:{full}");
return File.ReadAllText(full);
}
/// 只取可执行代码行:注释里为留档会复述旧写法,不应算违规。
private static List CodeLines(string relativePath) =>
ReadSource(relativePath)
.Split('\n')
.Select(l => l.Trim())
.Where(l => !l.StartsWith("///", StringComparison.Ordinal)
&& !l.StartsWith("//", StringComparison.Ordinal))
.ToList();
private static IEnumerable ActionsOf(Type controller) =>
controller.GetMethods(BindingFlags.Instance | BindingFlags.Public | BindingFlags.DeclaredOnly);
// ───────────────────────── 控制器层 ─────────────────────────
///
/// 仍在服务的控制器必须从可信作用域取租户。
///
/// AdoS8ConfigBindingsController 已整体退役(全部 Action 返回 410),
/// 不再注入任何服务、也不再解析作用域 —— 一个不查库、不写库的墓碑入口
/// 要作用域解析器没有意义。它的守卫改由 承担。
///
[Theory]
[InlineData(typeof(AdoS8MasterDataController))]
public void Controller_InjectsTrustedScopeResolver(Type controller)
{
var ctor = controller.GetConstructors().Single();
Assert.Contains(ctor.GetParameters(), p => p.ParameterType == typeof(S8TrustedScopeResolver));
}
/// 客户端不得再通过 query / route 指定作用域:这两个名字必须从 Action 签名彻底消失。
[Theory]
[InlineData(typeof(AdoS8ConfigBindingsController))]
[InlineData(typeof(AdoS8MasterDataController))]
public void ControllerActions_DoNotAcceptClientSuppliedScope(Type controller)
{
foreach (var action in ActionsOf(controller))
{
foreach (var p in action.GetParameters())
{
Assert.False(
string.Equals(p.Name, "factoryRefId", StringComparison.OrdinalIgnoreCase)
|| string.Equals(p.Name, "factoryId", StringComparison.OrdinalIgnoreCase)
|| string.Equals(p.Name, "tenantId", StringComparison.OrdinalIgnoreCase),
$"{controller.Name}.{action.Name} 仍接受客户端作用域参数 {p.Name}");
}
}
}
// ───────────────────────── 服务层签名 ─────────────────────────
/// 五个入口必须以 S8TrustedScope 作首参,且不得再出现 factoryRefId 形参。
[Theory]
[InlineData(typeof(S8MasterDataAdapter), "GetDepartmentsAsync")]
// S8-SYSUSER-ONLY-1:GetEmployeesAsync → GetOperatorUsersAsync(员工下拉换成账号下拉);
// S8OperatorBindingService 四个入口随该服务一并退役,见 OperatorBinding_IsRetired。
[InlineData(typeof(S8MasterDataAdapter), "GetOperatorUsersAsync")]
[InlineData(typeof(S8MasterDataAdapter), "GetLinesAsync")]
public void ServiceEntryPoints_TakeTrustedScopeAndRejectClientFactory(Type service, string method)
{
var m = service.GetMethod(method, BindingFlags.Instance | BindingFlags.Public);
Assert.NotNull(m);
var ps = m!.GetParameters();
Assert.NotEmpty(ps);
Assert.Equal(typeof(S8TrustedScope), ps[0].ParameterType);
Assert.DoesNotContain(ps, p =>
string.Equals(p.Name, "factoryRefId", StringComparison.OrdinalIgnoreCase)
|| string.Equals(p.Name, "factoryId", StringComparison.OrdinalIgnoreCase)
|| string.Equals(p.Name, "tenantId", StringComparison.OrdinalIgnoreCase));
}
// ───────────────────────── 服务层查询谓词 ─────────────────────────
///
/// S8-SYSUSER-ONLY-1:绑定服务整体退役,原先三条「ClearFilter 必须收口」「SysUser 查询必须带租户」
/// 「绑定/解绑必须按作用域绑行」的断言合并为一条 —— 这个写 EmployeeMaster 的写路径不得复活。
///
/// 那三条守的是同一件事:一个能把任意员工绑成本租户处理人的写入口,必须有租户边界。
/// 现在这个入口本身不存在了,守卫的对象也就从「边界写对没有」变成「入口有没有被引回来」。
///
[Fact]
public void OperatorBinding_IsRetired()
{
Assert.False(File.Exists(Path.Combine(PluginRoot, "Service", "S8", "S8OperatorBindingService.cs")),
"S8OperatorBindingService 已退役,不得复活");
var src = string.Join('\n', CodeLines("Controllers/S8/AdoS8ConfigBindingsController.cs"));
Assert.Contains("Status410Gone", src);
// 退役后的入口不应该还保有写主数据的能力。
Assert.DoesNotContain("EmployeeMaster", src);
Assert.DoesNotContain("_empRep", src);
}
/// 主数据三个入口的边界必须来自 scope 的租户,不得来自入参、更不得退回工厂。
[Fact]
public void MasterData_ScopeComesFromTrustedScopeOnly()
{
var src = string.Join('\n', CodeLines(MasterDataSrc));
Assert.Contains("x.TenantId == scope.TenantId", src);
Assert.Contains("u.TenantId == scope.TenantId", src);
Assert.DoesNotContain("x.FactoryRefId == scope.FactoryId", src);
// 客户端参数写法必须消失。
Assert.DoesNotContain("factoryRefId!.Value", src);
Assert.DoesNotContain("factoryRefId.HasValue", src);
}
}