Skip to content

Feature/workflow record #101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 26 commits into
base: main
Choose a base branch
from
Draft

Feature/workflow record #101

wants to merge 26 commits into from

Conversation

wooln
Copy link
Contributor

@wooln wooln commented May 7, 2025 •
edited
Loading

issue #52

  1. workflow http interceptor
  2. workflow gPRC interceptor (NOT streaming)
  3. workflow grpc bidirectional streaming TCC demo, record with Workflow.Do method, pass branch trans info via business body.

wooln added 25 commits April 25, 2025 21:11
@wooln
feat(OpDict): add rollback operation and update comments
c3da18b
@wooln
feat(workflow): add support for HTTP branches interceptor in workflows
c7ba719
@wooln
ci: update DTM server UpdateBranchSync: 1
f872c46 
- use config.yml(UpdateBranchSync: 0 with bug, maybe~~)
@wooln
test: simplify workflow test
b0f192d
@wooln
test(Dtmgrpc.IntegrationTests): enhance workflow test to cover SAGA a…
dc5300f 
…nd TCC patterns
@wooln
refactor(workflow): fix github action CI warnings
9e49ec2
@wooln
test(Dtmgrpc.IntegrationTests): add test for workflow grpc success
07dc028
@wooln
Execute_DoAndGrpc_Should_Success勉强通过
5fd5cd2
@wooln
feat: workflow interceptor
16dd6ec
@wooln
test(dtm): Add tests for Do and TCC workflow success and failure
50e9f43
@wooln
refactor: rollback AddHttpClient method in ServiceCollectionExtensions
db8efab 
- The unfinished version has an impact on DtmSample.
@wooln
feat(BusiGrpcService): implement stream TCC gRPC service and add basi…
b2cb647 
…c integration tests
@wooln
test(dtm): add stream grpc tcc and do integration tests for workflow
8786c31
@wooln
refactor(Dtmgrpc.IntegrationTests): improve gRPC streaming test with …
b10f066 
…a dedicated processor class
@wooln
refactor(grpc): stream unt test tcc
0f49350 
优化StreamTransOutTcc的单测，与普通业务调用单测保持一致。仅保留它为写拦截器做好用例
@wooln
替换grpc stream的双向流拦截器的实现，改为直接用workflow.Do实现分支事务实录
4ac3642
@wooln
test(Dtmgrpc.IntegrationTests): fix expected DtmFailureException in m…
1f8d88f 
…ultiple test cases
@wooln
refactor(Dtmgrpc.IntegrationTests): Fix MyGrpcProcesser multi-threade…
c838892 
…d concurrency bugs
@wooln
feat(test): add DtmBranchTransInfo to StreamRequest and update relate…
387f4ad 
…d services

- Add DtmBranchTransInfo message to busi.proto
- Update BusiApiService_Stream to use DtmBranchTransInfo
- Modify WorkflowGrpcStreamTest to include DtmBranchTransInfo in test cases
@wooln
refactor(BusiGrpcService): Add branchBarrier.Call demo with Transacti…
5379b89 
…onScope

- commented, because without database
@wooln
fix: fix workflow http interceptor missed adding query strings
a2fa337
@wooln
feat(BusiGrpcService): refactor HTTP tests and integrate Swagger
7719c96 
- Refactor HTTP test endpoints into a controller
- Add Swagger support for API documentation
n
@wooln
feature(BusiGrpcService): streamline gRPC service methods BranchBarri…
ea28bcd 
…er.call demo
@wooln
refactor(tests): Chinese comments to English
dc7f64e
@wooln
test: simplify BusiClient initialization
c0fd208
@codecov Codecov
Copy link

codecov bot commented May 7, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 17.39130% with 95 lines in your changes missing coverage. Please review.

Project coverage is 67.23%. Comparing base (16f174d) to head (68871fb).

Files with missing lines Patch % Lines
src/Dtmworkflow/WorkflowGrpcInterceptor.cs 0.00% 67 Missing ⚠️
src/Dtmworkflow/WorkflowHttpInterceptor.cs 16.66% 25 Missing ⚠️
src/Dtmworkflow/Workflow.Imp.cs 25.00% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   70.18%   67.23%   -2.95%     
==========================================
  Files          45       47       +2     
  Lines        1999     2103     +104     
  Branches      196      205       +9     
==========================================
+ Hits         1403     1414      +11     
- Misses        558      651      +93     
  Partials       38       38

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@wooln