Merge f4c7c77fe3 into 6b70c40f15

Moved issue template into ISSUE_TEMPLATE folder

nvm.sh

@@ -196,6 +196,9 @@ nvm_install_latest_npm() {
   nvm_echo 'Attempting to upgrade to the latest working version of npm...'
   local NODE_VERSION
   NODE_VERSION="$(nvm_strip_iojs_prefix "$(nvm_ls_current)")"
+  local NPM_VERSION
+  NPM_VERSION="$(npm --version 2>/dev/null)"
+
   if [ "${NODE_VERSION}" = 'system' ]; then
     NODE_VERSION="$(node --version)"
   elif [ "${NODE_VERSION}" = 'none' ]; then
@@ -206,8 +209,6 @@ nvm_install_latest_npm() {
     nvm_err 'Unable to obtain node version.'
     return 1
   fi
-  local NPM_VERSION
-  NPM_VERSION="$(npm --version 2>/dev/null)"
   if [ -z "${NPM_VERSION}" ]; then
     nvm_err 'Unable to obtain npm version.'
     return 2
@@ -700,10 +701,12 @@ nvm_ensure_version_installed() {
       nvm_err "N/A: version \"${PREFIXED_VERSION:-$PROVIDED_VERSION}\" is not yet installed."
     fi
     nvm_err ""
-    if [ "${IS_VERSION_FROM_NVMRC}" != '1' ]; then
+    if [ "${PROVIDED_VERSION}" = 'lts' ]; then
-        nvm_err "You need to run \`nvm install ${PROVIDED_VERSION}\` to install and use it."
+      nvm_err '`lts` is not an alias - you may need to run `nvm install --lts` to install and `nvm use --lts` to use it.'
-      else
+    elif [ "${IS_VERSION_FROM_NVMRC}" != '1' ]; then
-        nvm_err 'You need to run `nvm install` to install and use the node version specified in `.nvmrc`.'
+      nvm_err "You need to run \`nvm install ${PROVIDED_VERSION}\` to install and use it."
+    else
+      nvm_err 'You need to run `nvm install` to install and use the node version specified in `.nvmrc`.'
     fi
     return 1
   fi

test/fast/Unit tests/nvm_alias LTS-N

@@ -5,14 +5,26 @@ die () { echo "$@" ; cleanup ; exit 1; }
 
 cleanup() {
   unset -f nvm_download
+  rm -rf "${NVM_DIR}/alias/lts"
+  mv "${NVM_DIR}/alias/lts-backup" "${NVM_DIR}/alias/lts"
 }
 
 \. ../../../nvm.sh
 
-set -ex
-
 MOCKS_DIR="${PWD}/mocks"
 
+LTS_NAMES_PATH="${MOCKS_DIR}/LTS_names.txt"
+LTS_LIST="$(cat "${LTS_NAMES_PATH}" | tail -n +2)"
+
+mv "${NVM_DIR}/alias/lts" "${NVM_DIR}/alias/lts-backup" ||:
+mkdir -p "${NVM_DIR}/alias/lts"
+
+for LTS in $LTS_LIST; do
+  cp "${NVM_DIR}/alias/lts-backup/${LTS}" "${NVM_DIR}/alias/lts/"
+done
+
+set -ex
+
 # sample output at the time the test was written
 TAB_PATH="${MOCKS_DIR}/nodejs.org-dist-index.tab"
 nvm_download() {
@@ -21,8 +33,6 @@ nvm_download() {
 
 nvm_ls_remote >/dev/null || die "nvm_ls_remote_failed?!"
 
-LTS_NAMES_PATH="${MOCKS_DIR}/LTS_names.txt"
-
 N=0
 while IFS= read -r LTS; do
   if [ $N -gt 0 ]; then

test/fast/Unit tests/nvm_normalize_lts

@@ -2,12 +2,26 @@
 
 cleanup() {
     unset nvm_get_os
+    rm -rf "${NVM_DIR}/alias/lts"
+    mv "${NVM_DIR}/alias/lts-backup" "${NVM_DIR}/alias/lts"
 }
 
 die () { cleanup; echo "$@" ; exit 1; }
 
 \. ../../../nvm.sh
 
+MOCKS_DIR="../Unit tests/mocks"
+
+LTS_NAMES_PATH="${MOCKS_DIR}/LTS_names.txt"
+LTS_LIST="$(cat "${LTS_NAMES_PATH}" | tail -n +2)"
+
+mv "${NVM_DIR}/alias/lts" "${NVM_DIR}/alias/lts-backup" ||:
+mkdir -p "${NVM_DIR}/alias/lts"
+
+for LTS in $LTS_LIST; do
+  cp "${NVM_DIR}/alias/lts-backup/${LTS}" "${NVM_DIR}/alias/lts/"
+done
+
 ACTUAL="$(nvm_normalize_lts "foo")"
 EXPECTED='foo'
 [ "${ACTUAL}" = "${EXPECTED}" ] || die "expected >${EXPECTED}<, got >${ACTUAL}<"
@@ -20,15 +34,14 @@ if ACTUAL="$(nvm_normalize_lts lts/ARGON)"; then
   die "expected failure, got >${ACTUAL}<"
 fi
 
-MOCKS_DIR="../Unit tests/mocks"
+STAR="$(cat "${MOCKS_DIR}/lts-star.txt")"
-STAR="$(cat "$MOCKS_DIR/lts-star.txt")"
 
 ACTUAL="$(nvm_normalize_lts "${STAR}")"
 EXPECTED="${STAR}"
 [ "${ACTUAL}" = "${EXPECTED}" ] || die "expected >${EXPECTED}<, got >${ACTUAL}<"
 
-LTS_NAMES_PATH="${MOCKS_DIR}/LTS_names.txt"
+# or, back up LTS aliases, copy backups that are in LTS names
-LTS_LIST="$(cat "${LTS_NAMES_PATH}" | tail -n +2)"
+#
 
 INDEX=1
 printf '%s\n' "${LTS_LIST}" | while IFS= read -r LTS; do

test/slow/nvm use/Running 'nvm use lts' shows actionable error

@@ -0,0 +1,19 @@
+#!/bin/sh
+
+set -ex
+
+die () { echo "$@" ; exit 1; }
+
+\. ../../../nvm.sh
+
+# Deactivate any active node version
+nvm deactivate >/dev/null 2>&1 || die 'deactivate failed'
+
+# Attempt to use 'lts' without '--' and capture the error message
+ERROR_OUTPUT=$(nvm use lts 2>&1) || true
+
+EXPECTED_ERROR='`lts` is not an alias - you may need to run `nvm install --lts` to install and `nvm use --lts` to use it.'
+
+# Check if the error message matches the expected output
+echo "$ERROR_OUTPUT" | grep -q "$EXPECTED_ERROR" \
+  || die "Expected error message not found. Got: $ERROR_OUTPUT"