Skip to content

Commit

Permalink
Add double opt-in test cases
Browse files Browse the repository at this point in the history
  • Loading branch information
MaxwellGarceau committed Jan 1, 2025
1 parent 510effb commit 7080ffe
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
4 changes: 4 additions & 0 deletions tests/cypress/e2e/settings.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -195,6 +195,10 @@ describe('Admin can update plugin settings', () => {
});
});

/**
* NOTE: "Use Double Opt-In (Recommended)?" and "Update existing subscribers?"
* are handled in `subscribe.test.js`
*/
it('Admin can set list options settings', () => {
// Remove mailchimp JavaScript support.
cy.visit('/wp-admin/admin.php?page=mailchimp_sf_options');
Expand Down
5 changes: 5 additions & 0 deletions tests/cypress/e2e/subscribe.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,11 @@ describe('Subscribe actions', () => {
});
});

// TODO: BLOCKED - Need access to a service that can catch test emails so we can finish the email verification process.
it.skip('When double opt-in is enabled a subscriber must verify their email before their submission displays in Mailchimp', () => {

});

// TODO: This is a known bug. The back link goes to a 404 page on the Mailchimp account.
it.skip('after a double optin subscribe a user can click a back link to return to the website', () => {

Expand Down

0 comments on commit 7080ffe

Please sign in to comment.