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

feat: provide smoke-test command #10

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Conversation

jacderida
Copy link
Contributor

@jacderida jacderida commented Sep 21, 2023

This command performs a basic smoke test on the deployed network. We upload ten small files, increasing in size from 1k to 10k, and the re-download them all and make sure we got the same files back.

In the process of doing this, we are also testing that the faucet is working and we get a connection to the network.

This test only takes something like 30 seconds, whereas uploading the full test data can take 30+ minutes.

I tried to unit test this, but because of the files it generates, it was just too much hassle and probably not worth it to change the design to pass in a temporary location to read/write files from.

This command performs a basic smoke test on the deployed network. We upload ten small files,
increasing in size from 1k to 10k, and the re-download them all and make sure we got the same files
back.

In the process of doing this, we are also testing that the faucet is working and we get a connection
to the network.

This test only takes something like 30 seconds, whereas uploading the full test data can take 30+
minutes.
@joshuef
Copy link
Contributor

joshuef commented Sep 21, 2023

This test only takes something like 30 seconds, whereas uploading the full test data can take 30+ minutes.

How big is the full test data?

@jacderida jacderida merged commit 65011bd into maidsafe:main Sep 21, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants