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.

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

Test update land and reject land #50

Open
mubarak23 opened this issue Oct 12, 2024 · 7 comments · May be fixed by #115
Open

Test update land and reject land #50

mubarak23 opened this issue Oct 12, 2024 · 7 comments · May be fixed by #115
Assignees
@Kingsuite
Labels
help wanted Extra attention is needed ODHack9 test

Comments

@mubarak23
Copy link

mubarak23 commented Oct 12, 2024 •
edited by fishonamos
Loading

Test the following functions:

Update land
reject land

Implement and test logic for add_inspector and and remove inspector
@fishonamos fishonamos changed the title Unit test cases for register_land function Setup test with snfoundry and test register_land function. Oct 12, 2024
@fishonamos fishonamos added help wanted Extra attention is needed test labels Oct 12, 2024
@fishonamos fishonamos reopened this Oct 15, 2024
@Akshola00
Copy link
Contributor

@mubarak23 please assign this to me

@juandiegocv27
Copy link

Hi @mubarak23, please assign me this issue, it is available? could you give a better description please, I really like this project😎

@fishonamos fishonamos changed the title Setup test with snfoundry and test register_land function. refactor register_land function Oct 24, 2024
@fishonamos fishonamos changed the title refactor register_land function Test update land and reject land Oct 25, 2024
@Yunusabdul38
Copy link

Let me handle this issue!

@SoarinSkySagar
Copy link
Contributor

SoarinSkySagar commented Oct 25, 2024 •
edited
Loading

I already have experience in working on this codebase (including the tests), and would very much like to work on this one after the blocking PR has been merged.
Will have multiple testcases and make sure every one of the tests pass for the 4 given functions

ETA: 6 Hours of being assigned

@jaiminRaiyani
Copy link

I'd be happy to do this.

@sandragcarrillo
Copy link

Could I take a shot at this?

@Kingsuite
Copy link

kindly assign me this task as am a seasoned developer with experience in writing and testing smart contracts using Cairo and solidity. kindly assign me this task ETA 2-3days

@Kingsuite Kingsuite linked a pull request Oct 31, 2024 that will close this issue
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Kingsuite Kingsuite

Labels
help wanted Extra attention is needed ODHack9 test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: update test for reject_land and update fn Kingsuite/landver
9 participants
@mubarak23 @fishonamos @sandragcarrillo @jaiminRaiyani @juandiegocv27 @Yunusabdul38 @Akshola00 @SoarinSkySagar @Kingsuite