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

Add test for create_new_game() function #187

Closed
0xibs opened this issue Jan 9, 2025 · 6 comments · Fixed by #193
Closed

Add test for create_new_game() function #187

0xibs opened this issue Jan 9, 2025 · 6 comments · Fixed by #193
Assignees
Labels
difficulty:medium Has medium difficulty onchain

Comments

@0xibs
Copy link
Collaborator

0xibs commented Jan 9, 2025

Description
Add test for create_new_game() function located in game_actions.cairo. Also test all the assertions used in the function.

Expected to be completed within 2 days

@0xibs 0xibs added the onchain label Jan 9, 2025
@emarc99
Copy link
Contributor

emarc99 commented Jan 9, 2025

Can I do this? ETA 2days
Currently contributing to dojo here

@0xibs 0xibs added the difficulty:medium Has medium difficulty label Jan 9, 2025
@Birdmannn
Copy link

Hey dev, can I handle this?

I have written dojo code and I have worked with Marquis gaming ludo. I ensure you I can carry out these dojo tests

My environment have already been set up for dojo 1.0.9
ETA 2 days

@blessingbytes
Copy link
Contributor

Can I take up this test?

@No-bodyq
Copy link

No-bodyq commented Jan 9, 2025

Hello @princeibs can I take this issue

Copy link

co-lead bot commented Jan 9, 2025

Thank you @emarc99 for your interest in contributing to sivicstudio/starkludo. Endeavour to create a pull request before Sat, 11 Jan 2025 21:08:34 GMT so as not to get unassigned.

@emarc99
Copy link
Contributor

emarc99 commented Jan 11, 2025

Dropping my PR within the hour sir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty:medium Has medium difficulty onchain
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants