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

Process handler test #76

Merged

Conversation

captain-Akshay
Copy link
Contributor

fixes Issue :- #61
updated the mocha library and included a new library sinon which is must else the process handler exit the code to run so override the existing process.exit have to use sinon or jest to make sure that it doesn't exit the process completely

I tested the npm run test it is working fine with all the test cases passed !

@codesandbox
Copy link

codesandbox bot commented Oct 18, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@captain-Akshay
Copy link
Contributor Author

@EbonyLouis done!!

@finn-block finn-block merged commit eb67a75 into decentralized-identity:main Oct 18, 2023
1 check passed
@finn-block finn-block added hacktoberfest-accepted Accepted PRs for the hacking month of October and removed hacktoberfest-accepted Accepted PRs for the hacking month of October labels Oct 18, 2023
@finn-block
Copy link
Collaborator

This broke the CI container build (link). Please fix or I'll have to revert this.

@EbonyLouis
Copy link
Contributor

@captain-Akshay ^

@captain-Akshay
Copy link
Contributor Author

I will fix it right away @finn-tbd

@finn-block finn-block mentioned this pull request Oct 18, 2023
@EbonyLouis EbonyLouis added the hacktoberfest-accepted Accepted PRs for the hacking month of October label Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted PRs for the hacking month of October
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants