Merge #2923 #2924

2923: fix: callback potentially incorrect trigger on concurrent context r=yangby-cryptape,quake a=zhangsoledad

https://github.com/nervosnetwork/ckb/blob/develop/tx-pool/src/process.rs#L471
https://github.com/nervosnetwork/ckb/blob/develop/tx-pool/src/process.rs#L707-L709 
duplicate checked before submitting, but the lock released later,  there is still potentially inclusion.

2924: fix(test): make sure testnode graceful shutdown basic sync r=quake,yangby-cryptape a=zhangsoledad

tests::synchronizer::basic_sync do not graceful shutdown, may cause 
```
pure virtual method called                                                                                                                                                                   
terminate called without an active exception 
```
bump crossbeam-channel from 0.3.9 to 0.5.1 incidentally.


Co-authored-by: zhangsoledad <[email protected]>