darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-20 14:29:51 -04:00
Code Issues Packages Projects Releases Wiki Activity

Support broadcasts with no areas data

Browse Source 
Previously we used to return an empty CustomBroadcastAreas object,
which doesn't make sense for broadcasts created in this app.
This commit is contained in:
Ben Thorner
2021-09-06 11:57:32 +01:00
parent 411fda81c0
commit baf20e0075
2 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
app/models/broadcast_message.py
View File

@@ -99,10 +99,15 @@ class BroadcastMessage(JSONModel):
) )
return library_areas return library_areas
return CustomBroadcastAreas( polygons = self._dict['areas'].get('simple_polygons', [])
names=self._dict['areas']['names'],
polygons=self._dict['areas']['simple_polygons'], if polygons:
) return CustomBroadcastAreas(
names=self._dict['areas']['names'],
polygons=polygons,
)
return []
@property @property
def area_ids(self): def area_ids(self):

1
tests/app/models/test_broadcast_message.py
View File

@@ -41,6 +41,7 @@ def test_content_comes_from_attribute_not_template():
@pytest.mark.parametrize(('areas', 'expected_length'), [ @pytest.mark.parametrize(('areas', 'expected_length'), [
({'ids': []}, 0), ({'ids': []}, 0),
({'ids': ['wd20-E05009372']}, 1), ({'ids': ['wd20-E05009372']}, 1),
({'no data': 'just created'}, 0),
({'names': ['somewhere'], 'simple_polygons': [[[3.5, 1.5]]]}, 1) ({'names': ['somewhere'], 'simple_polygons': [[[3.5, 1.5]]]}, 1)
]) ])
def test_areas( def test_areas(