feat: x/precisebank module

.github/workflows/test.yml

@@ -70,3 +70,25 @@ jobs:
         run: |
           make test-scripts
         if: env.GIT_DIFF
+
+  test-fuzz:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.22"
+          check-latest: true
+      - uses: actions/checkout@v4
+      - uses: technote-space/[email protected]
+        with:
+          PATTERNS: |
+            .github/workflows/test.yml
+            **/**.sol
+            **/**.go
+            go.mod
+            go.sum
+            *.toml
+      - name: run fuzz tests
+        run: |
+          make test-fuzz
+        if: env.GIT_DIFF

CHANGELOG.md

@@ -17,6 +17,8 @@
 
 ### FEATURES
 
+- [\#69](https://github.com/cosmos/evm/pull/69) Add new `x/precisebank` module with bank decimal extension for EVM usage.
+
 ### STATE BREAKING
 
 - Refactored evmos/os into cosmos/evm

Makefile

@@ -130,6 +130,18 @@ else
 	go test -tags=test -mod=readonly $(ARGS)  $(EXTRA_ARGS) $(TEST_PACKAGES)
 endif
 
+# Use the old Apple linker to workaround broken xcode - https://github.com/golang/go/issues/65169
+ifeq ($(OS_FAMILY),Darwin)
+  FUZZLDFLAGS := -ldflags=-extldflags=-Wl,-ld_classic
+endif
+
+test-fuzz:
+	go test -tags=test $(FUZZLDFLAGS) -run NOTAREALTEST -v -fuzztime 10s -fuzz=FuzzMintCoins ./x/precisebank/keeper
+	go test -tags=test $(FUZZLDFLAGS) -run NOTAREALTEST -v -fuzztime 10s -fuzz=FuzzBurnCoins ./x/precisebank/keeper
+	go test -tags=test $(FUZZLDFLAGS) -run NOTAREALTEST -v -fuzztime 10s -fuzz=FuzzSendCoins ./x/precisebank/keeper
+	go test -tags=test $(FUZZLDFLAGS) -run NOTAREALTEST -v -fuzztime 10s -fuzz=FuzzGenesisStateValidate_NonZeroRemainder ./x/precisebank/types
+	go test -tags=test $(FUZZLDFLAGS) -run NOTAREALTEST -v -fuzztime 10s -fuzz=FuzzGenesisStateValidate_ZeroRemainder ./x/precisebank/types
+
 test-scripts:
 	@echo "Running scripts tests"
 	@pytest -s -vv ./scripts

ante/evm/04_validate.go

@@ -110,11 +110,9 @@ func CheckTxFee(txFeeInfo *tx.Fee, txFee *big.Int, txGasLimit uint64) error {
 		return nil
 	}
 
-	convertedAmount := sdkmath.NewIntFromBigInt(evmtypes.ConvertAmountFrom18DecimalsBigInt(txFee))
-
-	baseDenom := evmtypes.GetEVMCoinDenom()
-	if !txFeeInfo.Amount.AmountOf(baseDenom).Equal(convertedAmount) {
-		return errorsmod.Wrapf(errortypes.ErrInvalidRequest, "invalid AuthInfo Fee Amount (%s != %s)", txFeeInfo.Amount, convertedAmount)
+	evmExtendedDenom := evmtypes.GetEVMCoinExtendedDenom()
+	if !txFeeInfo.Amount.AmountOf(evmExtendedDenom).Equal(sdkmath.NewIntFromBigInt(txFee)) {
+		return errorsmod.Wrapf(errortypes.ErrInvalidRequest, "invalid AuthInfo Fee Amount (%s != %s)", txFeeInfo.Amount, txFee)
 	}
 
 	if txFeeInfo.GasLimit != txGasLimit {

ante/evm/04_validate_test.go

@@ -245,25 +245,23 @@ func (suite *EvmAnteTestSuite) TestCheckTxFee() {
 		},
 	}
 
-	for _, decimals := range []evmtypes.Decimals{
-		evmtypes.SixDecimals,
-		evmtypes.EighteenDecimals,
+	for _, chainID := range []string{
+		testconstants.ExampleChainID,
+		testconstants.SixDecimalsChainID,
 	} {
 		for _, tc := range testCases {
-			suite.Run(fmt.Sprintf("%d decimals, %s", decimals, tc.name), func() {
+			suite.Run(fmt.Sprintf("%s, %s", chainID, tc.name), func() {
 				// Call the configurator to set the EVM coin required for the
 				// function to be tested.
 				configurator := evmtypes.NewEVMConfigurator()
 				configurator.ResetTestConfig()
-				suite.Require().NoError(configurator.WithEVMCoinInfo(testconstants.ExampleAttoDenom, uint8(decimals)).Configure())
+				suite.Require().NoError(configurator.WithEVMCoinInfo(testconstants.ExampleChainCoinInfo[chainID]).Configure())
 
 				// If decimals is not 18 decimals, we have to convert txFeeInfo to original
 				// decimals representation.
-				originalAmount := amount
-				evmCoinDecimal := evmtypes.GetEVMCoinDecimals()
-				originalAmount = originalAmount.Quo(evmCoinDecimal.ConversionFactor())
+				evmExtendedDenom := evmtypes.GetEVMCoinExtendedDenom()
 
-				coins := sdktypes.Coins{sdktypes.Coin{Denom: "aatom", Amount: originalAmount}}
+				coins := sdktypes.Coins{sdktypes.Coin{Denom: evmExtendedDenom, Amount: amount}}
 
 				// This struct should hold values in the original representation
 				txFeeInfo := &tx.Fee{

ante/testutils/testutil.go

@@ -103,13 +103,19 @@ func (suite *AnteTestSuite) SetupTest() {
 	// get the denom and decimals set when initialized the chain
 	// to set them again
 	// when resetting the chain config
-	denom := evmtypes.GetEVMCoinDenom()       //nolint:staticcheck
-	decimals := evmtypes.GetEVMCoinDecimals() //nolint:staticcheck
+	denom := evmtypes.GetEVMCoinDenom()                 //nolint:staticcheck
+	extendedDenom := evmtypes.GetEVMCoinExtendedDenom() //nolint:staticcheck
+	decimals := evmtypes.GetEVMCoinDecimals()           //nolint:staticcheck
+
 	configurator := evmtypes.NewEVMConfigurator()
 	configurator.ResetTestConfig()
 	err := configurator.
 		WithChainConfig(chainConfig).
-		WithEVMCoinInfo(denom, uint8(decimals)).
+		WithEVMCoinInfo(evmtypes.EvmCoinInfo{
+			Denom:         denom,
+			ExtendedDenom: extendedDenom,
+			Decimals:      decimals,
+		}).
 		Configure()
 	suite.Require().NoError(err)