Add proper test to Backtracking/Sudoku#713
Add proper test to Backtracking/Sudoku#713pomkarnath98 merged 3 commits intoTheAlgorithms:masterfrom defaude:test/Backtracking/Sudoku
Conversation
This replaces the "main" test in the implementation file.
|
If the algorithm is working fine, then no need to modify it. Someone from the team has already approved and merged by reviewing it. |
|
Hi @pomkarnath98! I did not change the original implementation. I only removed the test / driver code from the implementation file and replaced it with a proper (more in-depth) test case. Why would you dis-encourage people to contribute to already-existing implementations? :( The contribution doc explicitly encourages that. |
|
@pomkarnath98 The repositories of TheAlgorithms are for educational purposes. Here are the reasons why tests are useful even for perfectly coded algorithms:
|
|
@defaude Please format your code with |
|
@defaude I'm really thankful that people like you are getting time to contribute to open sources. Your points are welcome @raklaptudirm. |
Describe your change:
Refactored the test for the Backtracking/Sudoku algorithm out into a proper jest test file.
Checklist:
Example:
UserProfile.jsis allowed butuserprofile.js,Userprofile.js,user-Profile.js,userProfile.jsare notFixes: #{$ISSUE_NO}.