From 9e96df4c3b009276c8bdeaff6c70cae687a9d3db Mon Sep 17 00:00:00 2001 From: unanmed <1319491857@qq.com> Date: Tue, 15 Sep 2026 16:53:50 +0800 Subject: [PATCH] fix(07-01): #06-01-1 align critical info with the yielded next value - move targetInfo assignment into the damage < referenceDamage branch so the reported critical info matches the yielded next value - un-skip the correct-expectation case in damage.test.ts (#06-01-1) --- packages-user/data-system/src/combat/damage.test.ts | 4 ++-- packages-user/data-system/src/combat/damage.ts | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages-user/data-system/src/combat/damage.test.ts b/packages-user/data-system/src/combat/damage.test.ts index 09c5589..d2549fe 100644 --- a/packages-user/data-system/src/combat/damage.test.ts +++ b/packages-user/data-system/src/combat/damage.test.ts @@ -575,8 +575,8 @@ describe('DamageContext critical generation', () => { expect(locatorResult.info).toHaveLength(0); }); - // 疑似 bug:产出的 info 应与 nextValue 对应,详见 06-TEST-FINDINGS.md #06-01-1,修复后取消 skip - it.skip('reports the damage info matching the yielded critical value', () => { + // 验证产出的 info 与 nextValue 同源,即临界点伤害信息对应该属性值 + it('reports the damage info matching the yielded critical value', () => { const fixture = createFixture(); const context = new modules.DamageContext( fixture.context, diff --git a/packages-user/data-system/src/combat/damage.ts b/packages-user/data-system/src/combat/damage.ts index d1f5b18..24ac186 100644 --- a/packages-user/data-system/src/combat/damage.ts +++ b/packages-user/data-system/src/combat/damage.ts @@ -228,9 +228,9 @@ export class DamageContext implements IDamageContext< if (middleInfo.damage < referenceDamage) { right = middle; + targetInfo = middleInfo; } else { left = middle; - targetInfo = middleInfo; } if (right - left <= 1) break; }