Skip to content

Commit 9926a62

Browse files
committed
fix: vote items in tests
1 parent 3bd3f0f commit 9926a62

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

tests/test_before_pectra_upgrade.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ def test_vote(helpers, accounts, vote_ids_from_env, bypass_events_decoding):
7272
assert vebo.getConsensusVersion() == 2
7373
# 7),9) Aragon agent doesnt have MANAGE_CONSENSUS_VERSION_ROLE on CS Fee oracle
7474
check_aragon_doesnt_have_manage_consensus_role_on_oracle(cs_fee_oracle)
75-
# 8) Update vebo consensus version equals to 1
75+
# 8) CS fee oracle consensus version equals to 1 before voting
7676
assert cs_fee_oracle.getConsensusVersion() == 1
7777
# 10) Old CS Verifier has VERIFIER_ROLE role on CSM before voting
7878
assert csm.hasRole(VERIFIER_ROLE, CS_VERIFIER_ADDRESS_OLD)
@@ -103,7 +103,7 @@ def test_vote(helpers, accounts, vote_ids_from_env, bypass_events_decoding):
103103
assert vebo.getConsensusVersion() == 3
104104
# 7),9) Aragon agent doesnt have MANAGE_CONSENSUS_VERSION_ROLE on CS Fee oracle
105105
check_aragon_doesnt_have_manage_consensus_role_on_oracle(cs_fee_oracle)
106-
# 8) Update vebo consensus version equals to 2 after voting
106+
# 8) CS fee oracle consensus version equals to 2 after voting
107107
assert cs_fee_oracle.getConsensusVersion() == 2
108108
# 10) Old CS Verifier has VERIFIER_ROLE role on CSM after voting
109109
assert not csm.hasRole(VERIFIER_ROLE, CS_VERIFIER_ADDRESS_OLD)

tests/test_before_pectra_upgrade_holesky.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,11 @@ def test_vote(helpers, accounts, vote_ids_from_env, bypass_events_decoding, stra
125125
assert vebo.getConsensusVersion() == 2
126126
# 7),9) Aragon agent doesnt have MANAGE_CONSENSUS_VERSION_ROLE on CS Fee oracle
127127
check_aragon_doesnt_have_manage_consensus_role_on_oracle(cs_fee_oracle)
128-
# 8) Update vebo consensus version equals to 1
128+
# 8) CS Verifier consensus version equals to 1
129129
assert cs_fee_oracle.getConsensusVersion() == 1
130130
# 10) Old CS Verifier has VERIFIER_ROLE role on CSM before voting
131131
assert csm.hasRole(VERIFIER_ROLE, CS_VERIFIER_ADDRESS_OLD)
132-
# 11) New CS Verifier doesnt have VERIFIER_ROLE role on CSM before voting
132+
# 11) New CS Verifier doesn't have VERIFIER_ROLE role on CSM before voting
133133
assert not csm.hasRole(VERIFIER_ROLE, CS_VERIFIER_ADDRESS)
134134

135135
# On-chain voting duration state before voting
@@ -185,7 +185,7 @@ def test_vote(helpers, accounts, vote_ids_from_env, bypass_events_decoding, stra
185185
assert vebo.getConsensusVersion() == 3
186186
# 7),9) Aragon agent doesnt have MANAGE_CONSENSUS_VERSION_ROLE on CS Fee oracle
187187
check_aragon_doesnt_have_manage_consensus_role_on_oracle(cs_fee_oracle)
188-
# 8) Update vebo consensus version equals to 2 after voting
188+
# 8) CS Verifier consensus version equals to 2 after voting
189189
assert cs_fee_oracle.getConsensusVersion() == 2
190190
# 10) Old CS Verifier doesn't have VERIFIER_ROLE role on CSM after voting
191191
assert not csm.hasRole(VERIFIER_ROLE, CS_VERIFIER_ADDRESS_OLD)

0 commit comments

Comments
 (0)