Feature/242 gateway assignment tests #7152
Conversation
…lesforceFoundation/NPSP into feature/242__gateway-assignment-tests
|
Tracking W-11793517 |
…lesforceFoundation/NPSP into feature/242__gateway-assignment-tests
| expect(getCCCheckBox(element)).toBeTruthy(); | ||
| }) | ||
|
|
||
| it("does not display gateways and payment methods when collapsed", async () => { |
There was a problem hiding this comment.
This and the test above could probably be the same test, just need to add the expectations after it's expanded 🤷
There was a problem hiding this comment.
I had initially combined the tests, but thought it might be more clear what was being tested if they were separate. I'll put them back together.
| document.body.appendChild(element); | ||
| await flushPromises(); | ||
|
|
||
| expect(element).toBeTruthy(); |
There was a problem hiding this comment.
What is this testing? Isn't Gateway Management Mode for the NPSP Settings page? The template has nothing to do with that, right?
There was a problem hiding this comment.
We can eliminate the test if that seems right. What it is testing is to make sure that gateway management isn't affected by that setting. It should always be null, so seems fine to cut it out.
Critical Changes
Changes
Issues Closed
Community Ideas Delivered
Features Intended for Future Release
Features for Elevate Customers
New Metadata
Deleted Metadata