Issues in the piggy bank smart contract testing tutorial

[hamiidreza]

Description If you follow the tutorial for piggy bank smart contract testing here, you might get a compiling error saying the function TestHost::new() should take 2 arguments, but it takes only 1. In fact, the second argument "TestStateBuilder::new()" turns out to be missing in the tutorial. Further, the tutorial has only 4 tests, whereas the Github code has 8 tests including the following ones as well:

• test_insert_smashed()
• test_smash_account_missing()
• test_smash_smashed()
• test_view()

Steps to Reproduce

Expected Result

Actual Result

Versions

• Software Version
• OS
• Browser
• Mobile device, if applicable

[abizjak]

Regarding tests, we added some recently.

Would you be willing to fix the bugs in documentation though? The ones regarding TestStateBuilder::new?

[hamiidreza]

Sure. I will make a pull request with the fix by today.

Get Outlook for iOShttps://aka.ms/o0ukef

---

From: Aleš Bizjak @.> Sent: Friday, August 19, 2022 6:48:14 PM To: Concordium/concordium-rust-smart-contracts @.> Cc: Hamidreza @.>; Author @.> Subject: Re: [Concordium/concordium-rust-smart-contracts] Issues in the piggy bank smart contract testing tutorial (Issue #159)

Regarding tests, we added some recently.

Would you be willing to fix the bugs in documentation though? The ones regarding TestStateBuilder::new?

— Reply to this email directly, view it on GitHubhttps://github.com/Concordium/concordium-rust-smart-contracts/issues/159#issuecomment-1220884645, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANDEHVNKIKIPOT5LVAAXWVDVZ63E5ANCNFSM57A6NSOA. You are receiving this because you authored the thread.Message ID: @.***>

[hamiidreza]

done.