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

User test not needed due to file being not in use #605

Merged
merged 1 commit into from
Mar 10, 2022

Conversation

sumitra19jha
Copy link
Contributor

What kind of change does this PR introduce?
refactoring

Issue Number:

Fixes #492

Did you add tests for your changes?
Please Note that the file in the resolver is no longer required

Snapshots/Videos:
Not needed

If relevant, did you update the documentation?
Not Needed

Summary
@palisadoes Code base is updated and this file was very old and is not being utilised so instead of righting test code and having unnecessary code expansion, I removed the file

Does this PR introduce a breaking change?
No

Other information

Have you read the contributing guide?
Yes

@sumitra19jha sumitra19jha changed the title User test done User test not needed due to file being not in use Mar 8, 2022
@palisadoes palisadoes merged commit 1d015c5 into PalisadoesFoundation:develop Mar 10, 2022
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.

Resolvers: Create tests for User.js
2 participants