Added missing tests to ensure - can be used as a long option

UrsaDK · Dec 14, 2024 · 3d04ca1 · 3d04ca1
1 parent 562d75c
commit 3d04ca1
Show file tree

Hide file tree

Showing 7 changed files with 226 additions and 0 deletions.
diff --git a/test/bats/github_26a.bats b/test/bats/github_26a.bats
@@ -0,0 +1,28 @@
+#!/usr/bin/env bats
+
+load ../test_helper
+export GETOPTS_LONG_TEST_BIN='getopts_long-longspec_with_dash'
+
+@test "${FEATURE}: toggle, silent" {
+    compare '-t' \
+            '---'
+}
+@test "${FEATURE}: toggle, verbose" {
+    compare '-t' \
+            '---'
+}
+
+@test "${FEATURE}: double toggle, silent" {
+    compare '-tt' \
+            '--- --toggle' \
+            '/^OPTIND: /d'
+    expect "${bash_getopts[6]}" == 'OPTIND: 2'
+    expect "${getopts_long[6]}" == 'OPTIND: 3'
+}
+@test "${FEATURE}: double toggle, verbose" {
+    compare '-tt' \
+            '--- --toggle' \
+            '/^OPTIND: /d'
+    expect "${bash_getopts[6]}" == 'OPTIND: 2'
+    expect "${getopts_long[6]}" == 'OPTIND: 3'
+}
diff --git a/test/bats/github_26b.bats b/test/bats/github_26b.bats
@@ -0,0 +1,30 @@
+#!/usr/bin/env bats
+
+load ../test_helper
+export GETOPTS_LONG_TEST_BIN='getopts_long-longspec_with_dash_colon'
+
+@test "${FEATURE}: option, silent" {
+    compare '-o user_val' \
+            '--- user_val'
+}
+@test "${FEATURE}: option, verbose" {
+    compare '-o user_val' \
+            '--- user_val'
+}
+
+# geopts_long does not support option-value adjoined syntax for long options.
+# Thus, let's compare it to bash getopts command that uses an invalid option.
+@test "${FEATURE}: option with adjacent value, silent" {
+    compare '-z' \
+            '---zz' \
+            '/^INVALID OPTION/d'
+    expect "${bash_getopts[1]}" == 'INVALID OPTION -- OPTARG=z'
+    expect "${getopts_long[1]}" == 'INVALID OPTION -- OPTARG=-zz'
+}
+@test "${FEATURE}: option with adjacent value, verbose" {
+    compare '-z' \
+            '---zz' \
+            '/-verbose: illegal option --/d'
+    expect "${bash_getopts[1]}" =~ '-verbose: illegal option -- z'
+    expect "${getopts_long[1]}" =~ '-verbose: illegal option -- -zz'
+}
diff --git a/test/bats/github_26c.bats b/test/bats/github_26c.bats
@@ -0,0 +1,22 @@
+#!/usr/bin/env bats
+
+load ../test_helper
+export GETOPTS_LONG_TEST_BIN='getopts_long-github_26'
+
+@test "${FEATURE}: toggle, silent" {
+    compare '-t user_arg' \
+            '---toggle user_arg'
+}
+@test "${FEATURE}: toggle, verbose" {
+    compare '-t user_arg' \
+            '---toggle user_arg'
+}
+
+@test "${FEATURE}: option, silent" {
+    compare '-o user_val' \
+            '---option user_val'
+}
+@test "${FEATURE}: option, verbose" {
+    compare '-o user_val' \
+            '---option user_val'
+}
diff --git a/test/bin/getopts_long-github_26-silent b/test/bin/getopts_long-github_26-silent
@@ -0,0 +1,38 @@
+#!/usr/bin/env bash
+
+TOPDIR="$(cd "$(dirname "${0}")"/../.. && pwd)"
+# shellcheck disable=SC1090
+source "${TOPDIR}/lib/getopts_long.bash"
+
+while getopts_long ':to:v: toggle option: variable: -toggle -option:' OPTKEY; do
+    case ${OPTKEY} in
+        't'|'toggle'|'-toggle')
+            printf 'toggle triggered'
+            ;;
+        '-'|'o'|'option'|'-option')
+            printf 'option supplied'
+            ;;
+        'v'|'variable')
+            printf 'value supplied'
+            ;;
+        '?')
+            printf "INVALID OPTION"
+            ;;
+        ':')
+            printf "MISSING ARGUMENT"
+            ;;
+        *)
+            printf "NEVER REACHED"
+            ;;
+    esac
+    printf ' -- '
+    [[ -z "${OPTARG+SET}" ]] && echo 'OPTARG is unset' || echo "OPTARG=${OPTARG}"
+done
+
+shift $(( OPTIND - 1 ))
+
+echo "OPTERR: ${OPTERR}"
+echo "OPTKEY: ${OPTKEY}"
+echo "OPTARG: ${OPTARG}"
+echo "OPTIND: ${OPTIND}"
+echo "\$@: ${*}"
diff --git a/test/bin/getopts_long-github_26-verbose b/test/bin/getopts_long-github_26-verbose
@@ -0,0 +1,35 @@
+#!/usr/bin/env bash
+
+TOPDIR="$(cd "$(dirname "${0}")"/../.. && pwd)"
+# shellcheck disable=SC1090
+source "${TOPDIR}/lib/getopts_long.bash"
+
+while getopts_long 'to:v: toggle option: variable: -toggle -option:' OPTKEY; do
+    case ${OPTKEY} in
+        't'|'toggle'|'-toggle')
+            printf 'toggle triggered'
+            ;;
+        '-'|'o'|'option'|'-option')
+            printf 'option supplied'
+            ;;
+        'v'|'variable')
+            printf 'value supplied'
+            ;;
+        '?')
+            printf "INVALID OPTION or MISSING ARGUMENT"
+            ;;
+        *)
+            printf "NEVER REACHED"
+            ;;
+    esac
+    printf ' -- '
+    [[ -z "${OPTARG+SET}" ]] && echo 'OPTARG is unset' || echo "OPTARG=${OPTARG}"
+done
+
+shift $(( OPTIND - 1 ))
+
+echo "OPTERR: ${OPTERR}"
+echo "OPTKEY: ${OPTKEY}"
+echo "OPTARG: ${OPTARG}"
+echo "OPTIND: ${OPTIND}"
+echo "\$@: ${*}"
diff --git a/test/bin/getopts_long-longspec_with_dash_colon-silent b/test/bin/getopts_long-longspec_with_dash_colon-silent
@@ -0,0 +1,38 @@
+#!/usr/bin/env bash
+
+TOPDIR="$(cd "$(dirname "${0}")"/../.. && pwd)"
+# shellcheck disable=SC1090
+source "${TOPDIR}/lib/getopts_long.bash"
+
+while getopts_long ':to:v: -: toggle option: variable:' OPTKEY; do
+    case ${OPTKEY} in
+        't'|'toggle')
+            printf 'toggle triggered'
+            ;;
+        '-'|'o'|'option')
+            printf 'option supplied'
+            ;;
+        'v'|'variable')
+            printf 'value supplied'
+            ;;
+        '?')
+            printf "INVALID OPTION"
+            ;;
+        ':')
+            printf "MISSING ARGUMENT"
+            ;;
+        *)
+            printf "NEVER REACHED"
+            ;;
+    esac
+    printf ' -- '
+    [[ -z "${OPTARG+SET}" ]] && echo 'OPTARG is unset' || echo "OPTARG=${OPTARG}"
+done
+
+shift $(( OPTIND - 1 ))
+
+echo "OPTERR: ${OPTERR}"
+echo "OPTKEY: ${OPTKEY}"
+echo "OPTARG: ${OPTARG}"
+echo "OPTIND: ${OPTIND}"
+echo "\$@: ${*}"
diff --git a/test/bin/getopts_long-longspec_with_dash_colon-verbose b/test/bin/getopts_long-longspec_with_dash_colon-verbose
@@ -0,0 +1,35 @@
+#!/usr/bin/env bash
+
+TOPDIR="$(cd "$(dirname "${0}")"/../.. && pwd)"
+# shellcheck disable=SC1090
+source "${TOPDIR}/lib/getopts_long.bash"
+
+while getopts_long 'to:v: -: toggle option: variable:' OPTKEY; do
+    case ${OPTKEY} in
+        't'|'toggle')
+            printf 'toggle triggered'
+            ;;
+        '-'|'o'|'option')
+            printf 'option supplied'
+            ;;
+        'v'|'variable')
+            printf 'value supplied'
+            ;;
+        '?')
+            printf "INVALID OPTION or MISSING ARGUMENT"
+            ;;
+        *)
+            printf "NEVER REACHED"
+            ;;
+    esac
+    printf ' -- '
+    [[ -z "${OPTARG+SET}" ]] && echo 'OPTARG is unset' || echo "OPTARG=${OPTARG}"
+done
+
+shift $(( OPTIND - 1 ))
+
+echo "OPTERR: ${OPTERR}"
+echo "OPTKEY: ${OPTKEY}"
+echo "OPTARG: ${OPTARG}"
+echo "OPTIND: ${OPTIND}"
+echo "\$@: ${*}"