-
Notifications
You must be signed in to change notification settings - Fork 39
Conversation
WIP: 3 more things to do
Co-authored-by: Anton Trunov <anton.a.trunov@gmail.com>
What needs to be done here to make it "ready for review"? |
@anton-trunov those sub-headings & their inner content: I'll be finishing those today, shortly after I finish with Sublime updates. Hopefully, you'll have a bunch of stuff to review tomorrow morning :) UPD: Made a lot for Sublime today, don't know if I'll finish this PR today too. I'll try, though. |
In the testing/debugging page we should also cover how to test if a contract's receiver throws an exception. Here is a recent user request:
|
Sandbox/Blueprint analogue would be to write something like this in tests: expect(res.transactions).toHaveTransaction({
success: false,
exitCode: 5 // or whatever exit code is desired / undesired :)
}); Where |
And one more popular request regarding testing: how to simulated time passing |
Hmm, I suppose simple "sleep/wait" will do. And, maybe, adding a separate test group, which won't overwrite the state of the blockchain between the individual tests in that group. On it! |
Wait, no I meant
|
Oh, I see. Sure! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tremendous improvement!
Let's add dump
usage example to this page, though. Maybe modify the play
receiver and show how to observe the results of dumping. I believe we don't have these dump logs anywhere in the Tact docs
Added a usage example to "Common debugging functions", right before the "Enabling debug mode ..." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great stuff!
Let's publish! |
Closes #4
WIP, but almost there.
Preview:
