Improved unit-test coverage from 0 to 60+ with keploy AI-agent #744
Conversation
There was a problem hiding this comment.
Thanks for your contribution! I'm being somewhat reluctant about this, though. While I appreciate the fact that you added tests, many of them are quite trivial. That's not a problem by itself, but what I particularly dislike about your contribution is the fact that code was generated by an AI. I might be old-fashioned in that regard, but letting an LLM run over a code base and contributing its output is not the purpose of open source in my opinion.
While I'll merge your PR (after you implemented the requested changes) for now, I'll probably update the contribution guidelines accordingly for the future.
| } | ||
|
|
||
| // Mock implementation of SecureCookieDecoder for testing purposes. | ||
| type mockSecureCookie struct { |
There was a problem hiding this comment.
Please don't mix mocks with actual application code. Perhaps put them inside the mocks directory? Testing code and business logic should be strictly separated.
Actually, I don't think the tests that make use of the mockResponseWriter are very sensible at all. They're super trivial and while I'd vote for testing actual application logic, I'm not a big fan of blowing up the code base just for the sake of test coverage. I'd probably get rid of the http_test.go entirely, sorry.
|
Hey @muety Noted, let me make the changes and do let me know if that would work |
|
Any updates on this? |
|
Hey Made the changes here, #753 |
Hey, I’ve added new unit tests to the Go codebase while testing the capabilities of my Keploy AI testing agent. The tests ensure reliable coverage by validating code builds, eliminating flaky tests, and improving overall test stability. Here’s what the AI checks for:
Ensures new tests build without errors.
Confirms no flaky tests are introduced.
Enhances coverage for previously untested areas.
Coverage Breakdown:
Total Coverage Increase : 13%
github.com/muety/wakapi/routes/relaygithub.com/muety/wakapi/helpers