diff-cover 9.2.2__tar.gz → 9.2.4__tar.gz

diff_cover-9.2.4/NOTICE

@@ -0,0 +1,5 @@
+diff-cover
+
+Copyright 2013-2014, edX
+
+Copyright 2015, Matt Bachmann

{diff_cover-9.2.2 → diff_cover-9.2.4}/PKG-INFO

@@ -1,8 +1,7 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.3
 Name: diff_cover
-Version: 9.2.2
+Version: 9.2.4
 Summary: Run coverage and linting reports on diffs
-Home-page: https://github.com/Bachmann1234/diff-cover
 License: Apache-2.0
 Author: See Contributors
 Requires-Python: >=3.9.17,<4.0.0
@@ -23,11 +22,12 @@ Classifier: Topic :: Software Development :: Quality Assurance
 Classifier: Topic :: Software Development :: Testing
 Provides-Extra: toml
 Requires-Dist: Jinja2 (>=2.7.1)
-Requires-Dist: Pygments (>=2.9.0,<3.0.0)
+Requires-Dist: Pygments (>=2.19.1,<3.0.0)
 Requires-Dist: chardet (>=3.0.0)
 Requires-Dist: pluggy (>=0.13.1,<2)
 Requires-Dist: setuptools (>=17.0.0) ; python_version < "3.8"
 Requires-Dist: tomli (>=1.2.1) ; extra == "toml"
+Project-URL: Homepage, https://github.com/Bachmann1234/diff-cover
 Project-URL: Repository, https://github.com/Bachmann1234/diff-cover
 Description-Content-Type: text/x-rst
 

{diff_cover-9.2.2 → diff_cover-9.2.4}/diff_cover/git_diff.py

@@ -5,6 +5,7 @@ Wrapper for `git diff` command.
 from textwrap import dedent
 
 from diff_cover.command_runner import CommandError, execute
+from diff_cover.util import to_unescaped_filename
 
 
 class GitDiffError(Exception):
@@ -109,7 +110,7 @@ class GitDiffTool:
         output = execute(["git", "ls-files", "--exclude-standard", "--others"])[0]
         if not output:
             return []
-        return [line for line in output.splitlines() if line]
+        return [to_unescaped_filename(line) for line in output.splitlines() if line]
 
 
 class GitDiffFileTool(GitDiffTool):

diff_cover-9.2.4/diff_cover/util.py

@@ -0,0 +1,56 @@
+import os.path
+import posixpath
+
+
+def to_unix_path(path):
+    """
+    Tries to ensure tha the path is a normalized unix path.
+    This seems to be the solution cobertura used....
+    https://github.com/cobertura/cobertura/blob/642a46eb17e14f51272c6962e64e56e0960918af/cobertura/src/main/java/net/sourceforge/cobertura/instrument/ClassPattern.java#L84
+
+    I know of at least one case where this will fail (\\) is allowed in unix paths.
+    But I am taking the bet that this is not common. We deal with source code.
+
+    :param path: string of the path to convert
+    :return: the unix version of that path
+    """
+    return posixpath.normpath(os.path.normcase(path).replace("\\", "/"))
+
+
+def to_unescaped_filename(filename: str) -> str:
+    """Try to unescape the given filename.
+
+    Some filenames given by git might be escaped with C-style escape sequences
+    and surrounded by double quotes.
+    """
+    if not (filename.startswith('"') and filename.endswith('"')):
+        return filename
+
+    # Remove surrounding quotes
+    unquoted = filename[1:-1]
+
+    # Handle C-style escape sequences
+    result = []
+    i = 0
+    while i < len(unquoted):
+        if unquoted[i] == "\\" and i + 1 < len(unquoted):
+            # Handle common C escape sequences
+            next_char = unquoted[i + 1]
+            result.append(
+                {
+                    "\\": "\\",
+                    '"': '"',
+                    "a": "a",
+                    "n": "\n",
+                    "t": "\t",
+                    "r": "\r",
+                    "b": "\b",
+                    "f": "\f",
+                }.get(next_char, next_char)
+            )
+            i += 2
+        else:
+            result.append(unquoted[i])
+            i += 1
+
+    return "".join(result)

{diff_cover-9.2.2 → diff_cover-9.2.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "diff_cover"
-version = "9.2.2"
+version = "9.2.4"
 description = "Run coverage and linting reports on diffs"
 authors = ["See Contributors"]
 homepage = "https://github.com/Bachmann1234/diff-cover"
@@ -41,7 +41,7 @@ diff-quality = 'diff_cover.diff_quality_tool:main'
 
 [tool.poetry.dependencies]
 python = "^3.9.17"
-Pygments = "^2.9.0"
+Pygments = "^2.19.1"
 Jinja2 = ">=2.7.1"
 pluggy = ">=0.13.1,<2"
 chardet = ">=3.0.0"
@@ -55,11 +55,11 @@ pytest-mock = "^3.14.0"
 pycodestyle = ">=2.9.1"
 flake8 = "^5.0.4"
 pyflakes = "^2.5.0"
-pylint = "^3.3.3"
+pylint = "^3.3.4"
 pylint-pytest = "^1.1.8"
 pydocstyle = "^6.1.1"
-black = "^24.8.0"
-isort = "^5.13.2"
+black = "^25.1.0"
+isort = "^6.0.1"
 doc8 = "1.1.2"
 
 [tool.poetry.extras]

{diff_cover-9.2.2 → diff_cover-9.2.4}/tests/fixtures/pycodestyle_violations_report_external_css.html

@@ -46,12 +46,12 @@
 <span class="normal"> 9</span>
 <span class="normal">10</span>
 <span class="normal">11</span>
-<span class="normal">12</span></pre></div></td><td class="code"><div><pre><span></span><a id="violations_test_file.py-1" name="violations_test_file.py-1"></a><span class="k">def</span> <span class="nf">func_1</span><span class="p">(</span><span class="n">apple</span><span class="p">,</span> <span class="n">my_list</span><span class="p">):</span>
+<span class="normal">12</span></pre></div></td><td class="code"><div><pre><span></span><a id="violations_test_file.py-1" name="violations_test_file.py-1"></a><span class="k">def</span><span class="w"> </span><span class="nf">func_1</span><span class="p">(</span><span class="n">apple</span><span class="p">,</span> <span class="n">my_list</span><span class="p">):</span>
 <a id="violations_test_file.py-2" name="violations_test_file.py-2"></a><span class="hll">    <span class="k">if</span> <span class="n">apple</span><span class="o">&lt;</span><span class="mi">10</span><span class="p">:</span>
 </span><a id="violations_test_file.py-3" name="violations_test_file.py-3"></a>        <span class="c1"># Do something </span>
 <a id="violations_test_file.py-4" name="violations_test_file.py-4"></a>        <span class="n">my_list</span><span class="o">.</span><span class="n">append</span><span class="p">(</span><span class="n">apple</span><span class="p">)</span>
 <a id="violations_test_file.py-5" name="violations_test_file.py-5"></a>    <span class="k">return</span> <span class="n">my_list</span><span class="p">[</span><span class="mi">1</span><span class="p">:]</span>
-<a id="violations_test_file.py-6" name="violations_test_file.py-6"></a><span class="hll"><span class="k">def</span> <span class="nf">func_2</span><span class="p">(</span><span class="n">spongebob</span><span class="p">,</span> <span class="n">squarepants</span><span class="p">):</span>
+<a id="violations_test_file.py-6" name="violations_test_file.py-6"></a><span class="hll"><span class="k">def</span><span class="w"> </span><span class="nf">func_2</span><span class="p">(</span><span class="n">spongebob</span><span class="p">,</span> <span class="n">squarepants</span><span class="p">):</span>
 </span><a id="violations_test_file.py-7" name="violations_test_file.py-7"></a><span class="w">    </span><span class="sd">&quot;&quot;&quot;A less messy function&quot;&quot;&quot;</span>
 <a id="violations_test_file.py-8" name="violations_test_file.py-8"></a>    <span class="k">for</span> <span class="n">char</span> <span class="ow">in</span> <span class="n">spongebob</span><span class="p">:</span>
 <a id="violations_test_file.py-9" name="violations_test_file.py-9"></a>        <span class="k">if</span> <span class="n">char</span> <span class="ow">in</span> <span class="n">squarepants</span><span class="p">:</span>

{diff_cover-9.2.2 → diff_cover-9.2.4}/tests/fixtures/snippet.css

@@ -1,8 +1,8 @@
 .hll { background-color: #ffcccc }
 .c { color: #3D7B7B; font-style: italic } /* Comment */
 .k { color: #008000; font-weight: bold } /* Keyword */
-.o { color: #666666 } /* Operator */
-.m { color: #666666 } /* Literal.Number */
+.o { color: #666 } /* Operator */
+.m { color: #666 } /* Literal.Number */
 .s { color: #BA2121 } /* Literal.String */
 .na { color: #687822 } /* Name.Attribute */
 .nb { color: #008000 } /* Name.Builtin */

diff_cover-9.2.4/tests/test_util.py

@@ -0,0 +1,23 @@
+import sys
+
+from diff_cover import util
+
+
+def test_to_unix_path():
+    """
+    Ensure the _to_unix_path static function handles paths properly.
+    """
+    assert util.to_unix_path("foo/bar") == "foo/bar"
+    assert util.to_unix_path("foo\\bar") == "foo/bar"
+    if sys.platform.startswith("win"):
+        assert util.to_unix_path("FOO\\bar") == "foo/bar"
+
+
+def test_to_unescaped_filename():
+    """Test the to_unescaped_filename function."""
+    assert util.to_unescaped_filename('"\\\\\\\\"') == "\\\\"
+    assert util.to_unescaped_filename('"\\\\"') == "\\"
+    assert util.to_unescaped_filename('"\\"\\\\"') == '"\\'
+    assert util.to_unescaped_filename('"\\""') == '"'
+    assert util.to_unescaped_filename("some_file.py") == "some_file.py"
+    assert util.to_unescaped_filename("#$%") == "#$%"

diff_cover-9.2.2/diff_cover/util.py

@@ -1,17 +0,0 @@
-import os.path
-import posixpath
-
-
-def to_unix_path(path):
-    """
-    Tries to ensure tha the path is a normalized unix path.
-    This seems to be the solution cobertura used....
-    https://github.com/cobertura/cobertura/blob/642a46eb17e14f51272c6962e64e56e0960918af/cobertura/src/main/java/net/sourceforge/cobertura/instrument/ClassPattern.java#L84
-
-    I know of at least one case where this will fail (\\) is allowed in unix paths.
-    But I am taking the bet that this is not common. We deal with source code.
-
-    :param path: string of the path to convert
-    :return: the unix version of that path
-    """
-    return posixpath.normpath(os.path.normcase(path).replace("\\", "/"))

diff_cover-9.2.2/tests/test_util.py

@@ -1,13 +0,0 @@
-import sys
-
-from diff_cover import util
-
-
-def test_to_unix_path():
-    """
-    Ensure the _to_unix_path static function handles paths properly.
-    """
-    assert util.to_unix_path("foo/bar") == "foo/bar"
-    assert util.to_unix_path("foo\\bar") == "foo/bar"
-    if sys.platform.startswith("win"):
-        assert util.to_unix_path("FOO\\bar") == "foo/bar"