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

Migrate unit tests from AWS to GKE #6082

Closed Shaddoll closed 4 weeks ago

Shaddoll commented 4 weeks ago

What changed? Migrate unit tests from AWS to GKE

Why? to complete https://github.com/uber/cadence/pull/5912

How did you test it? existing tests

Potential risks Tests being flaky can be reverted

Release notes

Documentation Changes

codecov[bot] commented 4 weeks ago

Codecov Report

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

Project coverage is 68.29%. Comparing base (f0f7efd) to head (871dd24).

Additional details and impacted files [see 10 files with indirect coverage changes](https://app.codecov.io/gh/uber/cadence/pull/6082/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/6082?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/6082?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uber). Last update [f0f7efd...871dd24](https://app.codecov.io/gh/uber/cadence/pull/6082?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 4 weeks ago

Pull Request Test Coverage Report for Build 018fdf34-e6ba-4f96-b216-41f8c3285b9e

Details


Files with Coverage Reduction New Missed Lines %
common/persistence/sql/sqlplugin/postgres/task.go 2 73.4%
service/history/task/transfer_active_task_executor.go 2 72.92%
service/matching/tasklist/matcher.go 2 89.35%
common/persistence/historyManager.go 2 66.67%
service/history/task/transfer_standby_task_executor.go 3 87.14%
service/history/task/task.go 3 84.81%
tools/cli/admin_db_decode_thrift.go 4 67.95%
service/history/execution/cache.go 6 74.61%
service/history/task/fetcher.go 6 85.05%
service/history/task/cross_cluster_task_processor.go 8 80.79%
<!-- Total: 67 -->
Totals Coverage Status
Change from base Build 018fdeeb-1615-4507-ae03-236a4f5abcab: -0.01%
Covered Lines: 103074
Relevant Lines: 147936

💛 - Coveralls