There is some complex logic that occurs when the Acquia Search module is disabled. We should define the expected behaviors and add tests for them.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cpliakas’s picture

Status: Active » Needs work
FileSize
2.57 KB

First use case, marking as needs work because there are others.

cpliakas’s picture

Status: Postponed » Needs review
FileSize
5.94 KB

Added the different use cases I could think of. The bot will fail because testDisableModuleCase2() has two use cases that I think should be addressed but aren't yet. Will file a separate issue where we can discuss the use cases, and will set this as postponed pending that discussion.

The last submitted patch, disable-tests-1791642-2.patch, failed testing.

cpliakas’s picture

Status: Needs work » Postponed

Status: Needs review » Postponed
Dane Powell’s picture

Issue summary: View changes
Status: Postponed » Closed (won't fix)

This branch of Acquia Connector is no longer supported, per the version policy on the project homepage. As such, I'm tentatively closing this issue.

If this issue still applies to a supported branch (currently 7.x-3.x, 8.x-1.x, or 8.x-2.x), please reopen and select the new target version. Thanks!