Skip to content
Open
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
d4cf821
Initial plan
Copilot Aug 22, 2025
1e33730
Upgrade pip to 25.2 and pip-tools to 7.5.0 with test fixtures
Copilot Aug 22, 2025
922ee68
Add regression test for relative import paths with pip >25
Copilot Aug 22, 2025
2880210
Remove experimental test fixtures, focus on debugging CI failures
Copilot Aug 27, 2025
eb40e93
Add regression test for relative path preservation in pip >25
Copilot Aug 27, 2025
7921dc7
Fix linting issues: remove trailing whitespace
Copilot Aug 27, 2025
4c688fb
Merge branch 'main' into copilot/fix-12663
AbhishekBhaskar Aug 27, 2025
00f69f5
Upgrade pip to 25.2 and pip-tools to 7.5.0 with test fixtures
Copilot Aug 22, 2025
512f7ee
Add regression test for relative import paths with pip >25
Copilot Aug 22, 2025
cd8fb76
Remove experimental test fixtures, focus on debugging CI failures
Copilot Aug 27, 2025
c381a47
Add regression test for relative path preservation in pip >25
Copilot Aug 27, 2025
b3ffd21
Fix linting issues: remove trailing whitespace
Copilot Aug 27, 2025
63212f4
Fix failing Python tests: correct dependency requirements and error m…
Copilot Aug 29, 2025
d5fe992
Merge branch 'main' into copilot/fix-12663
AbhishekBhaskar Aug 29, 2025
c356ad8
Fix lint and test errors: split long regex patterns and update error …
Copilot Aug 29, 2025
c30c8e2
Merge branch 'main' into copilot/fix-12663
markhallen Aug 29, 2025
71d6b70
raise error when piptools is not installed
AbhishekBhaskar Aug 29, 2025
ce2fffd
raise error when piptools is not installed
AbhishekBhaskar Aug 29, 2025
0522a8b
Merge branch 'main' into copilot/fix-12663
AbhishekBhaskar Aug 29, 2025
63ff03c
fix python slow tests
AbhishekBhaskar Aug 29, 2025
392a3bc
Merge branch 'copilot/fix-12663' of https://github.com/dependabot/dep…
AbhishekBhaskar Aug 29, 2025
d501c0a
fix lint error
AbhishekBhaskar Aug 29, 2025
5230929
fix python slow test error
AbhishekBhaskar Aug 29, 2025
7b00d98
fix python slow test error
AbhishekBhaskar Aug 29, 2025
eb5a69f
Merge branch 'main' into copilot/fix-12663
AbhishekBhaskar Aug 29, 2025
70c2dcc
Merge branch 'main' into copilot/fix-12663
AbhishekBhaskar Sep 1, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions python/helpers/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
pip==24.2
pip-tools==7.4.1
pip==25.2
pip-tools==7.5.0
flake8==7.3.0
hashin==1.0.5
pipenv==2024.4.1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -551,6 +551,64 @@
expect(updated_files.first.content).not_to include("boto3")
Comment thread
kbukum1 marked this conversation as resolved.
end
end

context "with relative import paths (regression test for pip >25 issue)" do
# This test verifies that the upgrade to pip 25.2 and pip-tools 7.5.0
# correctly preserves relative paths in generated comments instead of
# converting them to absolute paths. This was an issue with pip >25 and
# older pip-tools versions.
let(:dependency_files) do
[base_file, manifest_file, generated_file]
end
let(:base_file) do
Dependabot::DependencyFile.new(
name: "base.in",
content: "authlib"
)
end
let(:manifest_file) do
Dependabot::DependencyFile.new(
name: "requirements/test.in",
content: "-r ../base.in\nruff==0.12.3"
)
end
let(:generated_file) do
Dependabot::DependencyFile.new(
name: "requirements/test.txt",
content: fixture("requirements", "pip_compile_unpinned.txt")
)
end
let(:dependency_name) { "ruff" }
let(:dependency_version) { "0.12.4" }
let(:dependency_previous_version) { "0.12.3" }
let(:dependency_requirements) do
[{
file: "requirements/test.in",
requirement: nil,
groups: [],
source: nil
}]
end
let(:dependency_previous_requirements) do
[{
file: "requirements/test.in",
requirement: nil,
groups: [],
source: nil
}]
end

it "preserves relative paths in generated comments (not absolute)", :slow do
updated_content = updated_files.first.content

# The key assertion: ensure that relative paths in comments are preserved
# and not converted to absolute paths like "/tmp/..." or "/home/..."
expect(updated_content).not_to match(%r{# via -r /}) # No absolute paths starting with /
expect(updated_content).not_to include("/tmp/") # No temp directory paths
expect(updated_content).not_to include("/home/") # No home directory paths
expect(updated_content).not_to include("/opt/") # No opt directory paths
end
end
end

describe "#package_hashes_for" do
Expand Down
Loading