Merge pull request #85 from ryneeverett/fix-upgrade-exit

Return True on successfull Repo.upgrade().
mitsuhiko · Dec 26, 2016 · 5f5feb8 · 5f5feb8
2 parents ad72771 + 1f40fcd
commit 5f5feb8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/pipsi.py b/pipsi.py
@@ -304,6 +304,8 @@ def upgrade(self, package, editable=False):
                 except (IOError, OSError):
                     pass
 
+        return True
+
     def list_everything(self):
         venvs = {}
         python = '/Scripts/python.exe' if IS_WIN else '/bin/python'

diff --git a/testing/test_repo.py b/testing/test_repo.py
@@ -34,6 +34,7 @@ def test_simple_install(repo, home, bin, package, glob):
     repo.install(package)
     assert home.join(package).check()
     assert bin.listdir(glob)
+    assert repo.upgrade(package)
 
 
 @pytest.mark.xfail(sys.version_info[0] != 3,