tests: Modify regex test to expose too broad replacement

Jan-Philipp Litza · Jan-Philipp Litza · commit abdef4c23d59 · 2022-01-04T10:50:37.000+01:00
diff --git a/tests.py b/tests.py
@@ -133,11 +133,11 @@ def test_column(line, columns, expected):
     "line,regex,expected,replace",
     [
         (
-            '3.3.3.3 - - [20/May/2015:21:05:01 +0000] "GET / HTTP/1.1" 200 13358 "-" "useragent"',
+            '3.3.3.3 - - [20/May/2015:21:05:01 +0000] "GET /723.3.3.357 HTTP/1.1" 200 13358 "-" "useragent"',
             re.compile(
                 r"(?:^([^,]+) - - |.* - somefixedstring: ([^,]+) - .* - ([^,]+))"
             ),
-            '3.3.0.0 - - [20/May/2015:21:05:01 +0000] "GET / HTTP/1.1" 200 13358 "-" "useragent"',
+            '3.3.0.0 - - [20/May/2015:21:05:01 +0000] "GET /723.3.3.357 HTTP/1.1" 200 13358 "-" "useragent"',
             None,
         ),
         (

Original file line number	Diff line number	Diff line change
`@@ -133,11 +133,11 @@ def test_column(line, columns, expected):`
`133`	`133`	`"line,regex,expected,replace",`
`134`	`134`	`[`
`135`	`135`	`(`
`136`		`- '3.3.3.3 - - [20/May/2015:21:05:01 +0000] "GET / HTTP/1.1" 200 13358 "-" "useragent"',`
	`136`	`+ '3.3.3.3 - - [20/May/2015:21:05:01 +0000] "GET /723.3.3.357 HTTP/1.1" 200 13358 "-" "useragent"',`
`137`	`137`	`re.compile(`
`138`	`138`	`r"(?:^([^,]+) - - \|.* - somefixedstring: ([^,]+) - .* - ([^,]+))"`
`139`	`139`	`),`
`140`		`- '3.3.0.0 - - [20/May/2015:21:05:01 +0000] "GET / HTTP/1.1" 200 13358 "-" "useragent"',`
	`140`	`+ '3.3.0.0 - - [20/May/2015:21:05:01 +0000] "GET /723.3.3.357 HTTP/1.1" 200 13358 "-" "useragent"',`
`141`	`141`	`None,`
`142`	`142`	`),`
`143`	`143`	`(`