fix(ci): embed v0.9.20 runtime, update test suite, and ci workflows (#…

…2543) * Add v0.9.20 chain specs & configs Closes #2419 * re-add gssmr config.toml * rename files back to old names * wip * use raw methods for build spec command * fix(ci): fix `dot` unit tests (#2561) * fix dot tests, remove life runtime * remove life * fix(ci): fix dot/network unit tests (#2572) * move to integration test, hardcode protocol id * remove logs, restore original test * cr feedback * fix(ci): fix dot/rpc/modules unit tests #2581 * fix(ci): embed old `gssmr` and `dev` runtimes for integration tests (#2671) * restore old chain as gssmr-v3substrate, add genesis utils, update TestNodeRuntime_ValidateTransaction * upgrade chaindb * add todos to create new issues, test against old runtime * use v3substrate runtime for dot/core integration tests * lib/babe integration tests use old dev runtime * fix lint * fix grandpa stress tests * use old dev chain for stress tests * update rpc tests * revise polkadotjstests to old dev runtime * Update lib/genesis/test_utils.go Co-authored-by: Quentin McGaw <[email protected]> * Update lib/utils/utils.go Co-authored-by: Quentin McGaw <[email protected]> * Update lib/utils/utils.go Co-authored-by: Quentin McGaw <[email protected]> * Update lib/utils/utils.go Co-authored-by: Quentin McGaw <[email protected]> * Update lib/utils/utils.go Co-authored-by: Quentin McGaw <[email protected]> * Update lib/utils/utils.go Co-authored-by: Quentin McGaw <[email protected]> * remove config and go code from old runtimes * undo new require in go.mod Co-authored-by: Quentin McGaw <[email protected]> * remove debugging sleep * remove unneeded files from chains * fix up go.mod * restore old go.mod * Apply suggestions from code review Co-authored-by: Quentin McGaw <[email protected]> * add todo Co-authored-by: Dan Forbes <[email protected]> Co-authored-by: Quentin McGaw <[email protected]>
ChainSafe · Aug 15, 2022 · 0fff418 · 0fff418
1 parent 2e57b1e
commit 0fff418
Show file tree

Hide file tree

Showing 40 changed files with 2,317 additions and 795 deletions.
diff --git a/chain/dev-staking/dev-staking-spec-raw.json b/chain/dev-staking/dev-staking-spec-raw.json
diff --git a/chain/dev-staking/dev-staking-spec.json b/chain/dev-staking/dev-staking-spec.json
diff --git a/chain/dev-v3substrate/genesis-spec.json b/chain/dev-v3substrate/genesis-spec.json
diff --git a/chain/dev-v3substrate/genesis.json b/chain/dev-v3substrate/genesis.json
diff --git a/chain/dev/config.toml b/chain/dev/config.toml
@@ -18,7 +18,7 @@ digest = ""
 genesis = "./chain/dev/genesis-spec.json"
 
 [account]
-key = "alice"
+key = ""
 unlock = ""
 
 [core]
@@ -37,7 +37,18 @@ enabled = true
 ws = true
 port = 8545
 host = "localhost"
-modules = ["system", "author", "chain", "state", "rpc", "grandpa", "offchain", "childstate", "syncstate", "payment"]
+modules = [
+    "system",
+    "author",
+    "chain",
+    "state",
+    "rpc",
+    "grandpa",
+    "offchain",
+    "childstate",
+    "syncstate",
+    "payment",
+]
 ws-port = 8546
 
 [pprof]

diff --git a/chain/dev/genesis-spec.json b/chain/dev/genesis-spec.json
diff --git a/chain/dev/genesis.json b/chain/dev/genesis.json
diff --git a/chain/gssmr-staking/gssmr-staking-spec-raw.json b/chain/gssmr-staking/gssmr-staking-spec-raw.json
diff --git a/chain/gssmr-staking/gssmr-staking-spec.json b/chain/gssmr-staking/gssmr-staking-spec.json
diff --git a/chain/gssmr-v3substrate/genesis-spec.json b/chain/gssmr-v3substrate/genesis-spec.json
diff --git a/chain/gssmr-v3substrate/genesis.json b/chain/gssmr-v3substrate/genesis.json
diff --git a/chain/gssmr/config.toml b/chain/gssmr/config.toml
@@ -39,7 +39,18 @@ max-peers = 50
 enabled = false
 port = 8545
 host = "localhost"
-modules = ["system", "author", "chain", "state", "rpc", "grandpa", "offchain", "childstate", "syncstate", "payment"]
+modules = [
+    "system",
+    "author",
+    "chain",
+    "state",
+    "rpc",
+    "grandpa",
+    "offchain",
+    "childstate",
+    "syncstate",
+    "payment",
+]
 ws-port = 8546
 
 [pprof]

diff --git a/chain/gssmr/genesis-spec.json b/chain/gssmr/genesis-spec.json
diff --git a/chain/gssmr/genesis.json b/chain/gssmr/genesis.json
diff --git a/chain/westend-dev/westend-dev-spec-raw.json b/chain/westend-dev/westend-dev-spec-raw.json
diff --git a/chain/westend-dev/westend-dev-spec.json b/chain/westend-dev/westend-dev-spec.json
diff --git a/chain/westend-local/westend-local-spec-raw.json b/chain/westend-local/westend-local-spec-raw.json
diff --git a/chain/westend-local/westend-local-spec.json b/chain/westend-local/westend-local-spec.json
diff --git a/cmd/gossamer/import_runtime.go b/cmd/gossamer/import_runtime.go
@@ -29,7 +29,7 @@ func createGenesisWithRuntime(fp string) (string, error) {
 		return "", err
 	}
 
-	genesis.Genesis.Runtime["System"]["code"] = fmt.Sprintf("0x%x", runtime)
+	genesis.Genesis.Runtime["system"]["code"] = fmt.Sprintf("0x%x", runtime)
 	bz, err := json.MarshalIndent(genesis, "", "\t")
 	if err != nil {
 		return "", err

diff --git a/cmd/gossamer/import_runtime_test.go b/cmd/gossamer/import_runtime_test.go
@@ -29,5 +29,5 @@ func TestCreateGenesisWithRuntime(t *testing.T) {
 	g := new(genesis.Genesis)
 	err = json.Unmarshal([]byte(out), g)
 	require.NoError(t, err)
-	require.Equal(t, testHex, g.Genesis.Runtime["System"]["code"].(string))
+	require.Equal(t, testHex, g.Genesis.Runtime["system"]["code"].(string))
 }
diff --git a/cmd/gossamer/main_test.go b/cmd/gossamer/main_test.go
@@ -268,7 +268,7 @@ func TestBuildSpecCommandWithOutput(t *testing.T) {
 	buildSpecCommand := runTestGossamer(t,
 		"build-spec",
 		"--raw",
-		"--genesis-spec", utils.GetGssmrGenesisPathTest(t),
+		"--genesis-spec", utils.GetGssmrGenesisRawPathTest(t),
 		"--output", tmpOutputfile)
 
 	time.Sleep(5 * time.Second)

diff --git a/cmd/gossamer/toml_config_test.go b/cmd/gossamer/toml_config_test.go
@@ -34,7 +34,7 @@ func TestLoadConfigGssmr(t *testing.T) {
 	require.NotNil(t, cfg)
 
 	cfg.Global.BasePath = t.TempDir()
-	cfg.Init.Genesis = utils.GetGssmrGenesisPathTest(t)
+	cfg.Init.Genesis = utils.GetGssmrGenesisRawPathTest(t)
 
 	err := dot.InitNode(cfg)
 	require.NoError(t, err)

diff --git a/dot/build_spec_test.go b/dot/build_spec_test.go
@@ -136,10 +136,10 @@ func TestBuildFromDB(t *testing.T) {
 	}{
 		{name: "normal conditions", path: cfg.Global.BasePath,
 			want: &BuildSpec{genesis: &genesis.Genesis{
-				Name:       "Gossamer",
-				ID:         "gssmr",
+				Name:       "Gossamer Testnet",
+				ID:         "gssmr_test",
 				Bootnodes:  []string{},
-				ProtocolID: "/gossamer/gssmr/0",
+				ProtocolID: "gssmr_test",
 				Genesis: genesis.Fields{
 					Raw:     map[string]map[string]string{},
 					Runtime: map[string]map[string]interface{}{},

diff --git a/dot/core/service_integration_test.go b/dot/core/service_integration_test.go
@@ -10,7 +10,6 @@ import (
 	"fmt"
 	"math/big"
 	"os"
-	"path/filepath"
 	"testing"
 	"time"
 
@@ -52,7 +51,7 @@ func balanceKey(t *testing.T, pub []byte) (bKey []byte) {
 
 func generateTestValidRemarkTxns(t *testing.T, pubKey []byte, accInfo types.AccountInfo) ([]byte, runtime.Instance) {
 	t.Helper()
-	projectRootPath := filepath.Join(utils.GetProjectRootPathTest(t), "chain/gssmr/genesis.json")
+	projectRootPath := utils.GetGssmrV3SubstrateGenesisRawPathTest(t)
 	gen, err := genesis.NewGenesisFromJSONRaw(projectRootPath)
 	require.NoError(t, err)
 
@@ -409,6 +408,8 @@ func TestHandleChainReorg_WithReorg_Transactions(t *testing.T) {
 	require.Equal(t, transaction.NewValidTransaction(tx, validity), pending[0])
 }
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2702
 func TestMaintainTransactionPool_EmptyBlock(t *testing.T) {
 	accountInfo := types.AccountInfo{
 		Nonce: 0,
@@ -465,6 +466,8 @@ func TestMaintainTransactionPool_EmptyBlock(t *testing.T) {
 	require.Nil(t, head)
 }
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2702
 func TestMaintainTransactionPool_BlockWithExtrinsics(t *testing.T) {
 	accountInfo := types.AccountInfo{
 		Nonce: 0,

diff --git a/dot/network/gossip_test.go b/dot/network/gossip_test.go
@@ -84,7 +84,7 @@ func TestGossip(t *testing.T) {
 		Digest: types.NewDigest(),
 	}
 
-	_, err = nodeA.host.send(addrInfoB.ID, "", announceMessage)
+	_, err = nodeA.host.send(addrInfoB.ID, "/gossamer/test/0/block-announces/1", announceMessage)
 	require.NoError(t, err)
 
 	time.Sleep(TestMessageTimeout)

diff --git a/dot/rpc/modules/childstate_test.go b/dot/rpc/modules/childstate_test.go
@@ -42,47 +42,14 @@ func createTestTrieState(t *testing.T) (*trie.Trie, common.Hash) {
 }
 
 func TestChildStateModule_GetKeys(t *testing.T) {
-	//nolint
-	expStr := []string{
-		"0x11f3ba2e1cdd6d62f2ff9b5589e7ff81ba7fb8745735dc3be2a2c61a72c39e78",
-		"0x1cb6f36e027abb2091cfb5110ab5087f5e0621c4869aa60c02be9adcc98a0d1d",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da904f6d6860c6ef3990f6d34dd83a345aabe1d9d59de1283380100550a7b024501cb62d6cc40e3db35fcc5cf341814986e",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da923a05cabf6d3bde7ca3ef0d11596b5611cbd2d43530a44705ad088af313e18f80b53ef16b36177cd4b77b846f2a5f07c",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da94f9aea1afa791265fae359272badc1cf8eaf04151687736326c9fea17e25fc5287613693c912909cb226aa4794f26a48",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da97a26f8a0cd62e2d6addec8cbfdd102af1206960f920a23f7f4c43cc9081ec2ed0721f31a9bef2c10fd7602e16e08a32c",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da981b4d873b42c98d6628cf8d2b2373afe4603307f855321776922daeea21ee31720388d097cdaac66f05a6f8462b31757",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da9b0edae20838083f2cde1c4080db8cf8090b5ab205c6974c9ea841be688864633dc9ca8a357843eeacf2314649965fe22",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da9de1e86a9a8c739864cf3cc5ec2bea59fd43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da9e5e802737cce3a54b0bc9e3d3e6be26e306721211d5404bd9da88e0204360a1a9ab8b87c66c1bc2fcdd37f3c2222cc20",
-		"0x26aa394eea5630e07c48ae0c9558cef7b99d880ec681799c0cf30e8886371da9edeaa42c2163f68084a988529a0e2ec5e659a7a1628cdd93febc04a4e0646ea20e9f5f0ce097d9a05290d4a9e054df4e",
-		"0x26aa394eea5630e07c48ae0c9558cef7c21aab032aaa6e946ca50ad39ab66603",
-		"0x3a6368696c645f73746f726167653a64656661756c743a3a6368696c645f73746f726167655f6b6579",
-		"0x3a636f6465",
-		"0x3a66697273745f6b6579",
-		"0x3a6772616e6470615f617574686f726974696573",
-		"0x3a7365636f6e645f6b6579",
-		"0x426e15054d267946093858132eb537f1a47a9ff5cd5bf4d848a80a0b1a947dc3",
-		"0x426e15054d267946093858132eb537f1ba7fb8745735dc3be2a2c61a72c39e78",
-		"0x426e15054d267946093858132eb537f1d0b4a3f7631f0c0e761898fe198211de",
-		"0x4342193e496fab7ec59d615ed0dc5530d2d505c0e6f76fd7ce0796ebe187401c",
-		"0x492a52699edf49c972c21db794cfcf57ba7fb8745735dc3be2a2c61a72c39e78",
-		"0x5f3e4907f716ac89b6347d15ececedca138e71612491192d68deab7e6f563fe1",
-		"0x5f3e4907f716ac89b6347d15ececedca28dccb559b95c40168a1b2696581b5a7",
-		"0x5f3e4907f716ac89b6347d15ececedca5579297f4dfb9609e7e4c2ebab9ce40a",
-		"0x5f3e4907f716ac89b6347d15ececedcaac0a2cbf8e355f5ea6cb2de8727bfb0c",
-		"0x5f3e4907f716ac89b6347d15ececedcab49a2738eeb30896aacb8b3fb46471bd",
-		"0x5f3e4907f716ac89b6347d15ececedcac29a0310e1bb45d20cace77ccb62c97d",
-		"0x5f3e4907f716ac89b6347d15ececedcaf7dad0317324aecae8744b87fc95f2f3",
-		"0x8985776095addd4789fccbce8ca77b23ba7fb8745735dc3be2a2c61a72c39e78",
-		"0xcec5070d609dd3497f72bde07fc96ba04c014e6bf8b8c2c011e7290b85696bb3e535263148daaf49be5ddb1579b72e84524fc29e78609e3caf42e85aa118ebfe0b0ad404b5bdd25f",
-		"0xcec5070d609dd3497f72bde07fc96ba0726380404683fc89e8233450c8aa195066b8d48da86b869b6261626580d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d",
-		"0xcec5070d609dd3497f72bde07fc96ba0726380404683fc89e8233450c8aa1950c9b0c13125732d276175646980d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d",
-		"0xcec5070d609dd3497f72bde07fc96ba0726380404683fc89e8233450c8aa1950ed43a85541921049696d6f6e80d43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d",
-		"0xcec5070d609dd3497f72bde07fc96ba0726380404683fc89e8233450c8aa1950f5537bdb2a1f626b6772616e8088dc3417d5058ec4b4503e0c12ea1a0a89be200fe98922423d4334014fa6b0ee",
-		"0xe2e62dd81c48a88f73b6f6463555fd8eba7fb8745735dc3be2a2c61a72c39e78"}
-
 	tr, sr := createTestTrieState(t)
 
+	expKeys := tr.GetKeysWithPrefix([]byte{})
+	expHexKeys := make([]string, len(expKeys))
+	for idx, k := range expKeys {
+		expHexKeys[idx] = common.BytesToHex(k)
+	}
+
 	mockStorageAPI := new(apimocks.StorageAPI)
 	mockErrorStorageAPI1 := new(apimocks.StorageAPI)
 	mockErrorStorageAPI2 := new(apimocks.StorageAPI)
@@ -128,7 +95,7 @@ func TestChildStateModule_GetKeys(t *testing.T) {
 					Key: []byte(":child_storage_key"),
 				},
 			},
-			exp: expStr,
+			exp: expHexKeys,
 		},
 		{
 			name: "Get Keys with Hash",
@@ -142,7 +109,7 @@ func TestChildStateModule_GetKeys(t *testing.T) {
 					Hash: &hash,
 				},
 			},
-			exp: expStr,
+			exp: expHexKeys,
 		},
 		{
 			name: "GetStorageChild error",

diff --git a/dot/sync/chain_processor_integration_test.go b/dot/sync/chain_processor_integration_test.go
@@ -20,6 +20,8 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2703
 func TestChainProcessor_HandleBlockResponse_ValidChain(t *testing.T) {
 	syncer := newTestSyncer(t)
 	responder := newTestSyncer(t)
@@ -79,6 +81,8 @@ func TestChainProcessor_HandleBlockResponse_ValidChain(t *testing.T) {
 	}
 }
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2703
 func TestChainProcessor_HandleBlockResponse_MissingBlocks(t *testing.T) {
 	syncer := newTestSyncer(t)
 
@@ -154,6 +158,8 @@ func TestChainProcessor_HandleBlockResponse_NoBlockData(t *testing.T) {
 	require.Equal(t, ErrNilBlockData, err)
 }
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2703
 func TestChainProcessor_HandleBlockResponse_BlockData(t *testing.T) {
 	syncer := newTestSyncer(t)
 
@@ -183,6 +189,8 @@ func TestChainProcessor_HandleBlockResponse_BlockData(t *testing.T) {
 	}
 }
 
+// TODO: add test against latest gssmr runtime
+// See https://github.com/ChainSafe/gossamer/issues/2703
 func TestChainProcessor_ExecuteBlock(t *testing.T) {
 	syncer := newTestSyncer(t)
 

diff --git a/dot/sync/syncer_integration_test.go b/dot/sync/syncer_integration_test.go
@@ -124,7 +124,7 @@ func newTestSyncer(t *testing.T) *Service {
 }
 
 func newTestGenesisWithTrieAndHeader(t *testing.T) (*genesis.Genesis, *trie.Trie, *types.Header) {
-	fp := utils.GetGssmrGenesisRawPathTest(t)
+	fp := utils.GetGssmrV3SubstrateGenesisRawPathTest(t)
 	gen, err := genesis.NewGenesisFromJSONRaw(fp)
 	require.NoError(t, err)
 

diff --git a/dot/utils_test.go b/dot/utils_test.go
@@ -24,8 +24,7 @@ import (
 func newTestGenesisFile(t *testing.T, cfg *Config) (filename string) {
 	t.Helper()
 
-	fp := utils.GetGssmrGenesisPathTest(t)
-
+	fp := utils.GetGssmrV3SubstrateGenesisPathTest(t)
 	gssmrGen, err := genesis.NewGenesisFromJSON(fp, 0)
 	require.NoError(t, err)
 
@@ -63,7 +62,7 @@ func TestCreateJSONRawFile(t *testing.T) {
 				bs: &BuildSpec{genesis: NewTestGenesis(t)},
 				fp: filepath.Join(t.TempDir(), "/test.json"),
 			},
-			expectedHash: "23356cdb5d3537d39b735726707216c9e329c7b8a2c8a41b25da0f5f936b3caa",
+			expectedHash: "6453fb6118dab594944a7ed68111fdccc3a68253c9d71ac05f27aa2359507c85",
 		},
 	}
 	for _, tt := range tests {

diff --git a/lib/babe/build_integration_test.go b/lib/babe/build_integration_test.go
@@ -100,6 +100,8 @@ func createTestBlock(t *testing.T, babeService *Service, parent *types.Header,
 	return block
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestBuildBlock_ok(t *testing.T) {
 	ctrl := gomock.NewController(t)
 	telemetryMock := NewMockClient(ctrl)
@@ -139,6 +141,8 @@ func TestBuildBlock_ok(t *testing.T) {
 	require.Equal(t, 1, len(extsBytes))
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestApplyExtrinsic(t *testing.T) {
 	ctrl := gomock.NewController(t)
 	telemetryMock := NewMockClient(ctrl)
@@ -233,6 +237,8 @@ func TestApplyExtrinsic(t *testing.T) {
 	require.NoError(t, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestBuildAndApplyExtrinsic(t *testing.T) {
 	ctrl := gomock.NewController(t)
 	telemetryMock := NewMockClient(ctrl)
@@ -406,6 +412,8 @@ func TestDecodeExtrinsicBody(t *testing.T) {
 	require.True(t, contains)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestBuildBlockTimeMonitor(t *testing.T) {
 	metrics.Enabled = true
 	metrics.Unregister(buildBlockTimer)

diff --git a/lib/babe/verify_integration_test.go b/lib/babe/verify_integration_test.go
@@ -63,6 +63,8 @@ func newTestVerificationManager(t *testing.T, genCfg *types.BabeConfiguration) *
 	return vm
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_OnDisabled_InvalidIndex(t *testing.T) {
 	vm := newTestVerificationManager(t, nil)
 	babeService := createTestService(t, nil)
@@ -74,6 +76,8 @@ func TestVerificationManager_OnDisabled_InvalidIndex(t *testing.T) {
 	require.Equal(t, err, ErrInvalidBlockProducerIndex)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_OnDisabled_NewDigest(t *testing.T) {
 	kp, err := sr25519.GenerateKeypair()
 	require.NoError(t, err)
@@ -111,6 +115,8 @@ func TestVerificationManager_OnDisabled_NewDigest(t *testing.T) {
 	require.NoError(t, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_OnDisabled_DuplicateDigest(t *testing.T) {
 	kp, err := sr25519.GenerateKeypair()
 	require.NoError(t, err)
@@ -146,6 +152,8 @@ func TestVerificationManager_OnDisabled_DuplicateDigest(t *testing.T) {
 	require.Equal(t, ErrAuthorityAlreadyDisabled, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_VerifyBlock_Ok(t *testing.T) {
 	babeService := createTestService(t, nil)
 	rt, err := babeService.blockState.GetRuntime(nil)
@@ -168,6 +176,8 @@ func TestVerificationManager_VerifyBlock_Ok(t *testing.T) {
 	require.NoError(t, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_VerifyBlock_Secondary(t *testing.T) {
 	babeService := createTestService(t, nil)
 	rt, err := babeService.blockState.GetRuntime(nil)
@@ -276,6 +286,8 @@ func TestVerificationManager_VerifyBlock_MultipleEpochs(t *testing.T) {
 	require.NoError(t, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_VerifyBlock_InvalidBlockOverThreshold(t *testing.T) {
 	babeService := createTestService(t, nil)
 	rt, err := babeService.blockState.GetRuntime(nil)
@@ -299,6 +311,8 @@ func TestVerificationManager_VerifyBlock_InvalidBlockOverThreshold(t *testing.T)
 	require.Equal(t, ErrVRFOutputOverThreshold, errors.Unwrap(err))
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerificationManager_VerifyBlock_InvalidBlockAuthority(t *testing.T) {
 	babeService := createTestService(t, nil)
 	rt, err := babeService.blockState.GetRuntime(nil)
@@ -367,6 +381,8 @@ func TestVerifyPimarySlotWinner(t *testing.T) {
 	require.True(t, ok)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerifyAuthorshipRight(t *testing.T) {
 	babeService := createTestService(t, nil)
 	epochData, err := babeService.initiateEpoch(testEpochIndex)
@@ -386,6 +402,8 @@ func TestVerifyAuthorshipRight(t *testing.T) {
 	require.NoError(t, err)
 }
 
+// TODO: add test against latest dev runtime
+// See https://github.com/ChainSafe/gossamer/issues/2704
 func TestVerifyAuthorshipRight_Equivocation(t *testing.T) {
 	kp, err := sr25519.GenerateKeypair()
 	require.NoError(t, err)