Merge pull request #24 from Meisterschueler/fix/aprs_empty_comment

Empty APRS comment belongs to a receiver beacon
pull/26/head
Meisterschueler 2017-05-16 14:42:18 +02:00 zatwierdzone przez GitHub
commit b772ae7b47
2 zmienionych plików z 7 dodań i 1 usunięć

Wyświetl plik

@ -90,6 +90,9 @@ def parse_ogn_receiver_beacon(aprs_comment):
def parse_ogn_beacon(aprs_comment):
if not aprs_comment:
return {'beacon_type': 'receiver_beacon'}
ac_data = parse_ogn_aircraft_beacon(aprs_comment)
if ac_data:
ac_data.update({'beacon_type': 'aircraft_beacon'})

Wyświetl plik

@ -17,10 +17,13 @@ class TestStringMethods(unittest.TestCase):
if aprs['comment']:
parse_ogn_beacon(aprs['comment'])
def test_fail_none(self):
def test_fail_parse_aprs_none(self):
with self.assertRaises(TypeError):
parse_aprs(None)
def test_parse_ogn_none(self):
parse_ogn_beacon(None)
def test_fail_empty(self):
with self.assertRaises(AprsParseError):
parse_aprs("")