Skip to content

Commit

Permalink
dep cache uses lowercase keys
Browse files Browse the repository at this point in the history
  • Loading branch information
davidszotten committed Feb 4, 2016
1 parent 3a2de92 commit 2b925f1
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
2 changes: 1 addition & 1 deletion piptools/writer.py
Expand Up @@ -95,7 +95,7 @@ def _format_requirement(self, ireq, reverse_dependencies, primary_packages, incl
return line

# Annotate what packages this package is required by
required_by = reverse_dependencies.get(ireq.name, [])
required_by = reverse_dependencies.get(ireq.name.lower(), [])
if required_by:
line = line.ljust(24)
annotation = ', '.join(sorted(required_by))
Expand Down
10 changes: 10 additions & 0 deletions tests/test_writer.py
Expand Up @@ -32,6 +32,16 @@ def test_format_requirement_annotation(from_line, writer):
'test==1.2 ' + comment(' # via xyz'))


def test_format_requirement_annotation_case_sensitive(from_line, writer):
ireq = from_line('Test==1.2')
reverse_dependencies = {'test': ['xyz']}

assert (writer._format_requirement(ireq,
reverse_dependencies,
primary_packages=[]) ==
'Test==1.2 ' + comment(' # via xyz'))


def test_format_requirement_not_for_primary(from_line, writer):
"Primary packages should not get annotated."
ireq = from_line('test==1.2')
Expand Down

0 comments on commit 2b925f1

Please sign in to comment.