From 46cc6ac8e9448a7b2acec62f9678082e2505dfc7 Mon Sep 17 00:00:00 2001 From: Robert Alonso <17463757+rgalonso@users.noreply.github.com> Date: Thu, 10 Apr 2025 15:14:05 +0000 Subject: [PATCH] fix: remove subdirectory paths from test_new.diff When test_proces_diff processes this file, it fails to create the subdirectories, leading to test failure. Since the subdirectories aren't really needed, removing them allows the test to complete successfully. Updated number of expected passing tests to account for the additional tests which can run to completion now. --- .github/workflows/ci.yml | 2 +- tests/test_new.diff | 6 +++--- tests/test_process_diff.py | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 13987d1..e855120 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -17,4 +17,4 @@ jobs: - name: "Run tests" run: python -m unittest discover env: - SKIP_PROCESS_DIFF_TEST: true \ No newline at end of file + SKIP_PROCESS_DIFF_TEST: false \ No newline at end of file diff --git a/tests/test_new.diff b/tests/test_new.diff index cf311dd..3f32c41 100644 --- a/tests/test_new.diff +++ b/tests/test_new.diff @@ -563,7 +563,7 @@ index 0000000..d340f6a + * labels: urgent + */ +} -diff --git a/tests/package.move b/tests/package.move +diff --git a/package.move b/package.move new file mode 100644 index 0000000..d340f6a --- /dev/null @@ -589,11 +589,11 @@ index 0000000..d340f6a + * labels: urgent + */ +} -diff --git a/src/Dockerfile b/src/Dockerfile +diff --git a/Dockerfile b/Dockerfile new file mode 100644 index 0000000..d340f6a --- /dev/null -+++ b/src/Dockerfile ++++ b/Dockerfile @@ -0,0 +1,4 @@ +WORKDIR /app +ENV PYTHONPATH /app diff --git a/tests/test_process_diff.py b/tests/test_process_diff.py index 32556c2..f8ec875 100644 --- a/tests/test_process_diff.py +++ b/tests/test_process_diff.py @@ -78,7 +78,7 @@ class IssueUrlInsertionTest(unittest.TestCase): "Skipping because 'SKIP_PROCESS_DIFF_TEST' is 'true'") def test_url_insertion(self): self._setUp(['test_new.diff']) - self._standardTest(80) + self._standardTest(84) def test_line_numbering_with_deletions(self): self._setUp(['test_new_py.diff', 'test_edit_py.diff'])