-
Notifications
You must be signed in to change notification settings - Fork 61
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 test for Mark as Unpaid button #1325
Conversation
You have some conflicting files @gouravmpk can you resolve? |
@ecurrencyhodler getting this message |
looks good to me if you can resolve the merge conflict |
@gouravmpk you can fix the merge conflict with |
The tests are not passing @gouravmpk |
Looking into it |
You've still got prettier and jest tests failing. |
Yeah its not rendering in existing test because the button shows only when user is ORG admin or hasAccess |
Trying to mock store
Jest is still failing and I see a conflicting file: frontend/app/src/people/widgetViews/summaries/wantedSummaries/tests/CodingBounty.spec.tsx |
…ribes into button-rename
@ecurrencyhodler , I've put in a lot of effort to write the test, but unfortunately, it seems to be facing issues due to conditional rendering. Despite trying every solution I could think of, the problem persists. |
Hey thanks for the update. Let me sync with raph. |
@gouravmpk we are going to close this PR and I will pay you out for the bounty. We are going to handle this separately in a different issue. Thanks for your attempt. |
Describe your changes
Added test case (Don't merge)
Issue ticket number and link
#1196
Type of change
Unit test