Fix get_effective_balance (ethereum/consensus-specs#297)

Author: hwwhww

eth/beacon/helpers.py

@@ -7,6 +7,7 @@
 )
 
 from eth_utils import (
+    denoms,
     to_tuple,
     ValidationError,
 )
@@ -375,7 +376,7 @@ def get_effective_balance(validator: 'ValidatorRecord', max_deposit: int) -> int
     """
     Return the effective balance (also known as "balance at stake") for the ``validator``.
     """
-    return min(validator.balance, max_deposit)
+    return min(validator.balance, max_deposit * denoms.gwei)
 
 
 def get_new_validator_registry_delta_chain_tip(current_validator_registry_delta_chain_tip: Hash32,

tests/beacon/test_helpers.py

@@ -1,6 +1,7 @@
 import pytest
 
 from eth_utils import (
+    denoms,
     ValidationError,
 )
 
@@ -609,14 +610,19 @@ def mock_get_shard_committees_at_slot(state,
     ),
     [
         (
-            1,
+            1 * denoms.gwei,
             32,
-            1,
+            1 * denoms.gwei,
         ),
         (
-            33,
+            32 * denoms.gwei,
             32,
+            32 * denoms.gwei,
+        ),
+        (
+            33 * denoms.gwei,
             32,
+            32 * denoms.gwei,
         )
     ]
 )