[Fix] nvm_normalize_lts: error when an LTS name is not lowercase

nvm-sh · Sep 4, 2024 · ea126a6 · ea126a6
1 parent 9a28dbd
commit ea126a6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/nvm.sh b/nvm.sh
@@ -891,6 +891,10 @@ nvm_normalize_lts() {
  fi
  ;;
  *)
+ if [ "${LTS}" != "$(echo "${LTS}" | command tr '[:upper:]' '[:lower:]')" ]; then
+ nvm_err 'LTS names must be lowercase'
+ return 3
+ fi
  nvm_echo "${LTS}"
  ;;
  esac

diff --git a/test/fast/Unit tests/nvm_normalize_lts b/test/fast/Unit tests/nvm_normalize_lts
@@ -16,6 +16,9 @@ ACTUAL="$(nvm_normalize_lts "lts/*")"
 EXPECTED='lts/*'
 [ "${ACTUAL}" = "${EXPECTED}" ] || die "expected >${EXPECTED}<, got >${ACTUAL}<"
 
+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")"