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.

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

Add tests for various size limits for data types #586

Open
anton-trunov opened this issue Jul 21, 2024 · 2 comments · May be fixed by #817
Open

Add tests for various size limits for data types #586

anton-trunov opened this issue Jul 21, 2024 · 2 comments · May be fixed by #817
Assignees
Labels
Milestone

Comments

@anton-trunov
Copy link
Member

For instance, the docs say:

As the upper bound of the smart contract state size is around 65,000 items of type Cell, it constrains the storage limit of maps to be about 30,000 key-value pairs for the whole contract.

We should test doc statements like the above.

So, we need tests for at least the following items:

  • maximum map size
  • maximum cell number
  • maximum number of struct fields
@Gusarich
Copy link
Member

Gusarich commented Jul 21, 2024

Map size and the number of struct fields depend on the types of the fields/values. Also the max map size can vary depending on the keys due to how the patricia tree works under the hood.

@anton-trunov
Copy link
Member Author

Right, that is why we should just cover some typical use cases

@anton-trunov anton-trunov modified the milestones: v1.4.1, v1.5.0 Jul 25, 2024
@novusnota novusnota linked a pull request Sep 13, 2024 that will close this issue
7 tasks
@novusnota novusnota modified the milestones: v1.5.0, v1.6.0 Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants