Skip to content

Fix Tests in Simulator#28

Open
zkclay wants to merge 4 commits into
ProvableHQ:mainfrom
zkclay:fix_test_after_rename
Open

Fix Tests in Simulator#28
zkclay wants to merge 4 commits into
ProvableHQ:mainfrom
zkclay:fix_test_after_rename

Conversation

@zkclay

@zkclay zkclay commented Sep 15, 2024

Copy link
Copy Markdown

Hey all, I was taking a look at your testing approach for Aleo programs, and I wanted to contribute my first PR!

In this PR, I made a few different changes in order to for all the tests to pass in the simulator projects. I have summarized the the changes below:

  • Made a bunch of rename changes to simulator/src/contracts/pondoProgramsIndex.ts in order to track the renaming that was done in previous commits.
  • Fixed an issue in the token registry tests where the code for enforcing the initialize only once test was present in the leo program, but not the javascript.
  • Fixed an issue in the token_registry tests where the balances map key was encoded using JSON.stringify on the struct, but was decoded using .to_string() , resulting in the key not being found.
  • Fixed an issue in the oracle test where toSpliced(rank, 1) what throwing errors in my environment becuase it is newer javascript syntax, i replicated the functionality using older syntax.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants