search

rcaloras / bash-preexec

⚡ preexec and precmd functions for Bash just like Zsh.
MIT License
874 stars 93 forks source link

Fix a test for #131 #138

Closed akinomyoga closed 1 year ago

akinomyoga commented 1 year ago

This fixes an issue reported in https://github.com/rcaloras/bash-preexec/issues/121#issuecomment-1318281281 by @rycee:

Thanks, I tester just now with bats 1.8.2 and curiously I get

ok 19 preexec should execute a function with IFS defined to local scope
not ok 20 precmd should execute a function with IFS defined to local scope
# (in test file test/bash-preexec.bats, line 272)
#   `[ $status -eq 0 ]' failed
ok 21 preexec should set $? to be the exit code of preexec_function

This is an oversight of updating the test for change #131.

akinomyoga commented 1 year ago

Sorry, this is already fixed in the master. I withdraw this PR.