fix(complete): Restore nospace in bash

Author: mernen

clap_complete/src/env/shells.rs

@@ -41,12 +41,13 @@ _clap_complete_NAME() {
         _CLAP_IFS="$IFS" \
         _CLAP_COMPLETE_INDEX="$_CLAP_COMPLETE_INDEX" \
         _CLAP_COMPLETE_COMP_TYPE="$_CLAP_COMPLETE_COMP_TYPE" \
+        _CLAP_COMPLETE_SPACE="$_CLAP_COMPLETE_SPACE" \
         VAR="bash" \
         "COMPLETER" -- "${COMP_WORDS[@]}" \
     ) )
     if [[ $? != 0 ]]; then
         unset COMPREPLY
-    elif [[ $SUPPRESS_SPACE == 1 ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
+    elif [[ $_CLAP_COMPLETE_SPACE == false ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
         compopt -o nospace
     fi
 }

clap_complete/tests/snapshots/home/dynamic-env/exhaustive/bash/.bashrc

@@ -14,12 +14,13 @@ _clap_complete_exhaustive() {
         _CLAP_IFS="$IFS" \
         _CLAP_COMPLETE_INDEX="$_CLAP_COMPLETE_INDEX" \
         _CLAP_COMPLETE_COMP_TYPE="$_CLAP_COMPLETE_COMP_TYPE" \
+        _CLAP_COMPLETE_SPACE="$_CLAP_COMPLETE_SPACE" \
         COMPLETE="bash" \
         "exhaustive" -- "${COMP_WORDS[@]}" \
     ) )
     if [[ $? != 0 ]]; then
         unset COMPREPLY
-    elif [[ $SUPPRESS_SPACE == 1 ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
+    elif [[ $_CLAP_COMPLETE_SPACE == false ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
         compopt -o nospace
     fi
 }

clap_complete/tests/snapshots/register_minimal.bash

@@ -17,7 +17,7 @@ _clap_complete_my_app() {
     ) )
     if [[ $? != 0 ]]; then
         unset COMPREPLY
-    elif [[ $SUPPRESS_SPACE == 1 ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
+    elif [[ $_CLAP_COMPLETE_SPACE == false ]] && [[ "${COMPREPLY-}" =~ [=/:]$ ]]; then
         compopt -o nospace
     fi
 }

clap_complete/tests/testsuite/bash.rs

@@ -384,7 +384,7 @@ fn complete_dynamic_empty_option_value() {
 #[test]
 #[cfg(all(unix, feature = "unstable-dynamic"))]
 #[cfg(feature = "unstable-shell-tests")]
-fn complete_dynamic_dir_trailing_space() {
+fn complete_dynamic_dir_no_trailing_space() {
     if !common::has_command(CMD) {
         return;
     }
@@ -393,7 +393,7 @@ fn complete_dynamic_dir_trailing_space() {
     let mut runtime = common::load_runtime::<RuntimeBuilder>("dynamic-env", "exhaustive");
 
     let input = "exhaustive hint --file test\t";
-    let expected = snapbox::str!["exhaustive hint --file test     % exhaustive hint --file tests/ "];
+    let expected = snapbox::str!["exhaustive hint --file test     % exhaustive hint --file tests/"];
     let actual = runtime.complete(input, &term).unwrap();
     assert_data_eq!(actual, expected);
 }