feat(test): 完成ContextAwareStateMachine的单元测试覆盖

- 创建了ContextAwareStateMachineTests.cs,包含12个测试用例
- 覆盖了状态机的集成测试、上下文感知功能、状态变更事件等核心功能
- 测试了Init方法的上下文初始化、状态变更事件发送、多状态注册等功能
- 更新测试覆盖率从~45%提升至~47%,总测试数从286个增加到298个
- 完成第一批所有5个核心
This commit is contained in:
GeWuYou 2026-01-16 12:35:50 +08:00
parent 804ccee329
commit 70e16724c9
2 changed files with 262 additions and 11 deletions

View File

@ -2,7 +2,7 @@
> **生成日期**: 2026-01-16
> **最后更新**: 2026-01-16
> **当前版本**: Core测试覆盖率 ~45%
> **当前版本**: Core测试覆盖率 ~47%
> **目标**: 提升Core模块测试覆盖率至 85%+
---
@ -247,8 +247,6 @@
**源文件路径**: `GFramework.Core/state/ContextAwareStateMachine.cs`
**注意**: 此文件已创建但被删除,需要重新创建
**需要测试的内容**:
- ✅ 作为 ISystem 的集成测试
- ✅ Init 方法 - 初始化上下文感知状态
@ -266,11 +264,13 @@
**预计测试数**: 12-15 个
**实际测试数**: 12 个
**优先级**: 🔴 高
**创建路径**: `GFramework.Core.Tests/state/ContextAwareStateMachineTests.cs`
**状态**: ⚠️ 需要重新创建
**状态**: ✅ 已创建
---
@ -762,10 +762,10 @@
## 🎯 目标达成路径
### 当前状态
- **现有测试数**: 286
- **测试覆盖率**: ~45%
- **缺失测试**: 122-178
- **已完成文件**: 4/26 (ArchitectureConfigurationTests.cs, ArchitectureContextTests.cs, ArchitectureServicesTests.cs, ArchitectureEventsTests.cs)
- **现有测试数**: 298 个
- **测试覆盖率**: ~47%
- **缺失测试**: 110-166
- **已完成文件**: 5/26 (第一批全部完成)
### 第一批完成 1/5
- **当前测试数**: 240 个
@ -783,9 +783,14 @@
- **当前测试数**: 286 个
- **当前覆盖率**: ~45%
### 第一批完成后
- **预计测试数**: 228 + 53-67 = 281-295 个
- **预计覆盖率**: ~50-55%
### 第一批完成 5/5 ✅
- **当前测试数**: 298 个
- **当前覆盖率**: ~47%
- **第一批总计**: 70 个测试(预计 53-67 个)
### 第二批预计完成后
- **预计测试数**: 298 + 46-57 = 344-355 个
- **预计覆盖率**: ~55-60%
### 第二批完成后
- **预计测试数**: 281-295 + 46-57 = 327-352 个
@ -840,6 +845,7 @@
| 2026-01-16 | 完成 ArchitectureContextTests.cs | 创建了22个测试用例涵盖构造函数、命令/查询/事件发送、组件获取等功能 |
| 2026-01-16 | 完成 ArchitectureServicesTests.cs | 创建了15个测试用例涵盖服务初始化、上下文管理、服务独立性等功能 |
| 2026-01-16 | 完成 ArchitectureEventsTests.cs | 创建了9个测试用例涵盖生命周期事件、事件订阅、事件顺序等功能 |
| 2026-01-16 | 完成 ContextAwareStateMachineTests.cs | 创建了12个测试用例涵盖状态机集成、上下文感知、状态变更事件等功能 |
| | | |
---

View File

@ -0,0 +1,245 @@
using GFramework.Core.Abstractions.architecture;
using GFramework.Core.Abstractions.enums;
using GFramework.Core.Abstractions.state;
using GFramework.Core.Abstractions.system;
using GFramework.Core.architecture;
using GFramework.Core.command;
using GFramework.Core.environment;
using GFramework.Core.events;
using GFramework.Core.ioc;
using GFramework.Core.query;
using GFramework.Core.state;
using NUnit.Framework;
namespace GFramework.Core.Tests.state;
[TestFixture]
public class ContextAwareStateMachineTests
{
private TestContextAwareStateMachineV5? _stateMachine;
private ArchitectureContext? _context;
private EventBus? _eventBus;
[SetUp]
public void SetUp()
{
_eventBus = new EventBus();
_context = new ArchitectureContext(
new IocContainer(),
_eventBus,
new CommandBus(),
new QueryBus(),
new DefaultEnvironment());
_stateMachine = new TestContextAwareStateMachineV5();
_stateMachine.SetContext(_context);
}
[Test]
public void ContextAwareStateMachine_Should_Implement_ISystem_Interface()
{
Assert.That(_stateMachine, Is.InstanceOf<ISystem>());
}
[Test]
public void SetContext_Should_Set_Context_Property()
{
_stateMachine!.SetContext(_context!);
var context = _stateMachine.GetContext();
Assert.That(context, Is.SameAs(_context));
}
[Test]
public void GetContext_Should_Return_Context_Property()
{
_stateMachine!.SetContext(_context!);
var context = _stateMachine.GetContext();
Assert.That(context, Is.Not.Null);
Assert.That(context, Is.SameAs(_context));
}
[Test]
public void Init_Should_SetContext_On_All_ContextAware_States()
{
var state1 = new TestContextAwareStateV5();
var state2 = new TestContextAwareStateV5_2();
_stateMachine!.Register(state1);
_stateMachine.Register(state2);
Assert.That(state1.GetContext(), Is.Null);
Assert.That(state2.GetContext(), Is.Null);
_stateMachine.Init();
Assert.That(state1.GetContext(), Is.SameAs(_context));
Assert.That(state2.GetContext(), Is.SameAs(_context));
}
[Test]
public void Init_Should_Not_SetContext_On_NonContextAware_States()
{
var state = new TestStateV5();
_stateMachine!.Register(state);
_stateMachine.Init();
}
[Test]
public void Destroy_Should_Not_Throw_Exception()
{
Assert.That(() => _stateMachine!.Destroy(), Throws.Nothing);
}
[Test]
public void OnArchitecturePhase_Should_Not_Throw_Exception()
{
Assert.That(() => _stateMachine!.OnArchitecturePhase(ArchitecturePhase.Ready),
Throws.Nothing);
}
[Test]
public void ChangeTo_Should_Send_StateChangedEvent()
{
bool eventReceived = false;
StateChangedEvent? receivedEvent = null;
_eventBus!.Register<StateChangedEvent>(e =>
{
eventReceived = true;
receivedEvent = e;
});
var state1 = new TestStateV5();
var state2 = new TestStateV5();
_stateMachine!.Register(state1);
_stateMachine.Register(state2);
_stateMachine.Init();
_stateMachine.ChangeTo<TestStateV5>();
Assert.That(eventReceived, Is.True);
Assert.That(receivedEvent!.OldState, Is.Null);
Assert.That(receivedEvent.NewState, Is.InstanceOf<TestStateV5>());
}
[Test]
public void ChangeTo_Should_Send_StateChangedEvent_With_OldState()
{
bool eventReceived = false;
StateChangedEvent? receivedEvent = null;
_eventBus!.Register<StateChangedEvent>(e =>
{
eventReceived = true;
receivedEvent = e;
});
var state1 = new TestStateV5();
var state2 = new TestStateV5();
_stateMachine!.Register(state1);
_stateMachine.Register(state2);
_stateMachine.Init();
_stateMachine.ChangeTo<TestStateV5>();
_stateMachine.ChangeTo<TestStateV5>();
Assert.That(eventReceived, Is.True);
Assert.That(receivedEvent!.OldState, Is.InstanceOf<TestStateV5>());
Assert.That(receivedEvent.NewState, Is.InstanceOf<TestStateV5>());
}
[Test]
public void CanChangeTo_Should_Work_With_Registered_States()
{
var state = new TestStateV5();
_stateMachine!.Register(state);
var canChange = _stateMachine.CanChangeTo<TestStateV5>();
Assert.That(canChange, Is.True);
}
[Test]
public void Multiple_States_Should_Be_Registered()
{
var state1 = new TestStateV5 { Id = 1 };
var state2 = new TestStateV5 { Id = 2 };
var state3 = new TestStateV5 { Id = 3 };
_stateMachine!.Register(state1);
_stateMachine.Register(state2);
_stateMachine.Register(state3);
var canChange = _stateMachine.CanChangeTo<TestStateV5>();
Assert.That(canChange, Is.True);
}
[Test]
public void StateMachine_Lifecycle_Should_Be_Complete()
{
var state1 = new TestStateV5();
var state2 = new TestStateV5();
_stateMachine!.Register(state1);
_stateMachine.Register(state2);
_stateMachine.Init();
Assert.That(_stateMachine.Current, Is.Null);
_stateMachine.ChangeTo<TestStateV5>();
Assert.That(_stateMachine.Current, Is.InstanceOf<TestStateV5>());
_stateMachine.Destroy();
}
}
#region Test Classes
public class TestContextAwareStateMachineV5 : ContextAwareStateMachine
{
public Dictionary<Type, IState> GetStates() => States;
}
public class TestContextAwareStateV5 : ContextAwareStateBase
{
public override void OnEnter(IState? previous)
{
}
public override void OnExit(IState? next)
{
}
}
public class TestContextAwareStateV5_2 : ContextAwareStateBase
{
public override void OnEnter(IState? previous)
{
}
public override void OnExit(IState? next)
{
}
}
public class TestStateV5 : IState
{
public int Id { get; set; }
public bool CanTransitionTo(IState next) => true;
public void OnEnter(IState? previous)
{
}
public void OnExit(IState? next)
{
}
}
#endregion