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

Max width is not properly during dry run transaction trace #592

Closed
joe-p opened this issue Jun 22, 2022 · 0 comments · Fixed by #593
Closed

Max width is not properly during dry run transaction trace #592

joe-p opened this issue Jun 22, 2022 · 0 comments · Fixed by #593
Labels
new-bug Bug report that needs triage

Comments

@joe-p
Copy link
Contributor

joe-p commented Jun 22, 2022

Subject of the issue

maxValueWidth of the StackPrinterConfig doesn't seem to work.

Your environment

algosdk version 1.18.0

Steps to reproduce

  1. Call console.log(txn.appTrace({maxValueWidth: 5000, topOfStackFirst: false}))

Expected behaviour

The max width of values would be increased from the default width

Actual behaviour

Values are still truncated with the default width:

pc# |ln# |source                            |scratch |stack
1   |1   |pushbytes 0x6d657373616765 // ... |        |[]
10  |2   |pushbytes 0x68656c6c6f20776f72... |        |[0x6d657373616765]
24  |3   |app_global_put                    |        |[0x6d657373616765, 0x68656c6c6...
25  |4   |pushbytes 0x746869732069732061... |        |[]
59  |5   |log                               |        |[0x7468697320697320616e20696d7...
60  |6   |pushint 1                         |        |[]
62  |7   |return                            |        |[1]
63  |8   |                                  |        |[1]

As a sanity check I made sure that the spc object was getting passed correctly by changing topOfStackFirst which does indeed work. It seems to be a problem specific to the maxValueWidth

@joe-p joe-p added the new-bug Bug report that needs triage label Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-bug Bug report that needs triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant