Filter out closed AdSense accounts based on new API capabilities

Based on the recently released new AdSense API capabilities and the corresponding enhancements to the design doc, we need to ensure closed AdSense accounts are ignored by Site Kit. For example, if a user has only a single closed account, they will need to create a new AdSense account, which is the same as if they had no account at all.

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

• The AdSense GET:accounts data point should filter out any accounts from the API response that have a state: CLOSED, as they can be ignored from our perspective.
  • See the new account state documentation for background on the possible values.

Implementation Brief

• Using includes/Modules/AdSense.php,
  • Add a new public static function is_account_not_closed which should have the account array as parameter. The function should return a boolean, whether account.state is different from CLOSED.
  • Within GET:accounts in parse_data_response, instead of returning the result of $response->getAccounts (with the filtered account IDs), store the accounts in a variable ($accounts) and then use array_filter to filter the array, passing the array ($accounts) as first parameter and the function filter_account_with_state_closed as second parameter. Return the updated array.

Test Coverage

• No new tests to be added.

QA Brief

• Use a Google account where you have a closed AdSense account.
• Start the AdSense module setup and verify that this account does not show up or that there are no errors due to this account being chosen.
• Basically, if that closed AdSense account is the only one you have, you should land in the UI to create a new AdSense account.
• This should be tested both with and without the adsenseSetupV2 flag enabled.
• This isn’t exactly possible to set up in a reasonable timeframe, but @felixarntz has such an account, so he is going to QA this issue.

Changelog entry

• Ensure closed AdSense accounts are not considered for the AdSense account to use with the module.

[felixarntz]

@asvinb Just one thing regarding the IB: The filter_account_with_state_closed doesn’t really sound accurate. Since it’s a boolean check for whether the account is not closed, how about we call it is_account_open or is_account_not_closed or something like that? Other than that the IB looks good.

[felixarntz]

@wpdarren Since this is something which happens on the server-side, it wouldn’t really be possible to mock this via the tester plugin.

I think this piece is simple enough so that we don’t need to cover this as part of the bug bash.