Test coverage #1
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The entire publish workflow configuration file (.github/workflows/publish.yml) has been deleted, removing the automated publishing process for npm packages including both test and official releases. This change affects the CI/CD pipeline by eliminating the automatic package publishing functionality that was previously triggered by pushes, releases, and manual dispatches. ////// ci: 移除发布工作流配置 整个发布工作流配置文件 (.github/workflows/publish.yml) 已被删除，移除了 npm 包的自动发布流程，包括测试发布和正式发布。 此更改通过消除之前由推送、发布和手动调度触发的自动包发布功能来影响 CI/CD 流水线。 Change-Id: I125c45125558114dd68adfc1a366d29bfa463741 Signed-off-by: OhYee <[email protected]>
configure comprehensive test coverage thresholds, implement coverage checking script with directory-specific overrides, and update CI workflow to enforce coverage standards Configure comprehensive test coverage thresholds in coverage.yaml with 95% branch and line coverage requirements for both full and incremental coverage. Implement enhanced coverage checking script (scripts/check-coverage.ts) with YAML configuration loading, directory-specific threshold overrides, and detailed reporting capabilities. Update Makefile with new coverage targets including coverage-report and coverage-all. Enhance CI workflow (.github/workflows/ci.yml) to fetch complete history for incremental coverage checks and utilize make coverage command. Update jest.config.js to exclude non-critical modules from coverage calculation. Add comprehensive tests for Config class and DataAPI module. Change-Id: Ie206a4b17055bbd5e4409a9af992faf854bc2f7e Signed-off-by: OhYee <[email protected]>
Adjust the GitHub Actions workflow configuration to restrict test execution to push events only, removing the previous pull request trigger for main and master branches. 将 GitHub Actions 工作流配置调整为仅在 push 事件时执行测试，移除之前针对 main 和 master 分支的 pull request 触发器。 Change-Id: If7e83a49adb9b6fcfe7611cd9fa36efb8f23f4ad Signed-off-by: OhYee <[email protected]>
Some error handling tests in the ToolSet API were causing test process to hang due to unclosed SSE connections in the MCP SDK when making real network calls. These tests are now skipped and functionality will be covered by integration tests instead. 这些更改跳过了 ToolSet API 中的某些错误处理测试，因为当进行真实网络调用时，由于 MCP SDK 中未关闭的 SSE 连接，这些测试会导致测试进程挂起。这些测试现在被跳过，功能将通过集成测试来覆盖。 Change-Id: Ib4bfa9da455bad0c51f585af5409a8767f58790a Signed-off-by: OhYee <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix bugs
Bug detail
Pull request tasks
Update docs
Reason for update
Pull request tasks
Add contributor
Contributed content
Content detail
Others
Reason for update