-
Notifications
You must be signed in to change notification settings - Fork 400
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
Karma tests setup for Angular #117
Comments
@jadjoubran Is there any work being done on this issue, it would be really nice to have tests in the next release. Thanks. |
@kujtimiihoxha unfortunately this is not planned for the next release since there are more critical features.. especially JWT and everything around it However feel free to submit a Pull Request for this |
@jadjoubran I have tried to implement karma tests but did not succeed, I will try again to implement this feature in the coming days, as soon as I have it ready I will submit a Pull Request . |
@jadjoubran look at PR #219 |
@jadjoubran Tell me if you have any tests in mind and I will implement them ;). |
Thanks again 😄 Actually I just remembered that it would make sense to setup ngDescribe on top of this.. since it removes all the boilerplate |
I will see what I can do, I will try to implement this today. |
@jadjoubran tests can now use ngDescribe ;) tell me what you think about the folder structure I proposed and I will implement that ;) |
@kujtimiihoxha thank you so much |
@jadjoubran All done, take a look at my commit, tell me if there is anything else needed. |
Current repo is missing setup for Angular unit tests using ngAnnotate
The text was updated successfully, but these errors were encountered: