Bug: the azure-devops cache folder location is computed differently in different places

[jessehouwing]

The Azure DevOps CLI has a way to redirect the config and the cache directory to a different location.

In most code paths it's enough to only redirect the CONFIG directory and in that case the cache folder will automatically be placed underneath the config folder.

But there are a few places in which the cache folder is placed under the home directory instead.

Under home

azure-devops-cli-extension/azure-devops/azext_devops/devops_sdk/_file_cache.py

Line 114 in bd34a6f

azure_devops_cache_dir = os.getenv('AZURE_DEVOPS_CACHE_DIR', None)\

Under config

azure-devops-cli-extension/azure-devops/azext_devops/dev/common/file_cache.py

Line 15 in bd34a6f

os.path.join(AZ_DEVOPS_GLOBAL_CONFIG_DIR, 'cache'))

I'd expect a single consistent behavior.

https://github.com/search?q=repo%3AAzure%2Fazure-devops-cli-extension+os.getenv&type=code