diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2023-02-22 05:55:56 -0500 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2023-02-22 05:55:56 -0500 |
commit | 51f395dc29811cd7ab465c527a5a06514b0090b8 (patch) | |
tree | e65d46db0b39f7b13d68bb4b38f9a81a98ad7705 | |
parent | 545d8bc2ae37cf0d203ea3555547e35a4d34346e (diff) | |
download | python-coveragepy-git-51f395dc29811cd7ab465c527a5a06514b0090b8.tar.gz |
test: stricter regex to avoid false CodeQL alarms
Will fix:
https://github.com/nedbat/coveragepy/security/code-scanning/3
https://github.com/nedbat/coveragepy/security/code-scanning/4
(though tbh, not sure how to close those as fixed?)
-rw-r--r-- | tests/test_html.py | 4 | ||||
-rw-r--r-- | tests/test_xml.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/tests/test_html.py b/tests/test_html.py index 8893993c..5113cd06 100644 --- a/tests/test_html.py +++ b/tests/test_html.py @@ -663,8 +663,8 @@ def compare_html( """Specialized compare function for our HTML files.""" __tracebackhide__ = True # pytest, please don't show me this function. scrubs = [ - (r'/coverage.readthedocs.io/?[-.\w/]*', '/coverage.readthedocs.io/VER'), - (r'coverage.py v[\d.abcdev]+', 'coverage.py vVER'), + (r'/coverage\.readthedocs\.io/?[-.\w/]*', '/coverage.readthedocs.io/VER'), + (r'coverage\.py v[\d.abcdev]+', 'coverage.py vVER'), (r'created at \d\d\d\d-\d\d-\d\d \d\d:\d\d [-+]\d\d\d\d', 'created at DATE'), (r'created at \d\d\d\d-\d\d-\d\d \d\d:\d\d', 'created at DATE'), # Occasionally an absolute path is in the HTML report. diff --git a/tests/test_xml.py b/tests/test_xml.py index 005f9d5a..c21a9dd4 100644 --- a/tests/test_xml.py +++ b/tests/test_xml.py @@ -465,7 +465,7 @@ def compare_xml(expected: str, actual: str, actual_extra: bool = False) -> None: (r' timestamp="\d+"', ' timestamp="TIMESTAMP"'), (r' version="[-.\w]+"', ' version="VERSION"'), (r'<source>\s*.*?\s*</source>', '<source>%s</source>' % re.escape(source_path)), - (r'/coverage.readthedocs.io/?[-.\w/]*', '/coverage.readthedocs.io/VER'), + (r'/coverage\.readthedocs\.io/?[-.\w/]*', '/coverage.readthedocs.io/VER'), ] compare(expected, actual, scrubs=scrubs, actual_extra=actual_extra) |