Adding drush cache directory to core:status

[rjgwiz]

Adding drush cache directory as an output field in drush status (@command core:status) as a sanity check for the new cache-directory config/DRUSH_PATHS_CACHE_DIRECTORY env variable added in #3597.

Example usage: drush status --field=drush-cache-directory

[greg-1-anderson]

LGTM

[weitzman]

I think this should be omitted from the default fields.

[weitzman]

Or perhaps it should be in %paths such as drush st %paths. At least that makes sense from a config hierarchy perspective - (DRUSH.PATHS.CACHE_DIRECTORY)

[rjgwiz]

I think this should be omitted from the default fields.

Makes sense

Or perhaps it should be in %paths such as drush st %paths. At least that makes sense from a config hierarchy perspective - (DRUSH.PATHS.CACHE_DIRECTORY)

That makes sense from a drush runtime config perspective, though the other paths all seem to be arrays and there is only one cache dir (similar to temp dir) . Also when %paths gets printed, it's labeled "Other Paths" with a list of folders. I think cache path having its own label would be very helpful. thoughts @weitzman?

[rjgwiz]

I'm stumped on the ci error. I can't see where this change would be impacting UpdateDBTest::testFailedUpdate()

[greg-1-anderson]

Those same tests passed in ci/circleci: build and ci/circleci: build_highest. Re-running build_56 to see if perhaps it was a transient failure