Commit 2ea9238
test(ui,util): improve code coverage (#614)
* test(ui,util): improve code coverage for JEngine.UI and JEngine.Util
Add comprehensive tests to improve code coverage:
JEngine.Util:
- JActionAwaiterTests: tests for JActionAwaitable and JActionAwaiter
- JActionExecutionHandleTests: tests for JActionExecutionHandle,
JActionExecutionAwaiter, and JActionExecution structs
- JActionNestedExecutionTests: tests for sequential, parallel, and
cancellation scenarios
JEngine.UI:
- EditorUIRegistrationTests: verify handler registration
- MessageBoxTests: add real prefab tests using new SkipDontDestroyOnLoad
test hook for EditMode compatibility
- JDropdownTests: add formatter, edge case, and ForEnum tests
- JObjectFieldTests: add panel attachment and BindProperty tests
- JTextFieldTests: add panel attachment, placeholder, and edge case tests
Also adds SkipDontDestroyOnLoad test hook to MessageBox to enable
EditMode testing without DontDestroyOnLoad errors.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
* chore: update compiled assets and bundles for standalone mode
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
* fix(test): address code review feedback
- Convert sync [Test] to [UnityTest] with await for ExecuteAsync tests
- Rename misleading test names to match assertions
- Remove handler invocation tests with poor error handling
- Use discard for intentionally unused task variables
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
* fix(util): ensure JAction disposal if ExecuteAsync throws
Use 'using var' pattern consistently to ensure JAction is properly
disposed even if ExecuteAsync() throws an exception. This addresses
the code scanning alert about Dispose not being called on exception.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
* fix(util): add timeouts to WaitUntil calls in tests
Add 5-second timeouts to UniTask.WaitUntil calls to prevent tests from
hanging indefinitely if there's a regression. Uses CancellationTokenSource
with timeout for deterministic failure behavior.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
---------
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>1 parent a4666d0 commit 2ea9238
File tree
80 files changed
+1828
-165
lines changed- UnityProject
- Assets
- HotUpdate/Compiled
- AOT
- HybridCLRGenerate
- Obfuz/SymbolObfus
- StreamingAssets/yoo/main
- Packages
- com.jasonxudeveloper.jengine.ui
- Runtime
- Tests/Editor
- Components/Form
- Internal
- com.jasonxudeveloper.jengine.util/Tests/Editor
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
80 files changed
+1828
-165
lines changedBinary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Lines changed: 7 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Binary file not shown.
Binary file not shown.
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
231 | 231 | | |
232 | 232 | | |
233 | 233 | | |
| 234 | + | |
234 | 235 | | |
235 | 236 | | |
236 | 237 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| |||
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
| 63 | + | |
63 | 64 | | |
64 | 65 | | |
65 | 66 | | |
| |||
Lines changed: 133 additions & 52 deletions
Large diffs are not rendered by default.
0 commit comments