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

ARA-820 Update unit tests for userHasCreatePermission #1847

Merged
merged 4 commits into from
Mar 30, 2021

Conversation

nestorbe
Copy link
Contributor

🦅 Pull Request Description

This PR creates the unit test for userHasCreatePermission feature.

🚨 Test instructions

yarn test

@linear
Copy link

linear bot commented Mar 29, 2021

ARA-820 Update unit tests for userHasCreatePermission

GitHub reference: #1836

Create the necessary tests for the feature merged in this PR.

Note: created a separate issue for this because of ARA-625 that will change the test suites from AVA to Jest.

@nestorbe nestorbe requested review from c0rv0s and cbrzn March 29, 2021 23:50
Copy link
Contributor

@c0rv0s c0rv0s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nestorbe nestorbe merged commit f359b4b into master Mar 30, 2021
@nestorbe nestorbe deleted the ARA-820-userHasCreatePermission-test branch March 30, 2021 15:42
c0rv0s added a commit that referenced this pull request Mar 31, 2021
* feat: inform user of status of app installation

* ARA-820 Update unit tests for userHasCreatePermission (#1847)

* test: created test for userHasCreatePermission

* feat: inform user of status of app installation

Co-authored-by: Nestor Barboza <51724662+nestorbe@users.noreply.github.com>
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.

2 participants