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

Devnet optimizations #61

Merged
merged 24 commits into from
Mar 28, 2022
Merged

Devnet optimizations #61

merged 24 commits into from
Mar 28, 2022

Conversation

dribeiro-ShardLabs
Copy link
Contributor

@dribeiro-ShardLabs dribeiro-ShardLabs commented Mar 23, 2022

  • Run Flask inside a proper container. This will improve how multiple requests are handled. Instead of being "piled up" increasing the time they take to process, the requests will now be received and treated sequentially.

  • Add new optional CLI argument --lite-mode. When this argument is passed, devnet will not calculate transaction hashes for deploy transactions, and will instead use a numeric sequence, and it turns off block generation as well.

  • test_cli and test_cli_auth were changed to pytest test files, in order to simplify unit test management;

  • test_postman was changed to a single test function, as the features are meant to be tested in a single flow

Related issue: #52

@FabijanC
Copy link
Collaborator

FabijanC commented Mar 25, 2022

Can you add dev changes to the PR description (like testing changes)?

@dribeiro-ShardLabs
Copy link
Contributor Author

Can you add dev changes to the PR description (like testing changes)?

Done

.pylintrc Show resolved Hide resolved
test/util.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
starknet_devnet/starknet_wrapper.py Outdated Show resolved Hide resolved
starknet_devnet/server.py Outdated Show resolved Hide resolved
starknet_devnet/util.py Show resolved Hide resolved
test/test_cli_auth.py Outdated Show resolved Hide resolved
starknet_devnet/starknet_wrapper.py Outdated Show resolved Hide resolved
starknet_devnet/starknet_wrapper.py Outdated Show resolved Hide resolved
test/test_cli.py Outdated Show resolved Hide resolved
test/test_general_workflow_lite.py Outdated Show resolved Hide resolved
test/test_dump.py Outdated Show resolved Hide resolved
test/test_dump.py Outdated Show resolved Hide resolved
starknet_devnet/util.py Outdated Show resolved Hide resolved
starknet_devnet/util.py Outdated Show resolved Hide resolved
starknet_devnet/util.py Outdated Show resolved Hide resolved
@FabijanC FabijanC merged commit 47d198f into master Mar 28, 2022
@FabijanC FabijanC deleted the optimizations branch March 28, 2022 11:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants