S7FqcGenericApprovalGuardContractTests.cs 12 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264
  1. using Xunit;
  2. namespace Admin.NET.Plugin.AiDOP.Tests.ApprovalFlow;
  3. /// <summary>
  4. /// S7 成品检验单(<c>S7_FQC_INSPBILL</c>,源 qms_qcpp_inspbill)通用审批旁路封堵 · 源码契约测试(S7-FQC-GUARD-1)。
  5. ///
  6. /// 背景:守卫名单此前只有 S5_IQC_INSPBILL / S6_PROCESS_INSPECTION / IPQC_INSPECTION,
  7. /// S7 一直在名单外(早前批次显式记为 OUT OF SCOPE)。
  8. ///
  9. /// 为什么 S7 的绕过面是**全部**业务语义:本链的 <see cref="FqcInspBillFlowBizHandler"/> 三个回调
  10. /// (OnFlowStarted / OnNodeCompleted / OnFlowCompleted)**全是 no-op**,
  11. /// 业务写入一律由 <c>FqcInspBillFlowService</c> 在引擎之外、同事务内完成。于是通用审批:
  12. /// ① N1 approve → submit-result 从未写 pd/hgsl/bhgsl/clfs 就推到 N2;更严重的是 GW_RESULT
  13. /// 网关按 pd 算 disposition_required,pd 为 null 时算 0 → 不合格单静默绕过 N3 QE 处置直接结束;
  14. /// ② N2 approve → 跳过 TryEnqueueFqcPassOutboxAsync,合格出站 Outbox 永不入队;
  15. /// ③ N2 reject → FlowEngine.Reject 把实例置 Rejected 终态并取消所有待办,而专用 supervisor-reject
  16. /// 是 ReturnToPrev 回 N1 + 同事务复位 qms_fqcbj.FINSPECTSTATUS + 同步 mdp_std_fqc_task;
  17. /// ④ N3 approve → 推到 end 而 ado_s7_fqc_qe_disposition 永不写入,处置轨迹为空且无法补记。
  18. ///
  19. /// 沿用本仓既有做法——源码契约断言而非行为级集成测试:FlowTaskService 依赖
  20. /// SqlSugarRepository&lt;T&gt;,裸 xUnit 进程无法实例化。真实阻断由本批 Runtime R1–R4 覆盖。
  21. /// </summary>
  22. public class S7FqcGenericApprovalGuardContractTests
  23. {
  24. private const string BizType = "S7_FQC_INSPBILL";
  25. private const string Guard = "EnsureDedicatedEntryOnlyAsync";
  26. private static string FlowTaskService() => File.ReadAllText(FindFile(
  27. "server", "Plugins", "Admin.NET.Plugin.ApprovalFlow", "Service", "FlowTask", "FlowTaskService.cs"));
  28. private static string FlowConst() => File.ReadAllText(FindFile(
  29. "server", "Plugins", "Admin.NET.Plugin.AiDOP", "FinishedWarehouse", "FqcInspBillFlowBizHandler.cs"));
  30. private static string FlowService() => File.ReadAllText(FindFile(
  31. "server", "Plugins", "Admin.NET.Plugin.AiDOP", "FinishedWarehouse", "FqcInspBillFlowService.cs"));
  32. private static string Dict() => Slice(FlowTaskService(), "DedicatedEntryOnlyBizTypes = new()", "};");
  33. // ── 纳入守卫 ──
  34. /// <summary>S7 真实 BizType 必须在名单里,且与常量定义一致(不允许两处写不同字面量)。</summary>
  35. [Fact]
  36. public void S7BizType_IsInDedicatedEntryGuard()
  37. {
  38. Assert.Contains($"[\"{BizType}\"]", Dict());
  39. Assert.Contains($"public const string BizType = \"{BizType}\"", FlowConst());
  40. }
  41. /// <summary>§15 Regression Freeze:新增 S7 不得挤掉既有三条。</summary>
  42. [Theory]
  43. [InlineData("S5_IQC_INSPBILL")]
  44. [InlineData("S6_PROCESS_INSPECTION")]
  45. [InlineData("IPQC_INSPECTION")]
  46. public void ExistingGuardedBizTypes_ArePreserved(string bizType)
  47. {
  48. Assert.Contains($"[\"{bizType}\"]", Dict());
  49. }
  50. /// <summary>名单恰好四条 —— 防止本批顺手扩大守卫范围。</summary>
  51. [Fact]
  52. public void Guard_CoversExactlyFourBizTypes()
  53. {
  54. Assert.Equal(4, CountOf(Dict(), "\"] = \""));
  55. }
  56. // ── 四个通用入口全覆盖(§14)──
  57. /// <summary>approve / reject / batchApprove / batchReject 四个入口都必须先过同一守卫。</summary>
  58. [Theory]
  59. [InlineData("_engine.Approve(input.TaskId, input.Comment)")]
  60. [InlineData("_engine.Reject(input.TaskId, input.Comment)")]
  61. public void SingleActions_GuardPrecedesEngineCall(string engineCall)
  62. {
  63. var s = FlowTaskService();
  64. var at = s.IndexOf(engineCall, StringComparison.Ordinal);
  65. Assert.True(at > 0, $"未找到引擎调用点:{engineCall}");
  66. Assert.True(s.LastIndexOf(Guard + "(", at, StringComparison.Ordinal) > 0,
  67. $"引擎调用 {engineCall} 之前缺少 {Guard} 守卫");
  68. }
  69. /// <summary>
  70. /// 批量入口的循环体各自持有守卫。批量端点不会走单项方法体,
  71. /// 少一处就等于批量仍可绕过(§21 / §22)。
  72. /// </summary>
  73. [Fact]
  74. public void BatchActions_EachLoopBodyCallsGuard()
  75. {
  76. var s = FlowTaskService();
  77. Assert.Equal(4, CountOf(s, Guard + "(taskId)") + CountOf(s, Guard + "(input.TaskId)"));
  78. foreach (var batch in new[] { "BatchApprove", "BatchReject" })
  79. {
  80. var body = Slice(s, $"public async Task<BatchActionResult> {batch}(", "return result;");
  81. Assert.Contains(Guard + "(taskId)", body);
  82. }
  83. }
  84. // ── §11 Guard 必须早于任何 mutation ──
  85. /// <summary>守卫是动作方法的第一条语句,因此 fail 时任务状态/实例/日志/下一节点都还没动。</summary>
  86. [Theory]
  87. [InlineData("public async Task Approve(TaskActionInput input)")]
  88. [InlineData("public async Task Reject(TaskActionInput input)")]
  89. public void Guard_IsFirstStatementOfAction(string signature)
  90. {
  91. var body = Slice(FlowTaskService(), signature, "}");
  92. var lines = body.Split('\n').Select(l => l.Trim()).Where(l => l != "" && l != "{" && !l.StartsWith("//", StringComparison.Ordinal)).ToList();
  93. var first = lines.FirstOrDefault(l => !l.StartsWith("public ", StringComparison.Ordinal));
  94. Assert.Equal($"await {Guard}(input.TaskId);", first);
  95. }
  96. /// <summary>守卫自身只读不写:不得在判定过程里改任何实体。</summary>
  97. [Fact]
  98. public void Guard_PerformsNoMutation()
  99. {
  100. var guard = Slice(FlowTaskService(), $"private async Task {Guard}(long taskId)", "}\n\n /// <summary>");
  101. foreach (var write in new[] { "AsUpdateable", "InsertAsync", "DeleteAsync", "ExecuteCommandAsync", "UseTranAsync" })
  102. Assert.DoesNotContain(write, guard);
  103. }
  104. // ── §10 精确匹配 ──
  105. /// <summary>必须是精确字典查表,禁止 Contains / StartsWith 之类模糊匹配。</summary>
  106. [Fact]
  107. public void Guard_UsesExactDictionaryLookup()
  108. {
  109. var guard = Slice(FlowTaskService(), $"private async Task {Guard}(long taskId)", "}\n\n /// <summary>");
  110. Assert.Contains("DedicatedEntryOnlyBizTypes.TryGetValue(instance.BizType, out var hint)", guard);
  111. foreach (var fuzzy in new[] { ".Contains(", ".StartsWith(", ".EndsWith(", "ToUpper", "ToLower" })
  112. Assert.DoesNotContain(fuzzy, guard);
  113. }
  114. /// <summary>§16 Control:明确允许通用审批的 BizType 不得被纳入。</summary>
  115. [Theory]
  116. [InlineData("CONTRACT_REVIEW")]
  117. [InlineData("ORDER_REVIEW")]
  118. [InlineData("SMART_OPS_IMPROVEMENT")]
  119. [InlineData("EXCEPTION_REPORT")]
  120. [InlineData("MATERIAL_SHORTAGE")]
  121. public void Guard_DoesNotBlockGenericApprovalBizTypes(string bizType)
  122. {
  123. Assert.DoesNotContain(bizType, Dict());
  124. }
  125. /// <summary>§9 禁止散落 if:不得在四个动作里各写一遍 S7 的判断。</summary>
  126. [Fact]
  127. public void NoScatteredPerBizTypeIfStatements()
  128. {
  129. var s = FlowTaskService();
  130. // BizType 字面量只应出现在名单里(1 次),不应出现在任何 if 条件中
  131. Assert.Equal(1, CountOf(s, $"\"{BizType}\""));
  132. Assert.DoesNotContain($"== \"{BizType}\"", s);
  133. }
  134. // ── §12 权威来源与不泄漏 ──
  135. /// <summary>权威 BizType 必须从库中 instance 反查,不得取自请求体(否则可伪造绕过)。</summary>
  136. [Fact]
  137. public void Guard_ReadsBizTypeFromDatabase()
  138. {
  139. var guard = Slice(FlowTaskService(), $"private async Task {Guard}(long taskId)", "}\n\n /// <summary>");
  140. Assert.Contains("_taskRep.GetByIdAsync(taskId)", guard);
  141. Assert.Contains("_instanceRep.GetByIdAsync(task.InstanceId)", guard);
  142. Assert.DoesNotContain("input.BizType", guard);
  143. }
  144. /// <summary>§13 对外文案不得泄漏 RoleId / TenantId / 流程定义 Id 等内部实现。</summary>
  145. [Fact]
  146. public void GuardMessage_DoesNotLeakInternals()
  147. {
  148. var dict = Dict();
  149. foreach (var leak in new[] { "TenantId", "RoleId", "FlowId", "DefinitionId", "InstanceId" })
  150. Assert.DoesNotContain($"{leak}=", dict);
  151. }
  152. /// <summary>文案必须指向专用业务页,且不是裸技术异常。</summary>
  153. [Fact]
  154. public void GuardMessage_PointsToDedicatedPage()
  155. {
  156. var line = Dict().Split('\n').Single(l => l.Contains($"[\"{BizType}\"]", StringComparison.Ordinal));
  157. Assert.Contains("FQC检验单详情", line);
  158. Assert.Contains("不支持在审批中心直接同意/拒绝", line);
  159. foreach (var bad in new[] { "BizType not allowed", "NullReference", "500" })
  160. Assert.DoesNotContain(bad, line);
  161. }
  162. // ── 封堵的前提:专用入口必须仍然存在且直调引擎 ──
  163. /// <summary>四个专用入口都必须在(否则封堵等于封死业务)。</summary>
  164. [Theory]
  165. [InlineData("submit-result")]
  166. [InlineData("supervisor-approve")]
  167. [InlineData("supervisor-reject")]
  168. [InlineData("qe-submit-disposition")]
  169. public void DedicatedEntries_StillExist(string route)
  170. {
  171. Assert.Contains($"[HttpPost(\"{route}\")]", FlowService());
  172. }
  173. /// <summary>专用入口直调 FlowEngine,不经 FlowTaskService —— 故守卫不会误伤正常业务路径。</summary>
  174. [Fact]
  175. public void DedicatedEntries_CallEngineDirectly_NotViaFlowTaskService()
  176. {
  177. var s = FlowService();
  178. Assert.Contains("_flowEngine.Approve(", s);
  179. Assert.Contains("_flowEngine.ReturnToPrev(", s);
  180. Assert.DoesNotContain("FlowTaskService", s);
  181. Assert.DoesNotContain("flowTask/approve", s);
  182. }
  183. /// <summary>
  184. /// 冻结"为什么必须封堵"的两条根据:
  185. /// ① Handler 回调全 no-op(业务副作用不在引擎里);② 专用 reject 走 ReturnToPrev 而非置终态。
  186. /// 任一条将来被改动,本守卫的理由需要重新评估,测试在此提醒。
  187. /// </summary>
  188. [Fact]
  189. public void BypassRationale_HandlerCallbacksAreNoOp()
  190. {
  191. var h = FlowConst();
  192. foreach (var cb in new[] { "OnFlowStarted", "OnNodeCompleted", "OnFlowCompleted" })
  193. {
  194. var line = h.Split('\n').Single(l => l.Contains($"public Task {cb}(", StringComparison.Ordinal));
  195. Assert.Contains("Task.CompletedTask", line);
  196. }
  197. }
  198. /// <summary>专用 supervisor-reject 的业务副作用必须仍在(它们正是通用 Reject 绕过的东西)。</summary>
  199. [Fact]
  200. public void BypassRationale_DedicatedRejectHasBusinessSideEffects()
  201. {
  202. var body = Slice(FlowService(), "public async Task<FqcFlowStateOutput> SupervisorReject(", "public async Task<FqcFlowStateOutput> QeSubmitDisposition(");
  203. Assert.Contains("退回必须填写意见", body);
  204. Assert.Contains("_flowEngine.ReturnToPrev(", body);
  205. Assert.Contains("UPDATE qms_fqcbj SET FINSPECTSTATUS", body);
  206. Assert.Contains("SyncBillToStdSafeAsync", body);
  207. }
  208. private static int CountOf(string src, string needle)
  209. {
  210. int n = 0, i = 0;
  211. while ((i = src.IndexOf(needle, i, StringComparison.Ordinal)) >= 0) { n++; i += needle.Length; }
  212. return n;
  213. }
  214. private static string Slice(string src, string from, string to)
  215. {
  216. var a = src.IndexOf(from, StringComparison.Ordinal);
  217. Assert.True(a >= 0, $"未找到起点:{from}");
  218. var b = src.IndexOf(to, a + from.Length, StringComparison.Ordinal);
  219. return b > a ? src[a..b] : src[a..];
  220. }
  221. private static string FindFile(params string[] parts)
  222. {
  223. var dir = new DirectoryInfo(AppContext.BaseDirectory);
  224. while (dir != null)
  225. {
  226. var candidate = Path.Combine(new[] { dir.FullName }.Concat(parts).ToArray());
  227. if (File.Exists(candidate)) return candidate;
  228. dir = dir.Parent;
  229. }
  230. throw new FileNotFoundException(string.Join("/", parts));
  231. }
  232. }