http-request-translator icon indicating copy to clipboard operation
http-request-translator copied to clipboard
verified publisher icon owtf

Add tests for translator.py

Open dhruvagarwal opened this issue 9 years ago • 4 comments

  • [x] add tests for process_arguments
  • [ ] add tests for take_body

These two methods cover a big part of the module. Right now, the test coverage is 77%.

dhruvagarwal avatar Mar 05 '16 15:03 dhruvagarwal

@dhruvagarwal Is this ready to review? Or should I wait until the second item in your list is checked?

DePierre avatar Mar 15 '16 22:03 DePierre

@DePierre yes, this is ready for review. I'll add the tests for take_body in another pull

dhruvagarwal avatar Mar 21 '16 20:03 dhruvagarwal

@dhruvagarwal Do you think that is it important to test for other languages as well? I see that you used bash for the unit tests. Would it be interesting to use the other supported languages as well?

DePierre avatar Apr 06 '16 12:04 DePierre

@dhruvagarwal Are you still concerned by your PR?

DePierre avatar Apr 18 '16 16:04 DePierre