Skip to content
This repository has been archived by the owner on Jul 28, 2019. It is now read-only.

fix issue#2 (empty dependency installs) #17

Merged
merged 1 commit into from
Apr 15, 2016
Merged

fix issue#2 (empty dependency installs) #17

merged 1 commit into from
Apr 15, 2016

Conversation

rbuckheit
Copy link
Contributor

@rbuckheit rbuckheit commented Apr 15, 2016

summary

Fixes issue #2 (empty dependency installs cause 500 error). We now return an empty archive if user requests install of a package.json with no dependencies.

testing

  • Adds server end-to-end tests for the fixed test case
  • Adds server end-to-end tests for the /npm/hash route which was previously untested.
  • Adds server end-to-end tests for the DELETE /npm/install route which was previously untested.

@charleswli
Copy link

🌴

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants