The Wayback Machine - https://web.archive.org/web/20201103190339/https://github.com/TheAlgorithms/Java/pull/500
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 InsertionSort #500

Merged
merged 2 commits into from Aug 30, 2018
Merged

Conversation

@lq920320
Copy link

@lq920320 lq920320 commented Aug 29, 2018

No description provided.

Copy link
Collaborator

@varunu28 varunu28 left a comment

Can you remove print statements from the test

@lq920320
Copy link
Author

@lq920320 lq920320 commented Aug 29, 2018

@varunu28 Only this test or including other tests have print statements in my code?

@varunu28
Copy link
Collaborator

@varunu28 varunu28 commented Aug 29, 2018

@lq920320 Remove it from other ones too. I missed it in earlier reviews. We don't need a print statement in the tests. If it fails then Junit assertion generates enough logs for clarification

delete print statements from Test files.
@varunu28 varunu28 merged commit e4827bd into TheAlgorithms:Development Aug 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.