[Fix] `nvm install -s`: npm detection

- add `nvm_use_if_needed`
 - add `nvm_install_npm_if_needed`

nvm.sh

@@ -1926,6 +1926,24 @@ nvm_install_source() {
     command rm -f "${VERSION_PATH}" 2>/dev/null && \
     $make -j "${NVM_MAKE_JOBS}" ${MAKE_CXX-} install
   ); then
+    return $?
+  fi
+
+  nvm_err "nvm: install ${VERSION} failed!"
+  command rm -rf "${TMPDIR-}"
+  return 1
+}
+
+nvm_use_if_needed() {
+  if [ "_${1-}" = "_$(nvm_ls_current)" ]; then
+    return
+  fi
+  nvm use "$@"
+}
+
+nvm_install_npm_if_needed() {
+  local VERSION
+  VERSION="$(nvm_ls_current)"
   if ! nvm_has "npm"; then
     nvm_echo 'Installing npm...'
     if nvm_version_greater 0.2.0 "$VERSION"; then
@@ -1941,11 +1959,6 @@ nvm_install_source() {
     fi
   fi
   return $?
-  fi
-
-  nvm_err "nvm: install ${VERSION} failed!"
-  command rm -rf "${TMPDIR-}"
-  return 1
 }
 
 nvm_match_version() {
@@ -2505,7 +2518,7 @@ nvm() {
 
       fi
 
-      if [ "$EXIT_CODE" -eq 0 ] && nvm use "$VERSION"; then
+      if [ "$EXIT_CODE" -eq 0 ] && nvm_use_if_needed "${VERSION}" && nvm_install_npm_if_needed "${VERSION}"; then
         if [ -n "${LTS-}" ]; then
           nvm_ensure_default_set "lts/${LTS}"
         else
@@ -2516,6 +2529,8 @@ nvm() {
           nvm reinstall-packages "$REINSTALL_PACKAGES_FROM"
           EXIT_CODE=$?
         fi
+      else
+        EXIT_CODE=$?
       fi
       return $EXIT_CODE
     ;;
@@ -3216,9 +3231,9 @@ nvm() {
         nvm_is_iojs_version nvm_is_alias nvm_has_non_aliased \
         nvm_ls_remote nvm_ls_remote_iojs nvm_ls_remote_index_tab \
         nvm_ls nvm_remote_version nvm_remote_versions \
-        nvm_install_binary nvm_clang_version \
+        nvm_install_binary nvm_install_source nvm_clang_version \
         nvm_get_mirror nvm_get_download_slug nvm_download_artifact \
-        nvm_install_source nvm_check_file_permissions \
+        nvm_install_npm_if_needed nvm_use_if_needed nvm_check_file_permissions \
         nvm_print_versions nvm_compute_checksum nvm_checksum \
         nvm_get_checksum_alg nvm_get_checksum nvm_compare_checksum \
         nvm_version nvm_rc_version nvm_match_version \

test/fast/Unit tests/nvm_use_if_needed

@@ -0,0 +1,24 @@
+#!/bin/sh
+
+cleanup() { unset -f nvm_ls_current nvm; }
+die () { echo "$@" ; cleanup ; exit 1; }
+
+\. ../../../nvm.sh
+
+nvm_ls_current() { echo foo; }
+
+nvm() {
+  echo "nvm: $@"
+}
+
+OUTPUT="$(nvm_use_if_needed foo)"
+EXPECTED_OUTPUT=''
+
+[ "${OUTPUT}" = "${EXPECTED_OUTPUT}" ] || die "expected >${EXPECTED_OUTPUT}<; got >${OUTPUT}<"
+
+OUTPUT="$(nvm_use_if_needed bar)"
+EXPECTED_OUTPUT='nvm: use bar'
+
+[ "${OUTPUT}" = "${EXPECTED_OUTPUT}" ] || die "expected >${EXPECTED_OUTPUT}<; got >${OUTPUT}<"
+
+cleanup

test/installation_node/install hook

@@ -23,8 +23,7 @@ OUTPUT="$(NVM_INSTALL_THIRD_PARTY_HOOK=succeed nvm install "${VERSION}")"
 USE_OUTPUT="$(nvm use "${VERSION}")"
 EXPECTED_OUTPUT="${VERSION} node std binary ${VERSION_PATH}
 Downloading and installing node ${VERSION}...
-${USE_OUTPUT}
-${USE_OUTPUT}" # double use output is from the normal install in succeed()
+${USE_OUTPUT}"
 
 [ "${OUTPUT}" = "${EXPECTED_OUTPUT}" ] || die "expected >${EXPECTED_OUTPUT}<; got >${OUTPUT}<"