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

added create job tests #36

Open

M-Daeva wants to merge 1 commit into terra-money:master from M-Daeva:create-job-tests

M-Daeva commented Jul 9, 2023 •

edited

Loading

Hey! Let me make small contribution 😃

Added create job tests
Changed msgs_valid() usage because "return MsgError" is unreachable code as msgs_valid() never returns false
Implemented test_create_job_not_enough_funds_in_account() instead of test_create_job_no_funds_in_account() as it's wider requirement
Not sure if I added proper checking for test_create_job_invalid_condition()
Slightly linted files I modified

@vladjdk I hope you saw my job application 😉


          added create job tests

94c0754

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

Labels

None yet