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

Add Tests on DFS Algorithms #58

Closed
ZigRazor opened this issue Sep 30, 2021 · 3 comments · Fixed by #375
Closed

Add Tests on DFS Algorithms #58

ZigRazor opened this issue Sep 30, 2021 · 3 comments · Fixed by #375
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest hacktoberfest issue help wanted Extra attention is needed Priority:Low Priority Label for low priority issue test Something about test
Milestone
In-Depth Tests

Comments

@ZigRazor
Copy link
Owner

There is the necessity to write more test on DFS Algorithm.
Specialy on limit case.
Any contribution is appreciated.
@ZigRazor ZigRazor added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers test Something about test Priority:Low Priority Label for low priority issue hacktoberfest hacktoberfest issue labels Sep 30, 2021
@ZigRazor ZigRazor added this to the In-Depth Tests milestone Sep 30, 2021
@ZigRazor ZigRazor moved this to Todo in @ZigRazor's issues Feb 25, 2022
@ErikDervishi03 ErikDervishi03 mentioned this issue Dec 13, 2023
Merged
@ErikDervishi03
Copy link
Contributor

i created a new pull request #375.If this test is ok, i can write some other tests if needed.
@ZigRazor

@ZigRazor
Copy link
Owner Author

@ErikDervishi03 good job!
Adding tests is always a good idea, so if you want, and you have time, you can add all the tests you want.
Obviously more the tests covers different branch of a function or code in general more the library is bug free.
So if possible try to test the so called "limit case".
Another good task is mentioned in the #3 , also write some simple or more complex example is useful.
Thank you in Advance!

@ZigRazor ZigRazor linked a pull request Dec 13, 2023 that will close this issue
added a new DFStest #375
Merged
@ZigRazor
Copy link
Owner Author

After this, I Close this issue

@github-project-automation github-project-automation bot moved this from Todo to Done in @ZigRazor's issues Dec 13, 2023
@ErikDervishi03 ErikDervishi03 mentioned this issue Dec 16, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest hacktoberfest issue help wanted Extra attention is needed Priority:Low Priority Label for low priority issue test Something about test
Projects
@ZigRazor's issues
Status: Done
Milestone
In-Depth Tests
Development

Successfully merging a pull request may close this issue.

added a new DFStest ErikDervishi03/CXXGraph
2 participants
@ZigRazor @ErikDervishi03