Added fixes and tests for position packets

sometimes position packets have trailing spaces.  this patch fixes
the parsing of those packets.

Author: hemna

aprslib/parsing/position.py

@@ -156,8 +156,8 @@ def parse_compressed(body):
 def parse_normal(body):
     parsed = {}
 
-    match = re.findall(r"^(\d{2})([0-9 ]{2}\.[0-9 ]{2})([NnSs])([\/\\0-9A-Z])"
-                       r"(\d{3})([0-9 ]{2}\.[0-9 ]{2})([EeWw])([\x21-\x7e])(.*)$", body)
+    match = re.findall(r"^(\d{2})([0-9 ]{2}\.[0-9 ]{1,2})([NnSs])([\/\\0-9A-Z])"
+                       r"(\d{3})([0-9 ]{2}\.[0-9 ]{1,2})([EeWw])([\x21-\x7e])(.*)$", body)
 
     if match:
         parsed.update({'format': 'uncompressed'})
@@ -175,9 +175,12 @@ def parse_normal(body):
         ) = match[0]
 
         # position ambiguity
-        posambiguity = lat_min.count(' ')
+        # Strip trailing spaces before counting (trailing spaces are not part of ambiguity)
+        lat_min_stripped = lat_min.rstrip(' ')
+        lon_min_stripped = lon_min.rstrip(' ')
+        posambiguity = lat_min_stripped.count(' ')
 
-        if posambiguity != lon_min.count(' '):
+        if posambiguity != lon_min_stripped.count(' '):
             raise ParseError("latitude and longitude ambiguity mismatch")
 
         parsed.update({'posambiguity': posambiguity})
@@ -187,8 +190,9 @@ def parse_normal(body):
             lat_min = "30"
             lon_min = "30"
         else:
-            lat_min = lat_min.replace(' ', '5', 1)
-            lon_min = lon_min.replace(' ', '5', 1)
+            # Use stripped versions for ambiguity replacement
+            lat_min = lat_min_stripped.replace(' ', '5', 1)
+            lon_min = lon_min_stripped.replace(' ', '5', 1)
 
         # validate longitude and latitude
 

tests/test_parse_position.py

@@ -166,5 +166,19 @@ def test_position_packet_optional_course(self):
         _, result = parse_position(packet_type, packet)
         self.assertEqual(expected, result)
 
+    def test_position_with_trailing_space_in_minutes(self):
+        """Test position packet with trailing space in minutes field (real-world packet)"""
+        from aprslib.parsing import parse
+        packet = "N0NPO-2>APWW11,TCPIP*,qAC,T2SYDNEY:@215527h4424.4 N/10017.85W#"
+        result = parse(packet)
+
+        self.assertEqual(result['format'], 'uncompressed')
+        self.assertAlmostEqual(result['latitude'], 44.406666666666666, places=5)
+        self.assertAlmostEqual(result['longitude'], -100.2975, places=5)
+        self.assertEqual(result['posambiguity'], 0)  # Trailing spaces don't count as ambiguity
+        self.assertEqual(result['symbol_table'], '/')
+        self.assertEqual(result['symbol'], '#')
+        self.assertIn('timestamp', result)
+
 if __name__ == '__main__':
     unittest.main()