re-enable testCustomExecutor

[gudegg]

What is the purpose of the change

re-enable testCustomExecutor

Brief changelog

update UT

Follow this checklist to help us incorporate your contribution quickly and easily:

• 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.
• Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. 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.
• 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 integration-test in test module.
• Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
• If this contribution is large, please follow the Software Donation Guide.

[codecov-io]

Codecov Report

Merging #2917 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2917      +/-   ##
==========================================
+ Coverage   63.67%   63.73%   +0.05%     
==========================================
  Files         577      578       +1     
  Lines       25921    25928       +7     
  Branches     4536     4539       +3     
==========================================
+ Hits        16506    16524      +18     
+ Misses       7247     7231      -16     
- Partials     2168     2173       +5

Impacted Files	Coverage Δ
.../dubbo/rpc/cluster/router/script/ScriptRouter.java	59.18% <0%> (-6.73%)	⬇️
...org/apache/dubbo/rpc/filter/ActiveLimitFilter.java	77.77% <0%> (-5.56%)	⬇️
.../apache/dubbo/remoting/transport/AbstractPeer.java	63.04% <0%> (-4.35%)	⬇️
...c/main/java/org/apache/dubbo/rpc/RpcException.java	85.71% <0%> (-3.58%)	⬇️
...dubbo/rpc/cluster/router/MockInvokersSelector.java	91.42% <0%> (-2.69%)	⬇️
...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java	77.2% <0%> (-2.21%)	⬇️
...rg/apache/dubbo/common/timer/HashedWheelTimer.java	58.88% <0%> (-1.75%)	⬇️
.../exchange/support/header/HeaderExchangeServer.java	59.82% <0%> (-1.71%)	⬇️
...apache/dubbo/rpc/protocol/dubbo/DubboProtocol.java	65.83% <0%> (-0.84%)	⬇️
...dubbo/remoting/exchange/support/DefaultFuture.java	69.12% <0%> (-0.68%)	⬇️
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b000a3...2039856. Read the comment docs.

[beiwei30]

you need to remove @Ignore to verify.

[gudegg]

@beiwei30 I remove @Ignore,ci pass

[beiwei30]

it's cool, would you mind add further comments here to explain why it starts working?

[gudegg]

@beiwei30

    @Test
    public void testCustomExecutor() {
        Executor mockedExecutor = mock(Executor.class);
        CompletableFuture<Integer> completableFuture = CompletableFuture.supplyAsync(() -> {
            return 0;
        }, mockedExecutor);
        verify(mockedExecutor, times(1)).execute(any());
    }

最终调用 CompletableFuture#asyncSupplyStage

    static <U> CompletableFuture<U> asyncSupplyStage(Executor e,
                                                     Supplier<U> f) {
        if (f == null) throw new NullPointerException();
        CompletableFuture<U> d = new CompletableFuture<U>();
       //确保execute运行一次
        e.execute(new AsyncSupply<U>(d, f));
        return d;
    }