Tests/fin council #1070
Tests/fin council #1070
Conversation
|
|
||
| await expect(helper.finCouncil.collective.execute( | ||
| finCounselors.andy, | ||
| helper.democracy.vetoExternalCall(preimageHash), |
There was a problem hiding this comment.
А мы и в других тестах не проверяем, что 100% некого органа НЕ может ветировать демократию?
There was a problem hiding this comment.
В тех комитете мы проверяем что member может вето
В консулах вето демократии не проверяется вообще.
There was a problem hiding this comment.
Добавил для фин консулов и консулов
| )).to.be.rejectedWith('BadOrigin'); | ||
| }); | ||
|
|
||
| itSub('[Negative] FinCouncil can\' cancel Fellowship referendums', async ({helper}) => { |
| .rejectedWith('BadOrigin'); | ||
| }); | ||
|
|
||
| itSub('[Negative] TechComm member cannot cancel Fellowship referendums', async ({helper}) => { |
There was a problem hiding this comment.
Not TechComm but a member of fin council
| )).to.be.rejectedWith('BadOrigin'); | ||
| }); | ||
|
|
||
| itSub('[Negative] FinCouncil member can\' add a Fellowship member', async ({helper}) => { |
| await clearTechComm(sudoer); | ||
| }); | ||
|
|
||
| async function proposalFromMoreThanHalfCouncil(proposal: any) { |
There was a problem hiding this comment.
proposalFromMoreThanHalfFinCouncil
и всех подобных случаев. Все-таки обычный Council -- это именно Совет. А финансовый -- это отдельная сущность, все-таки. Можно легко запутаться, если это не разделять
|
|
||
| itSub('[Negative] FinCouncil cannot add/remove a TechComm member', async ({helper}) => { | ||
| const newCommMember = helper.arrange.createEmptyAccount(); | ||
| const addMemberProposal = helper.council.membership.addMemberCall(newCommMember.address); |
There was a problem hiding this comment.
Не тот коллектив! Надо поправить -- тут надо проверять тех коммитет
|
|
||
| itSub('[Negative] FinCouncil member cannot add/remove a TechComm member', async ({helper}) => { | ||
| const newCommMember = helper.arrange.createEmptyAccount(); | ||
| const addMemberProposal = helper.council.membership.addMemberCall(newCommMember.address); |
| }); | ||
|
|
||
|
|
||
| }); |
There was a problem hiding this comment.
Надо бы добавить тест на то, что ФинСовет/Член ФинСовета не может добавлять/удалять членов ФинСовета. Тоже самое про прайм мембера ФинСовета
There was a problem hiding this comment.
По идее надо еще добавить тест в ТехКомм, что он не может добавлять членов ФинСовета.
Но это уже по желанию и с одобрения @uandysmith
There was a problem hiding this comment.
По идее надо еще добавить тест в ТехКомм, что он не может добавлять членов ФинСовета.
такой есть
There was a problem hiding this comment.
Надо бы добавить тест на то, что ФинСовет/Член ФинСовета
Такой тоже есть. Прайм мембера добавлю
No description provided.