Refactor the test in the package org. Apache. Dubbo. Reactive.

[gzhao9]

Purpose of the Change

This PR is aimed at resolving the issue of duplicated creation of mock. This occurs in five test cases in ManyToManyMethodHandlerTest.java, ManyToOneMethodHandlerTest.java and OneToManyMethodHandlerTest.java, as highlighted in issue #13154.

Brief changelog

We created a reusable class creatObserverAdapter to create the mock, avoiding duplicate creation of the mock in 5 test cases.
Since in ManyToOneMethodHandlerTest.java and OneToManyMethodHandlerTest.java, the mock is both created duplicated in two test cases, we set this mock object as a class attribute and create it in @BeforeEach.
After refactoring, we removed duplicate lines of code for the following five test cases and improved readability and maintainability:

• ManyToManyMethodHandlerTest.java
  • testInvoke() Lines 43-52
• ManyToOneMethodHandlerTest.java
  • testInvoke() Lines 43-52
  • testError() Lines 68-77
• OneToManyMethodHandlerTest.java
  • testInvoke() Lines 43-52
  • testError() Lines 65-74

Verifying this change

Checklist

• Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
• Each commit in the pull request should have a meaningful subject line and body.
• Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
• Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
• Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
• Add some description to dubbo-website project if you are requesting to add a feature.
• GitHub Actions works fine on your own branch.
• If this contribution is large, please follow the Software Donation Guide.

[codecov-commenter]

Codecov Report

Merging #13175 (a1e5964) into 3.2 (3ca3504) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                3.2   #13175   +/-   ##
=========================================
  Coverage     69.38%   69.39%           
  Complexity        2        2           
=========================================
  Files          1648     1648           
  Lines         68693    68693           
  Branches      10014    10014           
=========================================
+ Hits          47661    47667    +6     
- Misses        16438    16441    +3     
+ Partials       4594     4585    -9     

see 26 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[AlbumenJ]

Is Creat correct? Maybe Create?

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication