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

-t supports text types and can set the number of concurrencies. #15

Merged
merged 6 commits into from
Aug 16, 2024
Merged

Conversation

iteyelmp
Copy link
Collaborator

@iteyelmp iteyelmp commented Aug 8, 2024

Change

  1. The --type parameter supports string calldata and blob in addition to 1 and 2.
  2. Upload adds a new parameter to specify the number of concurrent uploads.
  3. Update Readme.

Local test:

  1. git clone https://github.com/ethstorage/ethfs-cli.git
  2. git checkout -b new origin/type
  3. yarn install
  4. npm link

Test case:

On a chain that supports Blob:

  1. Create a contract.
  2. Set default file
  3. Use calldata to upload file.
  4. Use blob to upload file.
  5. Use calldata to upload files in a folder
  6. Use blob to upload files in a folder
  7. After uploading part of the calldata or blob, terminate the upload task and restart uploading again to test the speed of comparing the uploaded files.
  8. Download file.
  9. If all functions are normal, the test passes.

On a chain that does not support Blobs.

  1. Create a contract.
  2. Set default file
  3. Use calldata to upload file.
  4. Use calldata to upload the folder
  5. After uploading some files, terminate the upload task and restart uploading again.
  6. Download file.
  7. If all functions are normal, the test passes.

@iteyelmp iteyelmp changed the title -t supports calldata and blob -t supports text types and can set the number of concurrencies. Aug 13, 2024
@iteyelmp iteyelmp merged commit f0a2347 into main Aug 16, 2024
@iteyelmp iteyelmp deleted the type branch August 16, 2024 02:24
README.md Show resolved Hide resolved
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.

3 participants