go/staking/api: Check if thresholds for all kinds are defined in genesis

oasisprotocol · Feb 3, 2020 · ec01c47 · ec01c47
1 parent d1489f8
commit ec01c47
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changelog/2633.bugfix.md b/.changelog/2633.bugfix.md
@@ -0,0 +1 @@
+go/staking/api: Check if thresholds for all kinds are defined in genesis.
diff --git a/go/staking/api/sanity_check.go b/go/staking/api/sanity_check.go
@@ -143,9 +143,13 @@ func SanityCheckAccountShares(acct *Account, delegations map[signature.PublicKey
 
 // SanityCheck does basic sanity checking on the genesis state.
 func (g *Genesis) SanityCheck(now epochtime.EpochTime) error { // nolint: gocyclo
-	for thr, val := range g.Parameters.Thresholds {
+	for kind := KindEntity; kind <= KindMax; kind++ {
+		val, ok := g.Parameters.Thresholds[kind]
+		if !ok {
+			return fmt.Errorf("staking: sanity check failed: threshold for kind '%s' not defined", kind)
+		}
 		if !val.IsValid() {
-			return fmt.Errorf("staking: sanity check failed: threshold '%s' has invalid value", thr.String())
+			return fmt.Errorf("staking: sanity check failed: threshold '%s' has invalid value", kind)
 		}
 	}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		go/staking/api: Check if thresholds for all kinds are defined in genesis.