core, miner: drop transactions from the same sender when error occurs…

[ucwong]

This PR unifies the error handling in miner.

Whenever an error occur while applying a transaction, the transaction should be regarded as invalid and all following transactions from the same sender not executable because of the nonce restriction. The only exception is the nonceTooLow error which is handled separately.

[codecov]

Codecov Report

Merging #1490 (06dbee8) into master (0287be0) will increase coverage by 0.06%.
The diff coverage is 56.36%.

❗ Current head 06dbee8 differs from pull request most recent head df4ce32. Consider uploading reports for the commit df4ce32 to get more accurate results

@@            Coverage Diff             @@
##           master    #1490      +/-   ##
==========================================
+ Coverage   52.09%   52.15%   +0.06%     
==========================================
  Files         474      474              
  Lines       59489    59503      +14     
==========================================
+ Hits        30990    31033      +43     
+ Misses      26150    26117      -33     
- Partials     2349     2353       +4     

Impacted Files	Coverage Δ
cmd/cortex/accountcmd.go	0.00% <0.00%> (ø)
cmd/cortex/main.go	21.73% <0.00%> (-0.59%)	⬇️
miner/worker.go	0.00% <0.00%> (ø)
accounts/abi/bind/base.go	19.25% <33.33%> (+0.67%)	⬆️
core/types/transaction.go	65.71% <50.00%> (+0.37%)	⬆️
core/txpool/txpool.go	74.79% <81.25%> (+0.37%)	⬆️
ctxc/downloader/downloader.go	77.40% <81.81%> (+0.10%)	⬆️
consensus/cuckoo/consensus.go	7.24% <100.00%> (+0.41%)	⬆️
core/state_transition.go	40.55% <100.00%> (ø)
core/txpool/list.go	88.25% <100.00%> (ø)
... and 2 more

... and 13 files with indirect coverage changes