Complete CloudApiClient

[webbj74]

The CloudApiClient class is missing methods for the following API functions

• [x] List a site’s tasks. [See #11]
  • GET /sites/:site/tasks
• [x] Copy files from one site environment to another.
  • POST /sites/:site/files-copy/:source/:target
• [x] Deploy code from one site environment to another.
  • POST /sites/:site/code-deploy/:source/:target
• [x] Add a database.
  • POST /sites/:site/dbs
• [x] Copy a database from one site environment to another.
  • POST /sites/:site/dbs/:db/db-copy/:source/:target
• [x] Delete a database.
  • DELETE /sites/:site/dbs/:db
• [x] Move domain(s) from one site environment to another.
  • POST /sites/:site/domain-move/:source/:target
• [x] Deploy a specific VCS branch or tag to an environment.
  • POST /sites/:site/envs/:env/code-deploy
• [x] Delete a site environment database instance backup.
  • DELETE /sites/:site/envs/:env/dbs/:db/backups/:backup
• [x] Restore a site environment database instance backup.
  • POST /sites/:site/envs/:env/dbs/:db/backups/:backup/restore
• [x] Configure Live Development on an environment.
  • POST /sites/:site/envs/:env/livedev/:action
• [x] List an environment’s domains.
  • GET /sites/:site/envs/:env/domains
• [x] Get a domain record.
  • GET /sites/:site/envs/:env/domains/:domain
• [x] Delete a domain.
  • DELETE /sites/:site/envs/:env/domains/:domain
• [x] Add a domain name.
  • POST /sites/:site/envs/:env/domains/:domain
• [x] Purge the Varnish cache for a domain
  • DELETE /sites/:site/envs/:env/domains/:domain/cache

[cpliakas]

Added methods that consume GET resources for domains in commit f7ac13457388d2a9a0f73ab8b5edc3e77d143ed1. @webbj74, I need to figure out your awesome Mock response strategy to add tests!

[cpliakas]

Added method to purge Varnish cache at 4271ec57820bd8927e85e55a7d5d2c1d988dfd4f.

[webbj74]

@cpliakas -- I've followed the original unit test examples you provided, the change to check responses doesn't actually use a mock, it's just using a standard Guzzle event subscriber.

First add a callback function for the create_request event, like:

public function domainsCallListener(Event $e) {
  $this->assertEquals('https://cloudapi.example.com/v1/sites/myhostingstage%3Amysitegroup/dev/domains.json', $e['request']->getUrl());
}

Since most of these URLs are similar in our tests, I use a const to alias the most common chunk:

$this->assertEquals(self::REQUEST_PATH . '/dev/domains.json', $e['request']->getUrl());

After instantiating the cloudapiclient, but before calling the domains method, add the domainsCallListener method as an event listener for client.create_request events:

$cloudapi->getEventDispatcher()->addListener('client.create_request', array($this, 'domainsCallListener'));

There's probably a more "phpunit" way to handle this using its built-in mocking capability, but this seemed like a quick and easy way to deal with it.

[lahoosascoots]

PRs in for add/delete domains and delete backups.

[cpliakas]

Remaining calls added at https://github.com/acquia/acquia-sdk-php/tree/18-refactor.

Testing system has been refactored to make it a little easier to write tests. Wrapper methods are also removed from the client for transparency, and the response objects are a little more DRY and provide methods to get the data returned so you know what is available.

[cpliakas]

Merged in branch. API is complete minus #13 and #24 which will be addressed in those issues.