Skip to content

Commit

Permalink
apply review suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
mpoke committed Sep 4, 2023
1 parent ba0295d commit 2957dfa
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions tests/integration/distribution.go
Original file line number Diff line number Diff line change
Expand Up @@ -340,7 +340,7 @@ func (s *CCVTestSuite) TestEndBlockRD() {
}
}

// TestSendRewardsToProvider is effectively a unit test for TestSendRewardsToProvider(),
// TestSendRewardsToProvider is effectively a unit test for SendRewardsToProvider(),
// but is written as an integration test to avoid excessive mocking.
func (s *CCVTestSuite) TestSendRewardsToProvider() {
testCases := []struct {
Expand All @@ -352,7 +352,6 @@ func (s *CCVTestSuite) TestSendRewardsToProvider() {
{
name: "successful token transfer",
setup: func(ctx sdk.Context, keeper *consumerkeeper.Keeper, bankKeeper icstestingutils.TestBankKeeper) {
// setup transfer channel
s.SetupTransferChannel()

// register a consumer reward denom
Expand Down Expand Up @@ -382,7 +381,6 @@ func (s *CCVTestSuite) TestSendRewardsToProvider() {
{
name: "no reward denom",
setup: func(ctx sdk.Context, keeper *consumerkeeper.Keeper, bankKeeper icstestingutils.TestBankKeeper) {
// setup transfer channel
s.SetupTransferChannel()
},
expError: false,
Expand All @@ -391,7 +389,6 @@ func (s *CCVTestSuite) TestSendRewardsToProvider() {
{
name: "reward balance is zero",
setup: func(ctx sdk.Context, keeper *consumerkeeper.Keeper, bankKeeper icstestingutils.TestBankKeeper) {
// setup transfer channel
s.SetupTransferChannel()

// register a consumer reward denom
Expand All @@ -408,7 +405,6 @@ func (s *CCVTestSuite) TestSendRewardsToProvider() {
{
name: "no distribution transmission channel",
setup: func(ctx sdk.Context, keeper *consumerkeeper.Keeper, bankKeeper icstestingutils.TestBankKeeper) {
// setup transfer channel
s.SetupTransferChannel()

// register a consumer reward denom
Expand All @@ -432,7 +428,6 @@ func (s *CCVTestSuite) TestSendRewardsToProvider() {
{
name: "no recipient address",
setup: func(ctx sdk.Context, keeper *consumerkeeper.Keeper, bankKeeper icstestingutils.TestBankKeeper) {
// setup transfer channel
s.SetupTransferChannel()

// register a consumer reward denom
Expand Down

0 comments on commit 2957dfa

Please sign in to comment.