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

GETRANGE and GETRANGEHASH related tests #652

Open
evgeniiz321 opened this issue Sep 25, 2023 · 3 comments
Open

GETRANGE and GETRANGEHASH related tests #652

evgeniiz321 opened this issue Sep 25, 2023 · 3 comments
Assignees
Labels
feature Completely new functionality I3 Minimal impact S4 Routine U3 Regular

Comments

@evgeniiz321
Copy link
Contributor

Need to test assumptions about valid range value, e.g. requests like [X:0] should not be allowed. Need to take a closer look about other related tests as well.

@evgeniiz321 evgeniiz321 self-assigned this Sep 25, 2023
@roman-khimov roman-khimov added U3 Regular S4 Routine I3 Minimal impact feature Completely new functionality and removed testcases labels Dec 20, 2023
@cthulhu-rider
Copy link
Contributor

both 0 should be allowed and treated like full payload after nspcc-dev/neofs-node#3171

evgeniiz321 pushed a commit that referenced this issue Feb 27, 2025
closes #652

Signed-off-by: Evgeniy Zayats <zayatsevgeniy@nspcc.io>
@evgeniiz321
Copy link
Contributor Author

@cthulhu-rider

./neofs-cli --config /Users/evgeniiz/Projects/neofs-testcases-nspcc/neofs-testcases/wallet_config.yml object range --rpc-endpoint 'localhost:56016' --wallet '/Users/evgeniiz/Projects/neofs-testcases-nspcc/neofs-testcases/test-run-2025-02-27-17-48-46-358279/wallet-80546699-e4fa-474e-baf8-20983ccbc4c7.json' --cid '7jJXQEFv31e3Xe8SAoWpcHtKUQiFUPbGzg5mD7CPnnhW' --oid 'GNkZyWxZGKHXnxDAC3eMVATBYyDvyPPYW62ZvHnAbHnY' --range '0:0' --file '/Users/evgeniiz/Projects/neofs-testcases-nspcc/neofs-testcases/test-run-2025-02-27-17-48-46-358279/TestObjectsDir/01aaabc2-78dc-48ea-90e9-a3643ff4da1d'
Error:
return code: 1 
Output: can't get object payload range: init payload reading: zero range length

from latest master, is it a bug? or did I misunderstand something

@cthulhu-rider
Copy link
Contributor

cthulhu-rider commented Feb 27, 2025

./neofs-cli --version

? Must be fresh also

UPD: SDK version issue nspcc-dev/neofs-node#3182

evgeniiz321 pushed a commit that referenced this issue Feb 27, 2025
closes #652

Signed-off-by: Evgeniy Zayats <zayatsevgeniy@nspcc.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Completely new functionality I3 Minimal impact S4 Routine U3 Regular
Projects
None yet
Development

No branches or pull requests

3 participants