ED/EDMarketConnector
1
0
Fork 0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-13 07:47:14 +03:00
Code Issues Projects Releases Wiki Activity

Replace AssertionErrors with ValueError

Browse Source 
AssertionError comes from `assert`s, using it for this doesn't make sense as we're complaining about values
This commit is contained in:
A_D 2020-07-13 06:04:45 +02:00 committed by Athanasius
parent fd3a4852f2
commit ed9c1c6774
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
collate.py
View File

@ -45,7 +45,7 @@ def addcommodities(data):
if old and companion.category_map.get(commodity['categoryname'], True):
if new['symbol'] != old['symbol'] or new['name'] != old['name']:
raise AssertionError('{}: {!r} != {!r}'.format(key, new, old))
raise ValueError('{}: {!r} != {!r}'.format(key, new, old))
commodities[key] = new
@ -87,7 +87,7 @@ def addmodules(data):
for key, module in data['lastStarport'].get('modules').items():
# sanity check
if int(key) != module.get('id'):
raise AssertionError('id: {} != {}'.format(key, module['id']))
raise ValueError('id: {} != {}'.format(key, module['id']))
try:
new = outfitting.lookup(module, companion.ship_map, True)
@ -106,7 +106,7 @@ def addmodules(data):
size_pre -= 1
elif str(new.get(thing, '')) != old.get(thing):
raise AssertionError('{}: {} {!r}!={!r}'.format(key, thing, new.get(thing), old.get(thing)))
raise ValueError('{}: {} {!r}!={!r}'.format(key, thing, new.get(thing), old.get(thing)))
modules[int(key)] = new
@ -160,7 +160,7 @@ def addships(data):
size_pre -= 1
elif str(new.get(thing, '')) != old.get(thing):
raise AssertionError('{}: {} {!r} != {!r}'.format(key, thing, new.get(thing), old.get(thing)))
raise ValueError('{}: {} {!r} != {!r}'.format(key, thing, new.get(thing), old.get(thing)))
ships[int(key)] = new