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.

Sign up for GitHub

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

Jump to bottom

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

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

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

wants to merge 69 commits into from
+23,279 −1,362

Conversation

pranshustuff
Copy link

@pranshustuff pranshustuff commented Oct 29, 2024 •
edited
Loading

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.

pranshustuff and others added 30 commits October 20, 2024 07:39
@pranshustuff
added test cases
8797e09
@pranshustuff
Merge pull request #1 from pranshustuff/new_branch
4d7b7dd 
added test cases
@pranshustuff
added signIn test
8f99e07
@pranshustuff
Create test.yml
f831a3f
@pranshustuff
Update test.yml
e1a66b3
@pranshustuff
Update test.yml
26dec9b
@pranshustuff
Update test.yml
af50d1c
@pranshustuff
Update test.yml
1d8cb6b
@pranshustuff
Update test.yml
c6ef26f
@pranshustuff
Update test.yml
089609b
@pranshustuff
Update test.yml
ff0c713
@pranshustuff
Update test.yml
4edd954
@pranshustuff
Update builder.yml
4c87a38
@pranshustuff
Update test.yml
951a533
@pranshustuff
Update test.yml
456fac7
@pranshustuff
Update test.yml
48919b3
@pranshustuff
added sign Out test
5fd26a5
@pranshustuff
modified signOut
7025a7d
@pranshustuff
test yml file
16a32b0
@pranshustuff
yarn lock change
1081f51
@pranshustuff
added test for isAuthenticated
ec3288e
@pranshustuff
added isAuth test to github actions
6ff681f
@pranshustuff
Update builder.yml
16a9773
@pranshustuff
added test for getBasicUserInfo
286f5fd
@pranshustuff
Merge branch 'main' of https://github.com/pranshustuff/asgardeo-auth-…
1d99e60 
…react-sdk
@pranshustuff
modified test.yml
45995ed
@pranshustuff
added test for refreshAccessToken
a434400
@pranshustuff
added accesstoken test in test.yml
2fb5b9c
@pranshustuff
tsconfig changes
2d46316
@pranshustuff
added new tests
ecd9cbc
pranshustuff and others added 27 commits November 1, 2024 21:22
@pranshustuff
Update test.yml
fd6c5f0
@pranshustuff
Update test.yml
7a2808c
@pranshustuff
Update test.yml
8ea011c
@pranshustuff
Update test.yml
edff456
@pranshustuff
Update test.yml
824972a
@pranshustuff
Update test.yml
6b3afdb
@pranshustuff
Update test.yml
c78e80a
@pranshustuff
Update test.yml
51bb503
@pranshustuff
Update test.yml
392f875
@pranshustuff
Update test.yml
3a683f4
@pranshustuff
Update test.yml
f694dd3
@pranshustuff
Update test.yml
58b2688
@pranshustuff
Update test.yml
a931e25
@pranshustuff
installed dependencies
6245da9
@pranshustuff
removed break line in .babelrc
362b272
@pranshustuff
Update test.yml
412ec02
@pranshustuff
Update test.yml
c519fd2
@pranshustuff
Update test.yml
d369302
@pranshustuff
Update test.yml
03f6931
@pranshustuff
changing node version to see if things work
41680b8
@pranshustuff
maybe installing dependencies differently will help
ee8cb8e 
...i'm a broken man
@pranshustuff
hopefully crypto works now
b5dc2cb
@pranshustuff
maybe jest.setup.js was the problem
b0d997b
@pranshustuff
some changes to see if things work
a591606
@pranshustuff
god knows at this point
340a809
@pranshustuff
Update test.yml
ae7b6a4
@pranshustuff
added license header to unit test files
ce1890b
@pranshustuff
Copy link
Author

pranshustuff commented Nov 1, 2024 •
edited
Loading

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
Reviewers

@NipuniBhagya NipuniBhagya Awaiting requested review from NipuniBhagya NipuniBhagya is a code owner

@inthirakumaaran inthirakumaaran Awaiting requested review from inthirakumaaran inthirakumaaran is a code owner

@jeradrutnam jeradrutnam Awaiting requested review from jeradrutnam jeradrutnam is a code owner

@brionmario brionmario Awaiting requested review from brionmario brionmario is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

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