Revert burner contract to v1.2.2 version

[alpe]

In the current version the DB state is not dropped and no message returned. This test is used for integration tests in wasmd

[webmaster128]

Cleaning up the state was extracted into a separate execute method because otherwise it is not really usable. See #1567. You can send a {"cleanup":{}} to clear all state or limit it with e.g. {"cleanup":{"limit":15}} for the first 15 state entries.

The funds should be sent to the payout address during migrate as before.

[alpe]

With the old burner contract we were able to ensure that contracts can modify state and return data in a migration. How can we cover this now?

[webmaster128]

@chipshort let's just add an optional item count to the migration message that defaults to 0. Then the contract can cleanup state during migration and afterwards depending on the use case. This allows Alex can easily migrate their tests in wasmd where only a handful of storage keys are set before the migration.

[webmaster128]

@alpe with the next release, this should bring you back into business:

diff --git a/x/wasm/keeper/keeper_test.go b/x/wasm/keeper/keeper_test.go
index 69c9517c..f7dbb6f3 100644
--- a/x/wasm/keeper/keeper_test.go
+++ b/x/wasm/keeper/keeper_test.go
@@ -1322,7 +1322,7 @@ func TestMigrateReplacesTheSecondIndex(t *testing.T) {
        ctx = ctx.WithBlockHeight(ctx.BlockHeight() + 1) // increment for different block
        // when do migrate
        newCodeExample := StoreBurnerExampleContract(t, ctx, keepers)
-       migMsgBz := BurnerExampleInitMsg{Payout: example.CreatorAddr}.GetBytes(t)
+       migMsgBz := BurnerExampleInitMsg{Payout: example.CreatorAddr, Delete: 20}.GetBytes(t)
        _, err := keepers.ContractKeeper.Migrate(ctx, example.Contract, example.CreatorAddr, newCodeExample.CodeID, migMsgBz)
        require.NoError(t, err)
 
@@ -1367,7 +1367,7 @@ func TestMigrateWithDispatchedMessage(t *testing.T) {
        contractAddr, _, err := keepers.ContractKeeper.Instantiate(ctx, originalContractID, creator, fred, initMsgBz, "demo contract", deposit)
        require.NoError(t, err)
 
-       migMsgBz := BurnerExampleInitMsg{Payout: myPayoutAddr}.GetBytes(t)
+       migMsgBz := BurnerExampleInitMsg{Payout: myPayoutAddr, Delete: 20}.GetBytes(t)
        ctx = ctx.WithEventManager(sdk.NewEventManager()).WithBlockHeight(ctx.BlockHeight() + 1)
        data, err := keeper.Migrate(ctx, contractAddr, fred, burnerContractID, migMsgBz)
        require.NoError(t, err)
diff --git a/x/wasm/keeper/test_common.go b/x/wasm/keeper/test_common.go
index 82993cc0..ba7f9f58 100644
--- a/x/wasm/keeper/test_common.go
+++ b/x/wasm/keeper/test_common.go
@@ -822,6 +822,7 @@ func (m IBCReflectInitMsg) GetBytes(tb testing.TB) []byte {
 
 type BurnerExampleInitMsg struct {
        Payout sdk.AccAddress `json:"payout"`
+       Delete uint32         `json:"delete"`
 }
 
 func (m BurnerExampleInitMsg) GetBytes(tb testing.TB) []byte {