-
-
Notifications
You must be signed in to change notification settings - Fork 533
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
[test]: add unit tests for octokit module #354
[test]: add unit tests for octokit module #354
Comments
Hey where and how am i supposed to write this tests |
Hey @re-Tick I was thinking of using jest to write unit test cases for the function also do we need to add it in build time script? |
Yes, we can use jest for unit-test and we have to add a test job in github action |
Unit tests have to be added for octokit module. You can use jest or motcha for writing unit tests. |
Is this issue being worked upon by someone or is it available? @re-Tick |
The issue is open for all. You can propose your changes in a PR and mention this issue |
@re-Tick i have updated new files in typescript sdk please review it .i had to delete the last PR because of some coding mistakes |
Closing the issue as we dont support sdks nay more. |
Is there an existing feature request for this?
Summary
Currently, there are no test written in ts-sdk. And since ts-sdk does not have a application server therefore, we need to write unit tests for code coverage.
Write a unit test for integrations/octokit module's wrappedNodeFetch function. This will increase coverage for this module.
Why should this be worked on?
There should be unit tests to check the behaviour of modules individually.
The text was updated successfully, but these errors were encountered: