Merge pull request #152 from sfo/fix_temperature

akrherz · web-flow · commit a0768efb99eb · 2021-10-05T11:39:55.000-05:00
illegal temperature will raise ParserError
diff --git a/metar/Metar.py b/metar/Metar.py
@@ -85,8 +85,8 @@ class ParserError(Exception):
     re.VERBOSE,
 )
 TEMP_RE = re.compile(
-    r"""^(?P<temp>(M|-)?\d+|//|XX|MM)/
-        (?P<dewpt>(M|-)?\d+|//|XX|MM)?\s+""",
+    r"""^(?P<temp>(M|-)?\d{1,2}|//|XX|MM)/
+        (?P<dewpt>(M|-)?\d{1,2}|//|XX|MM)?\s+""",
     re.VERBOSE,
 )
 PRESS_RE = re.compile(
diff --git a/test/test_metar.py b/test/test_metar.py
@@ -653,3 +653,7 @@ def test_cor_auto_mod():
     m = Metar.Metar(code, year=2019)
 
     assert m.mod == 'COR AUTO'
+
+
+def test_issue136_temperature():
+    raisesParserError("METAR EDDM 022150Z 26006KT CAVOK 201/16")

Original file line number	Diff line number	Diff line change
`@@ -85,8 +85,8 @@ class ParserError(Exception):`
`85`	`85`	`re.VERBOSE,`
`86`	`86`	`)`
`87`	`87`	`TEMP_RE = re.compile(`
`88`		`- r"""^(?P<temp>(M\|-)?\d+\|//\|XX\|MM)/`
`89`		`- (?P<dewpt>(M\|-)?\d+\|//\|XX\|MM)?\s+""",`
	`88`	`+ r"""^(?P<temp>(M\|-)?\d{1,2}\|//\|XX\|MM)/`
	`89`	`+ (?P<dewpt>(M\|-)?\d{1,2}\|//\|XX\|MM)?\s+""",`
`90`	`90`	`re.VERBOSE,`
`91`	`91`	`)`
`92`	`92`	`PRESS_RE = re.compile(`