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.

Sign up for GitHub

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

Jump to bottom

[Add test result] Need to add the length limitation for the test-result field in the add-test-result command. #3

Closed
julia-aksyanova opened this issue Sep 2, 2020 · 1 comment
Closed

[Add test result] Need to add the length limitation for the test-result field in the add-test-result command. #3

julia-aksyanova opened this issue Sep 2, 2020 · 1 comment
Assignees
@DenisRybas
Projects
DCL
Milestone
v0.7: TestNet for...

Comments

@julia-aksyanova
Copy link
Collaborator

Severity:
enhancement

Build info:
0505970

Overview:
Need to add the length limitation for the test-result field in the add-test-result command (
zblcli tx compliancetest add-test-result --vid= --pid= --test-result= --test-date= --from=).
Actually, the test-result field is not limited.
It is suggested to make the next limitation that must be discussed:

  • the maximum available length for the test-result field is near 255-2500 symbols.

If the mentioned parameter contains 5000 symbols, you get the following error as a result of command execution.
image
@julia-aksyanova julia-aksyanova changed the title [Add model info] Need to add the length limitation for the test-result field in the add-test-result command. [Add test result] Need to add the length limitation for the test-result field in the add-test-result command. Sep 2, 2020
@ashcherbakov ashcherbakov added this to the v0.3 milestone Sep 10, 2020
@ashcherbakov ashcherbakov added this to Backlog in DCL Sep 10, 2020
adenishchenko pushed a commit that referenced this issue Nov 6, 2020
Fix linter errors #3
773693e
ashcherbakov added a commit that referenced this issue Nov 9, 2020
@adenishchenko @ashcherbakov
Fixed revoke for invalid vid & pid (#73)
987b718 
* Fixed revoke for invalid `vid` & `pid`

* Fixed linter errors

* FIx linter errors #2

* fix

* Fix linter errors #3

* Fix linter errors #4

* Fixed integration tests, linter issues

* Fixed linter issue

Co-authored-by: anton <anton@DenishchenkoA.vi.ubuntu.dsr-corporation.com>
Co-authored-by: ashcherbakov <alexander.sherbakov@dsr-corporation.com>
@ashcherbakov ashcherbakov removed this from the v0.3 milestone Nov 1, 2021
@ashcherbakov ashcherbakov moved this from Backlog to To do in DCL Jan 24, 2022
@ashcherbakov ashcherbakov moved this from To do to In progress in DCL Jan 27, 2022
@ashcherbakov
Copy link
Contributor

PR: #199

DCL automation moved this from In progress to Done Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DenisRybas DenisRybas

Labels
None yet
Projects
No open projects
DCL
  
Done
Milestone
v0.7: TestNet for latest Cosmos SDK
Development

No branches or pull requests
3 participants
@ashcherbakov @DenisRybas @julia-aksyanova