Skip to content
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

Added unit tests for APIs, Secure App and AuthContext hook #263

Open
wants to merge 68 commits into
base: main
Choose a base branch
from

Conversation

pranshustuff
Copy link

@pranshustuff pranshustuff commented Oct 29, 2024

Purpose

Fixes #138

Goals

The tests automate unit test for pull requests so code isn't merged without making sure all the main parts of the code still function as intended.

Approach

I used Jest and React Testing library to mock various elements of the use cases and just made sure the functions returned the expected results.

NOTE: The github actions integration part is not complete yet. You can find find it in the tests. yml file but The tests folder works perfectly.

@pranshustuff
Copy link
Author

I'm not really sure what you mean by "Send a follow up PR Test phase to the PR builder and the Release builder once"

@pranshustuff
Copy link
Author

pranshustuff commented Nov 1, 2024

I'm sorry I made so many commits, I didn't realize it was to the main branch.

Anyway, everything works now.
The only problem is that I made a pull request and there are no issues in my fork, so the notifications.yml ran into an error with the "send notification on issue creation". Which shouldn't be a problem on your side.

About the follow-up PR, I can't really create a new one since I've accidentally committed everything to this branch. I hope you see this comment. And I'll be waiting for feedback.

Hope to hear from you soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write unit tests for the SDK and integrate with Github actions
2 participants