cmd/geth: make tests run quicker + use less memory and disk (#21919)

# Conflicts:
#	cmd/geth/accountcmd_test.go
#	cmd/geth/consolecmd_test.go
#	cmd/geth/dao_test.go
#	cmd/geth/genesis_test.go
#	cmd/geth/les_test.go
This commit is contained in:
Martin Holst Swende 2020-11-30 14:43:20 +01:00 committed by Igor Mandrigin
parent 87a51988ba
commit 8b1d7681a2

162
diff.txt
View File

@ -2572,165 +2572,3 @@ Date: Mon Nov 30 14:43:20 2020 +0100
cmd/geth: make tests run quicker + use less memory and disk (#21919)
commit 566cb4c5f098a132d485a375cb7a02883c88a48f
Author: Pascal Dierich <pascal@merkleplant.xyz>
Date: Mon Nov 30 09:03:24 2020 +0100
accounts/keystore: add missing function doc for SignText (#21914)
Co-authored-by: Pascal Dierich <pascal@pascaldierich.com>
commit b71334ac3de38338e618aaf8ea6b4a884d2d80f5
Author: Kristofer Peterson <svenski123@users.noreply.github.com>
Date: Sun Nov 29 12:43:15 2020 +0000
accounts, signer: fix Ledger Live account derivation path (clef) (#21757)
* signer/core/api: fix derivation of ledger live accounts
For ledger hardware wallets, change account iteration as follows:
- ledger legacy: m/44'/60'/0'/X; for 0<=X<5
- ledger live: m/44'/60'/0'/0/X; for 0<=X<5
- ledger legacy: m/44'/60'/0'/X; for 0<=X<10
- ledger live: m/44'/60'/X'/0/0; for 0<=X<10
Non-ledger derivation is unchanged and remains as:
- non-ledger: m/44'/60'/0'/0/X; for 0<=X<10
* signer/core/api: derive ten default paths for all hardware wallets, plus ten legacy and ten live paths for ledger wallets
* signer/core/api: as .../0'/0/0 already included by default paths, do not include it again with ledger live paths
* accounts, signer: implement path iterators for hd wallets
Co-authored-by: Martin Holst Swende <martin@swende.se>
commit fa572cd2971fda0bf773d2817acb23385bac66c7
Author: Guillaume Ballet <gballet@gmail.com>
Date: Fri Nov 27 11:13:54 2020 +0000
crypto: signing builds with signify/minisign (#21798)
* internal/build: implement signify's signing func
* Add signify to the ci utility
* fix output file format
* Add unit test for signify
* holiman's + travis' feedback
* internal/build: verify signify's output
* crypto: move signify to common dir
* use go-minisign to verify binaries
* more holiman feedback
* crypto, ci: support minisign output
* only accept one-line trusted comments
* configurable untrusted comments
* code cleanup in tests
* revert to use ed25519 from the stdlib
* bug: fix for empty untrusted comments
* write timestamp as comment if trusted comment isn't present
* rename line checker to commentHasManyLines
* crypto: added signify fuzzer (#6)
* crypto: added signify fuzzer
* stuff
* crypto: updated signify fuzzer to fuzz comments
* crypto: repro signify crashes
* rebased fuzzer on build-signify branch
* hide fuzzer behind gofuzz build flag
* extract key data inside a single function
* don't treat \r as a newline
* travis: fix signing command line
* do not use an external binary in tests
* crypto: move signify to crypto/signify
* travis: fix formatting issue
* ci: fix linter build after package move
Co-authored-by: Marius van der Wijden <m.vanderwijden@live.de>
commit 429e7141f2f41c1d66dd4dd711a47ca9e0f0c2cb
Author: Nishant Das <nishdas93@gmail.com>
Date: Thu Nov 26 05:16:36 2020 +0800
p2p/discover: fix deadlock in discv5 message dispatch (#21858)
This fixes a deadlock that could occur when a response packet arrived
after a call had already received enough responses and was about to
signal completion to the dispatch loop.
Co-authored-by: Felix Lange <fjl@twurst.com>
commit 810f9e057daccda9eb5c04721b6dd4660f001cc5
Author: Alex Prut <1648497+alexprut@users.noreply.github.com>
Date: Wed Nov 25 21:00:23 2020 +0100
all: remove redundant conversions and import names (#21903)
commit f59ed3565d18c1fa676fd34f4fd41ecccad707e8
Author: Antoine Toulme <atoulme@users.noreply.github.com>
Date: Wed Nov 25 01:19:36 2020 -0800
graphql: always return 400 if errors are present in the response (#21882)
* Make sure to return 400 when errors are present in the response
* graphql: use less memory in chainconfig for tests
Co-authored-by: Martin Holst Swende <martin@swende.se>
commit c92faee66e8823de50fbe1dab4b0c4549802e730
Author: Alex Prut <1648497+alexprut@users.noreply.github.com>
Date: Wed Nov 25 09:24:50 2020 +0100
all: simplify nested complexity and if blocks ending with a return statement (#21854)
Changes:
Simplify nested complexity
If an if blocks ends with a return statement then remove the else nesting.
Most of the changes has also been reported in golint https://goreportcard.com/report/github.com/ethereum/go-ethereum#golint
commit 29efe1fc7eccf3eb99a69c52427efbdb30de1a9f
Author: Marius van der Wijden <m.vanderwijden@live.de>
Date: Wed Nov 25 08:53:20 2020 +0100
core/types: fixed typo (#21897)
commit 59b480ab4bf49e56d4564be9a1b5173d9612aa74
Author: Marius van der Wijden <m.vanderwijden@live.de>
Date: Tue Nov 24 16:09:17 2020 +0100
cmd/devp2p/internal/ethtest: add 'large announcement' tests (#21792)
* cmd/devp2p/internal/ethtest: added large announcement tests
* cmd/devp2p/internal/ethtest: added large announcement tests
* cmd/devp2p/internal/ethtest: refactored stuff a bit
* cmd/devp2p/internal/ethtest: added TestMaliciousStatus/Handshake
* cmd/devp2p/internal/ethtest: fixed rebasing issue
* happy linter, happy life
* cmd/devp2p/internal/ethtest: used readAndServe
* stuff
* cmd/devp2p/internal/ethtest: fixed test cases
commit 7e7a3f0f71d60510afb5cd06282c1db729c1f60b
Author: ligi <ligi@ligi.de>
Date: Tue Nov 24 16:02:53 2020 +0100
github: Remove vulnerability.md (#21894)
This type is automatically offered by github after changing to the new style and a security.md being present
commit bddd103a9f0af27ef533f04e06ea429cf76b6d46
Author: Zsolt Felföldi <zsfelfoldi@gmail.com>
Date: Tue Nov 24 10:55:17 2020 +0100
les: fix GetProofsV2 bug (#21896)