uber / cadence

Cadence is a distributed, scalable, durable, and highly available orchestration engine to execute asynchronous long-running business logic in a scalable and resilient way.
https://cadenceworkflow.io
MIT License
7.97k stars 773 forks source link

[code-coverage] Add tests for mutable_state_util.go #6062

Closed ketsiambaku closed 3 weeks ago

ketsiambaku commented 1 month ago

What changed? Added tests to improve code coverage in service/history/execution/mutable_state_util_test.go

Why? Code Coverage Initiative πŸ“ˆ

How did you test it?

Potential risks

Release notes

Documentation Changes

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.10%. Comparing base (c6a145d) to head (d96623b).

:exclamation: Current head d96623b differs from pull request most recent head bb0c768

Please upload reports for the commit bb0c768 to get more accurate results.

Additional details and impacted files [see 40 files with indirect coverage changes](https://app.codecov.io/gh/uber/cadence/pull/6062/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/uber/cadence/pull/6062?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber) > `Ξ” = absolute (impact)`, `ΓΈ = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/uber/cadence/pull/6062?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber). Last update [c6a145d...bb0c768](https://app.codecov.io/gh/uber/cadence/pull/6062?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber).
coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 018fdb1f-40a9-4f14-ab60-448573a7c3cf

Details


Files with Coverage Reduction New Missed Lines %
common/task/weighted_round_robin_task_scheduler.go 2 89.05%
service/history/replication/task_processor.go 2 81.25%
service/matching/tasklist/matcher.go 2 89.35%
common/persistence/nosql/nosql_task_store.go 3 85.52%
service/matching/tasklist/task_list_manager.go 3 75.49%
common/archiver/filestore/historyArchiver.go 4 80.95%
common/task/fifo_task_scheduler.go 4 83.51%
service/frontend/api/handler.go 6 62.45%
service/history/task/transfer_standby_task_executor.go 7 85.92%
service/history/engine/engineimpl/poll_mutable_state.go 9 74.16%
<!-- Total: 55 -->
Totals Coverage Status
Change from base Build 018fd17e-a401-4eba-8372-591d1e581574: 0.03%
Covered Lines: 103128
Relevant Lines: 147930

πŸ’› - Coveralls