test: unittest for stakingmiddleware module

.github/workflows/build.yml

@@ -1,14 +1,6 @@
 ---
-on:
-  push:
-    paths:
-      - '**.go'      
-      - 'go.sum'
-  pull_request:
-    paths:
-      - '**.go'
-      - 'go.sum'
-name: build
+on: [push, pull_request]
+name: Build
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -19,27 +11,27 @@ jobs:
     runs-on: ubuntu-latest
     name: build
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Setup go
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v4
         with:
-          go-version: 1.19
+          go-version: '1.20'
       - run: go build ./...
 
   tidy:
     runs-on: ubuntu-latest
     name: tidy
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Setup go
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v4
         with:
-          go-version: 1.19
+          go-version: '1.20'
       - run: |
           go mod tidy
           CHANGES_IN_REPO=$(git status --porcelain)
           if [[ -n "$CHANGES_IN_REPO" ]]; then
             echo "Repository is dirty. Showing 'git status' and 'git --no-pager diff' for debugging now:"
             git status && git --no-pager diff
             exit 1
-          fi
+          fi

.github/workflows/codeql.yml

@@ -26,7 +26,7 @@ jobs:
         uses: actions/checkout@v3
       - uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: '1.20'
       # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
         uses: github/codeql-action/init@v2

.github/workflows/interchaintest.yml

@@ -33,15 +33,15 @@ jobs:
           push: true
           platforms: linux/amd64
           tags: |
-            ghcr.io/notional-labs/centauri-ictest:latest
+            ghcr.io/composablefi/centauri-ictest:latest
   test-start-cosmos-chain:
     runs-on: ubuntu-latest
     needs: build-and-push-image
     steps:
-      - name: Set up Go 1.19
+      - name: Set up Go 1.20
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: '1.20'
 
       - name: checkout code
         uses: actions/checkout@v3
@@ -54,10 +54,10 @@ jobs:
     runs-on: ubuntu-latest
     needs: build-and-push-image
     steps:
-      - name: Set up Go 1.19
+      - name: Set up Go 1.20
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: '1.20'
 
       - name: checkout code
         uses: actions/checkout@v3
@@ -70,10 +70,10 @@ jobs:
     runs-on: ubuntu-latest
     needs: build-and-push-image
     steps:
-      - name: Set up Go 1.19
+      - name: Set up Go 1.20
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: '1.20'
 
       - name: checkout code
         uses: actions/checkout@v3
@@ -86,10 +86,10 @@ jobs:
     runs-on: ubuntu-latest
     needs: build-and-push-image
     steps:
-      - name: Set up Go 1.19
+      - name: Set up Go 1.20
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: '1.20'
 
       - name: checkout code
         uses: actions/checkout@v3
@@ -102,10 +102,10 @@ jobs:
   #   runs-on: ubuntu-latest
   #   needs: build-and-push-image
   #   steps:
-  #     - name: Set up Go 1.19
+  #     - name: Set up Go 1.20
   #       uses: actions/setup-go@v3
   #       with:
-  #         go-version: 1.19
+  #         go-version: 1.20
 
   #     - name: checkout code
   #       uses: actions/checkout@v3
@@ -118,10 +118,10 @@ jobs:
   #   runs-on: ubuntu-latest
   #   needs: build-and-push-image
   #   steps:
-  #     - name: Set up Go 1.19
+  #     - name: Set up Go 1.20
   #       uses: actions/setup-go@v3
   #       with:
-  #         go-version: 1.19
+  #         go-version: 1.20
 
   #     - name: checkout code
   #       uses: actions/checkout@v3

.github/workflows/lint.yml

@@ -17,7 +17,7 @@ jobs:
     steps:
       - uses: actions/setup-go@v3
         with:
-          go-version: 1.19
+          go-version: "1.20"
       - uses: actions/checkout@v3
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v3

.github/workflows/tests.yml

@@ -12,10 +12,10 @@ jobs:
     name: test
     steps:
       - name: Install Go
-        uses: actions/setup-go@v3
+        uses: actions/setup-go@v4
         with:
-          go-version: 1.19
+          go-version: '1.20'
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Test
         run: go test ./...

Dockerfile

@@ -7,13 +7,15 @@ ARG RUNNER_IMAGE="gcr.io/distroless/static-debian11"
 # Builder
 # --------------------------------------------------------
 
-FROM golang:${GO_VERSION}-alpine as builder
+FROM golang:${GO_VERSION}-alpine3.18 as builder
 
 ARG GIT_VERSION
 ARG GIT_COMMIT
 
 RUN apk add --no-cache \
     ca-certificates \
+    musl-dev \
+    openssl-dev \
     build-base \
     linux-headers
 

x/stakingmiddleware/keeper/keeper_test.go

@@ -0,0 +1,110 @@
+package keeper_test
+
+import (
+	"testing"
+	"time"
+
+	tmproto "github.com/cometbft/cometbft/proto/tendermint/types"
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	moduletestutil "github.com/cosmos/cosmos-sdk/types/module/testutil"
+	authtypes "github.com/cosmos/cosmos-sdk/x/auth/types"
+	govtypes "github.com/cosmos/cosmos-sdk/x/gov/types"
+	"github.com/notional-labs/composable/v6/app"
+	"github.com/notional-labs/composable/v6/app/helpers"
+	stakingmiddlewarekeeper "github.com/notional-labs/composable/v6/x/stakingmiddleware/keeper"
+	stakingmiddlewaretypes "github.com/notional-labs/composable/v6/x/stakingmiddleware/types"
+	"github.com/stretchr/testify/suite"
+)
+
+type KeeperTestSuite struct {
+	suite.Suite
+
+	ctx sdk.Context
+	// querier sdk.Querier
+	app       *app.ComposableApp
+	msgServer stakingmiddlewaretypes.MsgServer
+}
+
+func (suite *KeeperTestSuite) SetupTest() {
+	suite.app = helpers.SetupComposableAppWithValSet(suite.T())
+	suite.ctx = suite.app.BaseApp.NewContext(false, tmproto.Header{Height: 1, ChainID: "centauri-1", Time: time.Now().UTC()})
+	encCfg := moduletestutil.MakeTestEncodingConfig()
+	key := suite.app.GetKey(stakingmiddlewaretypes.StoreKey)
+	keeper := stakingmiddlewarekeeper.NewKeeper(
+		encCfg.Codec,
+		key,
+		suite.app.AccountKeeper,
+		suite.app.BankKeeper,
+		authtypes.NewModuleAddress(govtypes.ModuleName).String(),
+	)
+	keeper.RegisterKeepers(suite.app.StakingKeeper)
+	suite.msgServer = stakingmiddlewarekeeper.NewMsgServerImpl(keeper)
+}
+
+func TestKeeperTestSuite(t *testing.T) {
+	suite.Run(t, new(KeeperTestSuite))
+}
+
+var (
+	newParams = stakingmiddlewaretypes.Params{
+		BlocksPerEpoch:                           10,
+		AllowUnbondAfterEpochProgressBlockNumber: 7,
+	}
+	failParams = stakingmiddlewaretypes.Params{
+		BlocksPerEpoch:                           3,
+		AllowUnbondAfterEpochProgressBlockNumber: 3,
+	}
+	failAllowParams = stakingmiddlewaretypes.Params{
+		BlocksPerEpoch:                           6,
+		AllowUnbondAfterEpochProgressBlockNumber: 10,
+	}
+)
+
+func (suite *KeeperTestSuite) TestSetParams() {
+	for _, tc := range []struct {
+		desc           string
+		expectedParams stakingmiddlewaretypes.Params
+		malleate       func() error
+		shouldErr      bool
+		expectedErr    string
+	}{
+		{
+			desc:           "Case success",
+			expectedParams: newParams,
+			malleate: func() error {
+				return suite.app.StakingMiddlewareKeeper.SetParams(suite.ctx, newParams)
+			},
+			shouldErr: false,
+		},
+		{
+			desc:           "Case fail: BlocksPerEpoch < 5",
+			expectedParams: failParams,
+			malleate: func() error {
+				return suite.app.StakingMiddlewareKeeper.SetParams(suite.ctx, failParams)
+			},
+			shouldErr:   true,
+			expectedErr: "BlocksPerEpoch must be greater than or equal to 5",
+		},
+		{
+			desc:           "Case fail: BlocksPerEpoch < AllowUnbondAfterEpochProgressBlockNumber",
+			expectedParams: failAllowParams,
+			malleate: func() error {
+				return suite.app.StakingMiddlewareKeeper.SetParams(suite.ctx, failAllowParams)
+			},
+			shouldErr:   true,
+			expectedErr: "AllowUnbondAfterEpochProgressBlockNumber must be less than or equal to BlocksPerEpoch",
+		},
+	} {
+		tc := tc
+		suite.Run(tc.desc, func() {
+			suite.SetupTest()
+			err := tc.malleate()
+			if !tc.shouldErr {
+				res := suite.app.StakingMiddlewareKeeper.GetParams(suite.ctx)
+				suite.Equal(res, tc.expectedParams)
+			} else {
+				suite.Equal(err.Error(), tc.expectedErr)
+			}
+		})
+	}
+}