Skip to content
This repository has been archived by the owner on Jan 26, 2023. It is now read-only.

calling account.create and account.addFeatures with empty features should Error #169

Closed
tt0mmy opened this issue Jul 14, 2022 · 1 comment · Fixed by #172
Closed

calling account.create and account.addFeatures with empty features should Error #169

tt0mmy opened this issue Jul 14, 2022 · 1 comment · Fixed by #172
Assignees
Labels
bug Something isn't working

Comments

@tt0mmy
Copy link

tt0mmy commented Jul 14, 2022

Describe the bug
Per spec https://github.com/many-protocol/specification/blob/main/attributes/network/cddl/9_account.cddl#L96 and https://github.com/many-protocol/specification/blob/main/attributes/network/cddl/9_account.cddl#L248, calling account.create and account.addFeatures should require at least one feature in the set.

To Reproduce
Steps to reproduce the behavior:

  1. call account.create or account.addFeatures with an empty set of features.
  2. observe that no error is returned

Expected behavior
An error should be returned if an empty set of features is provided.

Additional context
probably need changes here https://github.com/liftedinit/many-framework/blob/main/src/many-ledger/src/module.rs#L49

@tt0mmy tt0mmy changed the title calling account.create without any features should throw calling account.create without any features should Error Jul 14, 2022
@tt0mmy tt0mmy added the bug Something isn't working label Jul 14, 2022
@tt0mmy tt0mmy changed the title calling account.create without any features should Error calling account.create and account.addFeatures features should Error Jul 15, 2022
@fmorency
Copy link
Contributor

#172

@tt0mmy tt0mmy changed the title calling account.create and account.addFeatures features should Error calling account.create and account.addFeatures with empty features should Error Jul 18, 2022
fmorency added a commit that referenced this issue Jul 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants