S8MasterDataScopeGuardTests.cs 8.3 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171
  1. using Admin.NET.Plugin.AiDOP.Controllers.S8;
  2. using Admin.NET.Plugin.AiDOP.Infrastructure;
  3. using Admin.NET.Plugin.AiDOP.Service.S8;
  4. using System.Reflection;
  5. using Xunit;
  6. namespace Admin.NET.Plugin.AiDOP.Tests.S8;
  7. /// <summary>
  8. /// S8-P0-3-MASTERDATA-TRUSTED-SCOPE-1:S8 主数据 / 操作员绑定的跨租户越权守卫。
  9. ///
  10. /// <para>背景(2026-09-02 以普通租户账号 UATExceptionA 实测,HTTP 200):</para>
  11. /// <list type="bullet">
  12. /// <item><c>GET /api/aidop/s8/config/operator-bindings</c>(不带任何参数)→ 500 行,
  13. /// 按工厂分布 {1000: 498, 1329900200002: 2},**全部属于其它租户,本租户 0 行**;</item>
  14. /// <item><c>GET /api/aidop/s8/config/operator-bindings?factoryRefId=1000</c> → 500 条外租户员工;</item>
  15. /// <item><c>GET /api/aidop/s8/master-data/employees?factoryRefId=1000</c> → 500 条外租户员工工号 + 真实姓名。</item>
  16. /// </list>
  17. ///
  18. /// <para>根因:两个控制器都未接入 <see cref="S8TrustedScopeResolver"/>(KNOWN-ISSUES I-001 的 6 个之二),
  19. /// 服务层用 <c>ClearFilter()</c> + **客户端传入的** <c>factoryRefId</c> 作边界,
  20. /// 且 <c>ListAsync</c> 的该谓词还是 <c>WhereIF(factoryRefId.HasValue, …)</c> —— 可选,不传即无边界。</para>
  21. ///
  22. /// <para>本测试锁住修复后的口径:作用域只能来自服务端认证身份;客户端 factoryRefId / tenantId
  23. /// 不得出现在任何 Action 或 Service 签名上,杜绝「以后有人又把它接回去」。</para>
  24. /// </summary>
  25. public class S8MasterDataScopeGuardTests
  26. {
  27. private static readonly string PluginRoot =
  28. Path.GetFullPath(Path.Combine(AppContext.BaseDirectory, "../../../../Admin.NET.Plugin.AiDOP"));
  29. private const string BindingSrc = "Service/S8/S8OperatorBindingService.cs";
  30. private const string MasterDataSrc = "Service/S8/S8MasterDataAdapter.cs";
  31. private static string ReadSource(string relativePath)
  32. {
  33. var full = Path.Combine(PluginRoot, relativePath);
  34. Assert.True(File.Exists(full), $"源码文件不存在,路径需同步更新:{full}");
  35. return File.ReadAllText(full);
  36. }
  37. /// <summary>只取可执行代码行:注释里为留档会复述旧写法,不应算违规。</summary>
  38. private static List<string> CodeLines(string relativePath) =>
  39. ReadSource(relativePath)
  40. .Split('\n')
  41. .Select(l => l.Trim())
  42. .Where(l => !l.StartsWith("///", StringComparison.Ordinal)
  43. && !l.StartsWith("//", StringComparison.Ordinal))
  44. .ToList();
  45. private static IEnumerable<MethodInfo> ActionsOf(Type controller) =>
  46. controller.GetMethods(BindingFlags.Instance | BindingFlags.Public | BindingFlags.DeclaredOnly);
  47. // ───────────────────────── 控制器层 ─────────────────────────
  48. [Theory]
  49. [InlineData(typeof(AdoS8ConfigBindingsController))]
  50. [InlineData(typeof(AdoS8MasterDataController))]
  51. public void Controller_InjectsTrustedScopeResolver(Type controller)
  52. {
  53. var ctor = controller.GetConstructors().Single();
  54. Assert.Contains(ctor.GetParameters(), p => p.ParameterType == typeof(S8TrustedScopeResolver));
  55. }
  56. /// <summary>客户端不得再通过 query / route 指定作用域:这两个名字必须从 Action 签名彻底消失。</summary>
  57. [Theory]
  58. [InlineData(typeof(AdoS8ConfigBindingsController))]
  59. [InlineData(typeof(AdoS8MasterDataController))]
  60. public void ControllerActions_DoNotAcceptClientSuppliedScope(Type controller)
  61. {
  62. foreach (var action in ActionsOf(controller))
  63. {
  64. foreach (var p in action.GetParameters())
  65. {
  66. Assert.False(
  67. string.Equals(p.Name, "factoryRefId", StringComparison.OrdinalIgnoreCase)
  68. || string.Equals(p.Name, "factoryId", StringComparison.OrdinalIgnoreCase)
  69. || string.Equals(p.Name, "tenantId", StringComparison.OrdinalIgnoreCase),
  70. $"{controller.Name}.{action.Name} 仍接受客户端作用域参数 {p.Name}");
  71. }
  72. }
  73. }
  74. // ───────────────────────── 服务层签名 ─────────────────────────
  75. /// <summary>五个入口必须以 S8TrustedScope 作首参,且不得再出现 factoryRefId 形参。</summary>
  76. [Theory]
  77. [InlineData(typeof(S8OperatorBindingService), "ListAsync")]
  78. [InlineData(typeof(S8OperatorBindingService), "ListSysUsersAsync")]
  79. [InlineData(typeof(S8OperatorBindingService), "BindAsync")]
  80. [InlineData(typeof(S8OperatorBindingService), "UnbindAsync")]
  81. [InlineData(typeof(S8MasterDataAdapter), "GetDepartmentsAsync")]
  82. [InlineData(typeof(S8MasterDataAdapter), "GetEmployeesAsync")]
  83. [InlineData(typeof(S8MasterDataAdapter), "GetLinesAsync")]
  84. public void ServiceEntryPoints_TakeTrustedScopeAndRejectClientFactory(Type service, string method)
  85. {
  86. var m = service.GetMethod(method, BindingFlags.Instance | BindingFlags.Public);
  87. Assert.NotNull(m);
  88. var ps = m!.GetParameters();
  89. Assert.NotEmpty(ps);
  90. Assert.Equal(typeof(S8TrustedScope), ps[0].ParameterType);
  91. Assert.DoesNotContain(ps, p =>
  92. string.Equals(p.Name, "factoryRefId", StringComparison.OrdinalIgnoreCase)
  93. || string.Equals(p.Name, "factoryId", StringComparison.OrdinalIgnoreCase)
  94. || string.Equals(p.Name, "tenantId", StringComparison.OrdinalIgnoreCase));
  95. }
  96. // ───────────────────────── 服务层查询谓词 ─────────────────────────
  97. /// <summary>
  98. /// 绑定服务里所有 ClearFilter 都必须收口到唯一的 ScopedEmployees 帮助方法,
  99. /// 且该方法带 FactoryRefId == scope.FactoryId 强制边界。
  100. /// 出现第二处裸 ClearFilter 即视为回退。
  101. /// </summary>
  102. [Fact]
  103. public void OperatorBinding_ClearFilterIsFunneledThroughScopedHelper()
  104. {
  105. var lines = CodeLines(BindingSrc);
  106. var clearFilterLines = lines.Where(l => l.Contains("ClearFilter()", StringComparison.Ordinal)).ToList();
  107. Assert.Single(clearFilterLines);
  108. var src = string.Join('\n', lines);
  109. Assert.Contains("_empRep.AsQueryable().ClearFilter()", src);
  110. Assert.Contains(".Where(x => x.FactoryRefId == scope.FactoryId)", src);
  111. // 旧的「可选工厂谓词」写法必须消失。
  112. Assert.DoesNotContain("WhereIF(factoryRefId.HasValue", src);
  113. Assert.DoesNotContain("factoryRefId!.Value", src);
  114. }
  115. /// <summary>SysUser 相关查询必须按可信租户,不得裸主键集合。</summary>
  116. [Fact]
  117. public void OperatorBinding_SysUserQueriesAreTenantBound()
  118. {
  119. var src = string.Join('\n', CodeLines(BindingSrc));
  120. Assert.Contains("u.TenantId == scope.TenantId", src);
  121. Assert.Contains("x.TenantId == scope.TenantId", src);
  122. }
  123. /// <summary>主数据三个入口的工厂边界必须来自 scope,不得来自入参。</summary>
  124. [Fact]
  125. public void MasterData_ScopeComesFromTrustedScopeOnly()
  126. {
  127. var src = string.Join('\n', CodeLines(MasterDataSrc));
  128. Assert.Contains("x.FactoryRefId == scope.FactoryId && x.IsActive", src);
  129. Assert.Contains("u.TenantId == scope.TenantId", src);
  130. // 客户端参数写法必须消失。
  131. Assert.DoesNotContain("factoryRefId!.Value", src);
  132. Assert.DoesNotContain("factoryRefId.HasValue", src);
  133. }
  134. /// <summary>
  135. /// 绑定 / 解绑必须先按可信作用域绑行再写。写路径上不得出现「只按主键取员工」的写法
  136. /// (旧实现 <c>_empRep.GetFirstAsync(x =&gt; x.Id == dto.EmployeeId)</c> 正是如此)。
  137. /// </summary>
  138. [Fact]
  139. public void BindAndUnbind_ResolveEmployeeThroughScopedLookup()
  140. {
  141. var src = string.Join('\n', CodeLines(BindingSrc));
  142. Assert.Contains("ScopedEmployee(scope, dto.EmployeeId)", src);
  143. Assert.Contains("ScopedEmployee(scope, employeeId)", src);
  144. Assert.DoesNotContain("_empRep.GetFirstAsync(x => x.Id == dto.EmployeeId)", src);
  145. Assert.DoesNotContain("_empRep.GetFirstAsync(x => x.Id == employeeId)", src);
  146. }
  147. }