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

Write unit tests for functions #8

Closed
abskrj opened this issue Dec 26, 2022 · 10 comments
Closed

Write unit tests for functions #8

abskrj opened this issue Dec 26, 2022 · 10 comments
Assignees
@uzaxirr
Labels
good first issue Good for newcomers

Comments

@abskrj
Copy link
Member

abskrj commented Dec 26, 2022

We need to write UTs for all the functions, whose UTs are not written yet.

You can even raise PR with at least 1 UT of a function.
@abskrj abskrj added the good first issue Good for newcomers label Dec 26, 2022
@SamantaTarun
Copy link

I would like to contribute to this issue. Please assign it to me.

@abskrj
Copy link
Member Author

abskrj commented Dec 29, 2022

@tarunsamanta2k20 still working?

@SamantaTarun
Copy link

@abhishekraj272 I am busy with my own work. Please assign it to someone else.

@SamantaTarun SamantaTarun removed their assignment Dec 29, 2022
@aloner-pro
Copy link

Can you show some examples where unit tests are already written?

@abskrj
Copy link
Member Author

abskrj commented Dec 30, 2022

@aloner-pro https://www.digitalocean.com/community/tutorials/how-to-write-unit-tests-in-go-using-go-test-and-the-testing-package

@abskrj
Copy link
Member Author

abskrj commented Dec 30, 2022

Let me know if you are working on it, will assign this to you

@uzaxirr
Copy link
Member

uzaxirr commented Dec 30, 2022

@aloner-pro
This is also a good reference for that same, Do let us know if you doesn’t find this issue suitable for you

@aloner-pro
Copy link

Currently looking into it.

@uzaxirr
Copy link
Member

uzaxirr commented Dec 31, 2022

Currently looking into it.

I'll be taking this up as it needs to be done on priority and it also acts as an blocker for #16

@uzaxirr uzaxirr self-assigned this Dec 31, 2022
@uzaxirr
Copy link
Member

uzaxirr commented Jan 1, 2023

closed via #21

@uzaxirr uzaxirr closed this as completed Jan 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@uzaxirr uzaxirr

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@abskrj @SamantaTarun @aloner-pro @uzaxirr