darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-06-05 05:48:38 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove redundant conditional for CF Redis

Browse Source 
This is now used in all environments and we've removed support for
non-CF Redis.
This commit is contained in:
Leo Hemsted
2022-04-20 11:41:33 +01:00
parent bf083b28aa
commit 0457850fc0
2 changed files with 1 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/cloudfoundry_config.py
View File

@@ -9,5 +9,4 @@ def extract_cloudfoundry_config():
os.environ['SQLALCHEMY_DATABASE_URI'] = vcap_services['postgres'][0]['credentials']['uri'].replace('postgres',
'postgresql')
# Redis config
if 'redis' in vcap_services:
os.environ['REDIS_URL'] = vcap_services['redis'][0]['credentials']['uri']
os.environ['REDIS_URL'] = vcap_services['redis'][0]['credentials']['uri']

8
tests/app/test_cloudfoundry_config.py
View File

@@ -29,11 +29,3 @@ def test_extract_cloudfoundry_config_populates_other_vars(os_environ, vcap_servi
assert os.environ['SQLALCHEMY_DATABASE_URI'] == 'postgresql uri'
assert os.environ['REDIS_URL'] == 'redis uri'
def test_extract_cloudfoundry_config_copes_if_redis_not_set(os_environ, vcap_services):
del vcap_services['redis']
os.environ['VCAP_SERVICES'] = json.dumps(vcap_services)
extract_cloudfoundry_config()
assert 'REDIS_URL' not in os.environ