Skip to content

Commit

Permalink
(#970) Default SCOPE_ALLOW_BINARY_CONSOLE was false, now is true.
Browse files Browse the repository at this point in the history
Fix the console integration test to account for this.
  • Loading branch information
jrcheli committed Jun 2, 2022
1 parent 39ac57c commit a0ed7bc
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions test/integration/console/scope-test
Original file line number Diff line number Diff line change
Expand Up @@ -77,12 +77,15 @@ fi
endtest

#
# cat binary file
# cat binary file with ALLOW_BINARY env false
# (ensure the $BINARY_MSG appears)
#

starttest cat_binary

export SCOPE_ALLOW_BINARY_CONSOLE=false
ldscope cat /bin/bash > /dev/null
unset SCOPE_ALLOW_BINARY_CONSOLE
evaltest

grep -- "$BINARY_MSG" $EVT_FILE > /dev/null
Expand All @@ -92,7 +95,7 @@ endtest


#
# cat binary file with ALLOW_BINARY env set
# cat binary file with ALLOW_BINARY env true
# (ensure the $BINARY_MSG does not appear)
#
starttest cat_binary_with_ALLOW_BINARY_env
Expand All @@ -109,6 +112,13 @@ fi

endtest

#########################################################################
#
# For all remaining tests, set SCOPE_ALLOW_BINARY_CONSOLE false
#
#########################################################################
export SCOPE_ALLOW_BINARY_CONSOLE=false


#
# curl png
Expand Down

0 comments on commit a0ed7bc

Please sign in to comment.