Remove ‘Regions of England’ library

It made for a good early demo to show how we could have different
libraries, but we’d don’t think there’s a strong user need for being
able to broadcast to a region of England.

Regions also have the problem that:
- they are ambiguous – both England and Scotland have a region called
  ‘South east’
- Northern Ireland doesn’t have formal regions

This commit removes the regions library.
This commit is contained in:
Chris Hill-Scott
2020-08-12 17:40:08 +01:00
parent a85134ee22
commit 8ef1e98745
5 changed files with 1 additions and 13 deletions

View File

@@ -25,11 +25,6 @@ def test_loads_libraries():
'Electoral Wards of the United Kingdom',
True,
),
(
'regions-of-england',
'Regions of England',
False,
),
]
@@ -50,9 +45,6 @@ def test_examples():
countries = broadcast_area_libraries.get('countries').get_examples()
assert countries == 'England, Northern Ireland, Scotland, and Wales'
regions = broadcast_area_libraries.get('regions-of-england').get_examples()
assert regions == 'East Midlands, East of England, London, and 5 more…'
counties = broadcast_area_libraries.get(
'counties-and-unitary-authorities-in-england-and-wales',
).get_examples()
@@ -187,10 +179,9 @@ def test_repository_has_all_libraries():
repo = BroadcastAreasRepository()
libraries = repo.get_libraries()
assert len(libraries) == 4
assert len(libraries) == 3
assert [
'Counties and Unitary Authorities in England and Wales',
'Countries',
'Electoral Wards of the United Kingdom',
'Regions of England',
] == sorted([name for _, name, _is_group in libraries])