@franco-h

• Hi friend, I noticed that org.apache.shardingsphere.agent.core.bytebuddy.transformer.advice.ComposeConstructorAdviceTest was written by you, I wonder if it is possible for us to have unit tests in this place to avoid the use of java.lang.Object? A key change made after Mockito 3.5.0 makes mocking a java.lang.* object deprecated. Reference NullPointerException in java.lang.reflect.Method.getParameterTypes mockito/mockito#2026 .

• Refer to Add unit test for ComposeClassStaticMethodAroundAdvice and ComposeConstructorAdvice (Issue #10958) #13958 .

• update: I removed the Mock interface and instantiated the class that implements the interface to avoid the restrictions on Mock objects after Mockito 3.5.0. The related changes caused unit tests to fail, and there may be room for improvement.

• 

• 

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.79%. Comparing base (90247fc) to head (7a12b68).
⚠️ Report is 13295 commits behind head on master.

@@             Coverage Diff              @@
##             master   #17665      +/-   ##
============================================
- Coverage     58.81%   58.79%   -0.03%     
- Complexity     2125     2126       +1     
============================================
  Files          3596     3599       +3     
  Lines         53536    53572      +36     
  Branches       9143     9144       +1     
============================================
+ Hits          31489    31497       +8     
- Misses        19373    19400      +27     
- Partials       2674     2675       +1     

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

Good Job!