Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Merged
merged 15 commits into from
Aug 15, 2022
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
831e93f
Add v0.9.20 chain specs & configs
danforbes Apr 29, 2022
0b2d14a
re-add gssmr config.toml
timwu20 May 12, 2022
b1f738d
rename files back to old names
timwu20 May 12, 2022
d73826b
wip
timwu20 May 16, 2022
325243e
use raw methods for build spec command
timwu20 May 17, 2022
ddb2f33
fix(ci): fix `dot` unit tests (#2561)
timwu20 May 26, 2022
d49ab35
fix(ci): fix dot/network unit tests (#2572)
timwu20 Jun 2, 2022
d6e1735
fix(ci): fix dot/rpc/modules unit tests #2581
timwu20 Jun 8, 2022
4bcac72
fix(ci): embed old `gssmr` and `dev` runtimes for integration tests (…
timwu20 Aug 3, 2022
8748a19
remove debugging sleep
timwu20 Aug 9, 2022
e81751f
remove unneeded files from chains
timwu20 Aug 9, 2022
7d9c16f
fix up go.mod
timwu20 Aug 11, 2022
80ae03a
restore old go.mod
timwu20 Aug 11, 2022
38e052d
Apply suggestions from code review
timwu20 Aug 12, 2022
f47a4b4
add todo
timwu20 Aug 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
83 changes: 83 additions & 0 deletions chain/dev-staking/dev-staking-spec-raw.json
View file
Open in desktop

Large diffs are not rendered by default.

89 changes: 89 additions & 0 deletions chain/dev-staking/dev-staking-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

316 changes: 316 additions & 0 deletions chain/dev-v3substrate/genesis-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

57 changes: 57 additions & 0 deletions chain/dev-v3substrate/genesis.json
View file
Open in desktop

Large diffs are not rendered by default.

15 changes: 13 additions & 2 deletions chain/dev/config.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ digest = ""
genesis = "./chain/dev/genesis-spec.json"

[account]
key = "alice"
key = ""
qdm12 marked this conversation as resolved.
Show resolved Hide resolved
unlock = ""

[core]
Expand All @@ -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]
Expand Down
342 changes: 45 additions & 297 deletions chain/dev/genesis-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

100 changes: 45 additions & 55 deletions chain/dev/genesis.json
View file
Open in desktop

Large diffs are not rendered by default.

100 changes: 100 additions & 0 deletions chain/gssmr-staking/gssmr-staking-spec-raw.json
View file
Open in desktop

Large diffs are not rendered by default.

136 changes: 136 additions & 0 deletions chain/gssmr-staking/gssmr-staking-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

346 changes: 346 additions & 0 deletions chain/gssmr-v3substrate/genesis-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

56 changes: 56 additions & 0 deletions chain/gssmr-v3substrate/genesis.json
View file
Open in desktop

Large diffs are not rendered by default.

13 changes: 12 additions & 1 deletion chain/gssmr/config.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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]
Expand Down
396 changes: 77 additions & 319 deletions chain/gssmr/genesis-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

107 changes: 53 additions & 54 deletions chain/gssmr/genesis.json
View file
Open in desktop

Large diffs are not rendered by default.

160 changes: 160 additions & 0 deletions chain/westend-dev/westend-dev-spec-raw.json
View file
Open in desktop

Large diffs are not rendered by default.

205 changes: 205 additions & 0 deletions chain/westend-dev/westend-dev-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

176 changes: 176 additions & 0 deletions chain/westend-local/westend-local-spec-raw.json
View file
Open in desktop

Large diffs are not rendered by default.

224 changes: 224 additions & 0 deletions chain/westend-local/westend-local-spec.json
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion cmd/gossamer/import_runtime.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
bz, err := json.MarshalIndent(genesis, "", "\t")
if err != nil {
return "", err
Expand Down
2 changes: 1 addition & 1 deletion cmd/gossamer/import_runtime_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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))
}
2 changes: 1 addition & 1 deletion cmd/gossamer/main_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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),
qdm12 marked this conversation as resolved.
Show resolved Hide resolved
"--output", tmpOutputfile)

time.Sleep(5 * time.Second)
Expand Down
2 changes: 1 addition & 1 deletion cmd/gossamer/toml_config_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
6 changes: 3 additions & 3 deletions dot/build_spec_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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{}{},
Expand Down
7 changes: 5 additions & 2 deletions dot/core/service_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import (
"fmt"
"math/big"
"os"
"path/filepath"
"testing"
"time"

Expand Down Expand Up @@ -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)
qdm12 marked this conversation as resolved.
Show resolved Hide resolved
gen, err := genesis.NewGenesisFromJSONRaw(projectRootPath)
require.NoError(t, err)

Expand Down Expand Up @@ -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
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both these todos I am addressing

timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2702
func TestMaintainTransactionPool_EmptyBlock(t *testing.T) {
accountInfo := types.AccountInfo{
Nonce: 0,
Expand Down Expand Up @@ -465,6 +466,8 @@ func TestMaintainTransactionPool_EmptyBlock(t *testing.T) {
require.Nil(t, head)
}

// TODO: add test against latest gssmr runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2702
func TestMaintainTransactionPool_BlockWithExtrinsics(t *testing.T) {
accountInfo := types.AccountInfo{
Nonce: 0,
Expand Down
2 changes: 1 addition & 1 deletion dot/network/gossip_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
49 changes: 8 additions & 41 deletions dot/rpc/modules/childstate_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}

qdm12 marked this conversation as resolved.
Show resolved Hide resolved
mockStorageAPI := new(apimocks.StorageAPI)
mockErrorStorageAPI1 := new(apimocks.StorageAPI)
mockErrorStorageAPI2 := new(apimocks.StorageAPI)
Expand Down Expand Up @@ -128,7 +95,7 @@ func TestChildStateModule_GetKeys(t *testing.T) {
Key: []byte(":child_storage_key"),
},
},
exp: expStr,
exp: expHexKeys,
},
{
name: "Get Keys with Hash",
Expand All @@ -142,7 +109,7 @@ func TestChildStateModule_GetKeys(t *testing.T) {
Hash: &hash,
},
},
exp: expStr,
exp: expHexKeys,
},
{
name: "GetStorageChild error",
Expand Down
8 changes: 8 additions & 0 deletions dot/sync/chain_processor_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ import (
"github.com/stretchr/testify/require"
)

// TODO: add test against latest gssmr runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2703
func TestChainProcessor_HandleBlockResponse_ValidChain(t *testing.T) {
syncer := newTestSyncer(t)
responder := newTestSyncer(t)
Expand Down Expand Up @@ -79,6 +81,8 @@ func TestChainProcessor_HandleBlockResponse_ValidChain(t *testing.T) {
}
}

// TODO: add test against latest gssmr runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2703
func TestChainProcessor_HandleBlockResponse_MissingBlocks(t *testing.T) {
syncer := newTestSyncer(t)

Expand Down Expand Up @@ -154,6 +158,8 @@ func TestChainProcessor_HandleBlockResponse_NoBlockData(t *testing.T) {
require.Equal(t, ErrNilBlockData, err)
}

// TODO: add test against latest gssmr runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2703
func TestChainProcessor_HandleBlockResponse_BlockData(t *testing.T) {
syncer := newTestSyncer(t)

Expand Down Expand Up @@ -183,6 +189,8 @@ func TestChainProcessor_HandleBlockResponse_BlockData(t *testing.T) {
}
}

// TODO: add test against latest gssmr runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2703
func TestChainProcessor_ExecuteBlock(t *testing.T) {
syncer := newTestSyncer(t)

Expand Down
2 changes: 1 addition & 1 deletion dot/sync/syncer_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down
5 changes: 2 additions & 3 deletions dot/utils_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down Expand Up @@ -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 {
Expand Down
8 changes: 8 additions & 0 deletions lib/babe/build_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,8 @@ func createTestBlock(t *testing.T, babeService *Service, parent *types.Header,
return block
}

// TODO: add test against latest dev runtime
qdm12 marked this conversation as resolved.
Show resolved Hide resolved
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestBuildBlock_ok(t *testing.T) {
ctrl := gomock.NewController(t)
telemetryMock := NewMockClient(ctrl)
Expand Down Expand Up @@ -139,6 +141,8 @@ func TestBuildBlock_ok(t *testing.T) {
require.Equal(t, 1, len(extsBytes))
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestApplyExtrinsic(t *testing.T) {
ctrl := gomock.NewController(t)
telemetryMock := NewMockClient(ctrl)
Expand Down Expand Up @@ -233,6 +237,8 @@ func TestApplyExtrinsic(t *testing.T) {
require.NoError(t, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestBuildAndApplyExtrinsic(t *testing.T) {
ctrl := gomock.NewController(t)
telemetryMock := NewMockClient(ctrl)
Expand Down Expand Up @@ -406,6 +412,8 @@ func TestDecodeExtrinsicBody(t *testing.T) {
require.True(t, contains)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestBuildBlockTimeMonitor(t *testing.T) {
metrics.Enabled = true
metrics.Unregister(buildBlockTimer)
Expand Down
18 changes: 18 additions & 0 deletions lib/babe/verify_integration_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ func newTestVerificationManager(t *testing.T, genCfg *types.BabeConfiguration) *
return vm
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestVerificationManager_OnDisabled_InvalidIndex(t *testing.T) {
vm := newTestVerificationManager(t, nil)
babeService := createTestService(t, nil)
Expand All @@ -74,6 +76,8 @@ func TestVerificationManager_OnDisabled_InvalidIndex(t *testing.T) {
require.Equal(t, err, ErrInvalidBlockProducerIndex)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestVerificationManager_OnDisabled_NewDigest(t *testing.T) {
kp, err := sr25519.GenerateKeypair()
require.NoError(t, err)
Expand Down Expand Up @@ -111,6 +115,8 @@ func TestVerificationManager_OnDisabled_NewDigest(t *testing.T) {
require.NoError(t, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestVerificationManager_OnDisabled_DuplicateDigest(t *testing.T) {
kp, err := sr25519.GenerateKeypair()
require.NoError(t, err)
Expand Down Expand Up @@ -146,6 +152,8 @@ func TestVerificationManager_OnDisabled_DuplicateDigest(t *testing.T) {
require.Equal(t, ErrAuthorityAlreadyDisabled, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// 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)
Expand All @@ -168,6 +176,8 @@ func TestVerificationManager_VerifyBlock_Ok(t *testing.T) {
require.NoError(t, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// 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)
Expand Down Expand Up @@ -276,6 +286,8 @@ func TestVerificationManager_VerifyBlock_MultipleEpochs(t *testing.T) {
require.NoError(t, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// 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)
Expand All @@ -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
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// 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)
Expand Down Expand Up @@ -367,6 +381,8 @@ func TestVerifyPimarySlotWinner(t *testing.T) {
require.True(t, ok)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestVerifyAuthorshipRight(t *testing.T) {
babeService := createTestService(t, nil)
epochData, err := babeService.initiateEpoch(testEpochIndex)
Expand All @@ -386,6 +402,8 @@ func TestVerifyAuthorshipRight(t *testing.T) {
require.NoError(t, err)
}

// TODO: add test against latest dev runtime
timwu20 marked this conversation as resolved.
Show resolved Hide resolved
// See https://github.com/ChainSafe/gossamer/issues/2704
func TestVerifyAuthorshipRight_Equivocation(t *testing.T) {
kp, err := sr25519.GenerateKeypair()
require.NoError(t, err)
Expand Down
Loading