#16 Fix 'rhizomeops' tests that fail on Solaris

This commit is contained in:
Andrew Bettison 2012-09-13 14:08:52 +09:30
parent 3f4ee701ec
commit 1f4688db99
4 changed files with 24 additions and 24 deletions

View File

@ -523,8 +523,8 @@ int monitor_announce_bundle(rhizome_manifest *m)
service ? service : "", service ? service : "",
m->version, m->version,
m->fileLength, m->fileLength,
sender, sender ? sender : "",
recipient, recipient ? recipient : "",
m->dataFileName?m->dataFileName:""); m->dataFileName?m->dataFileName:"");
for(i=monitor_socket_count -1;i>=0;i--) { for(i=monitor_socket_count -1;i>=0;i--) {
if (monitor_sockets[i].flags & MONITOR_RHIZOME) { if (monitor_sockets[i].flags & MONITOR_RHIZOME) {

View File

@ -37,7 +37,7 @@ extract_stdout_keyvalue_optional() {
local _var="$1" local _var="$1"
local _label="$2" local _label="$2"
local _rexp="$3" local _rexp="$3"
local _line=$(replayStdout | grep "^$_label:") local _line=$(replayStdout | $GREP "^$_label:")
local _value= local _value=
local _return=1 local _return=1
if [ -n "$_line" ]; then if [ -n "$_line" ]; then
@ -502,7 +502,7 @@ instances_see_each_other() {
[ $I = $J ] && continue [ $I = $J ] && continue
local logvar=LOG${I#+} local logvar=LOG${I#+}
local sidvar=SID${J#+} local sidvar=SID${J#+}
if ! grep "ADD OVERLAY NODE sid=${!sidvar}" "${!logvar}"; then if ! $GREP "ADD OVERLAY NODE sid=${!sidvar}" "${!logvar}"; then
return 1 return 1
fi fi
done done

View File

@ -81,12 +81,12 @@ assert_manifest_complete() {
assertGrep "$manifest" "^date=$rexp_date\$" assertGrep "$manifest" "^date=$rexp_date\$"
assertGrep "$manifest" "^version=$rexp_version\$" assertGrep "$manifest" "^version=$rexp_version\$"
assertGrep "$manifest" "^filesize=$rexp_filesize\$" assertGrep "$manifest" "^filesize=$rexp_filesize\$"
if grep -q '^filesize=0$' "$manifest"; then if $GREP -q '^filesize=0$' "$manifest"; then
assertGrep --matches=0 "$manifest" "^filehash=" assertGrep --matches=0 "$manifest" "^filehash="
else else
assertGrep "$manifest" "^filehash=$rexp_filehash\$" assertGrep "$manifest" "^filehash=$rexp_filehash\$"
fi fi
if grep -q '^service=file$' "$manifest"; then if $GREP -q '^service=file$' "$manifest"; then
assertGrep "$manifest" "^name=" assertGrep "$manifest" "^name="
fi fi
} }
@ -112,7 +112,7 @@ assert_stdout_add_file() {
shift shift
unpack_manifest_for_grep "$filename" unpack_manifest_for_grep "$filename"
opt_name=false opt_name=false
if replayStdout | grep -q '^service:file$'; then if replayStdout | $GREP -q '^service:file$'; then
opt_name=true opt_name=true
fi fi
fieldnames='service|manifestid|secret|filesize|filehash|name' fieldnames='service|manifestid|secret|filesize|filehash|name'
@ -136,7 +136,7 @@ assert_stdout_add_file() {
${opt_manifestid:-true} && assertStdoutGrep --matches=1 "^manifestid:$re_manifestid\$" ${opt_manifestid:-true} && assertStdoutGrep --matches=1 "^manifestid:$re_manifestid\$"
${opt_secret:-true} && assertStdoutGrep --matches=1 "^secret:$re_secret\$" ${opt_secret:-true} && assertStdoutGrep --matches=1 "^secret:$re_secret\$"
${opt_filesize:-true} && assertStdoutGrep --matches=1 "^filesize:$re_filesize\$" ${opt_filesize:-true} && assertStdoutGrep --matches=1 "^filesize:$re_filesize\$"
if replayStdout | grep -q '^filesize:0$'; then if replayStdout | $GREP -q '^filesize:0$'; then
assertStdoutGrep --matches=0 "^filehash:" assertStdoutGrep --matches=0 "^filehash:"
else else
${opt_filehash:-true} && assertStdoutGrep --matches=1 "^filehash:$re_filehash\$" ${opt_filehash:-true} && assertStdoutGrep --matches=1 "^filehash:$re_filehash\$"
@ -155,7 +155,7 @@ unpack_manifest_for_grep() {
re_version="$rexp_version" re_version="$rexp_version"
re_secret="$rexp_bundlesecret" re_secret="$rexp_bundlesecret"
re_name=$(escape_grep_basic "${filename##*/}") re_name=$(escape_grep_basic "${filename##*/}")
local filesize=$(sed -n -e '/^filesize=/s///p' "$filename.manifest" 2>/dev/null) local filesize=$($SED -n -e '/^filesize=/s///p' "$filename.manifest" 2>/dev/null)
if [ "$filesize" = 0 ]; then if [ "$filesize" = 0 ]; then
re_filesize=0 re_filesize=0
re_filehash= re_filehash=
@ -166,17 +166,17 @@ unpack_manifest_for_grep() {
fi fi
# If there is a manifest file that looks like it matches this payload # If there is a manifest file that looks like it matches this payload
# file, then use its file hash to check the rhizome list '' output. # file, then use its file hash to check the rhizome list '' output.
local filehash=$(sed -n -e '/^filehash=/s///p' "$filename.manifest" 2>/dev/null) local filehash=$($SED -n -e '/^filehash=/s///p' "$filename.manifest" 2>/dev/null)
if [ "$filehash" = "$re_filehash" ]; then if [ "$filehash" = "$re_filehash" ]; then
re_manifestid=$(sed -n -e '/^id=/s///p' "$filename.manifest") re_manifestid=$($SED -n -e '/^id=/s///p' "$filename.manifest")
re_version=$(sed -n -e '/^version=/s///p' "$filename.manifest") re_version=$($SED -n -e '/^version=/s///p' "$filename.manifest")
re_service=$(sed -n -e '/^service=/s///p' "$filename.manifest") re_service=$($SED -n -e '/^service=/s///p' "$filename.manifest")
re_service=$(escape_grep_basic "$re_service") re_service=$(escape_grep_basic "$re_service")
re_sender=$(sed -n -e '/^sender=/s///p' "$filename.manifest") re_sender=$($SED -n -e '/^sender=/s///p' "$filename.manifest")
re_recipient=$(sed -n -e '/^recipient=/s///p' "$filename.manifest") re_recipient=$($SED -n -e '/^recipient=/s///p' "$filename.manifest")
case "$re_service" in case "$re_service" in
file) file)
re_name=$(sed -n -e '/^name=/s///p' "$filename.manifest") re_name=$($SED -n -e '/^name=/s///p' "$filename.manifest")
re_name=$(escape_grep_basic "$re_name") re_name=$(escape_grep_basic "$re_name")
;; ;;
*) *)
@ -201,7 +201,7 @@ assert_manifest_newer() {
strip_signatures() { strip_signatures() {
for file; do for file; do
cat -v "$file" | sed -e '/^^@/,$d' >"tmp.$file" && mv -f "tmp.$file" "$file" cat -v "$file" | $SED -e '/^^@/,$d' >"tmp.$file" && mv -f "tmp.$file" "$file"
done done
} }
@ -218,7 +218,7 @@ extract_manifest() {
local _manifestfile="$2" local _manifestfile="$2"
local _label="$3" local _label="$3"
local _rexp="$4" local _rexp="$4"
local _value=$(sed -n -e "/^$_label=$_rexp\$/s/^$_label=//p" "$_manifestfile") local _value=$($SED -n -e "/^$_label=$_rexp\$/s/^$_label=//p" "$_manifestfile")
assert --message="$_manifestfile contains valid '$_label=' line" \ assert --message="$_manifestfile contains valid '$_label=' line" \
--dump-on-fail="$_manifestfile" \ --dump-on-fail="$_manifestfile" \
[ -n "$_value" ] [ -n "$_value" ]
@ -264,13 +264,13 @@ compute_filehash() {
rhizome_http_server_started() { rhizome_http_server_started() {
local logvar=LOG${1#+} local logvar=LOG${1#+}
grep 'RHIZOME HTTP SERVER,.*START.*port=[0-9]' "${!logvar}" $GREP 'RHIZOME HTTP SERVER,.*START.*port=[0-9]' "${!logvar}"
} }
get_rhizome_server_port() { get_rhizome_server_port() {
local _var="$1" local _var="$1"
local _logvar=LOG${2#+} local _logvar=LOG${2#+}
local _port=$(sed -n -e '/.*RHIZOME HTTP SERVER.*START/{s/.*port=\([0-9]\{1,\}\).*/\1/p;q}' "${!_logvar}") local _port=$($SED -n -e '/.*RHIZOME HTTP SERVER.*START/{s/.*port=\([0-9]\{1,\}\).*/\1/p;q}' "${!_logvar}")
assert --message="instance $2 Rhizome HTTP server port number is known" [ -n "$_port" ] assert --message="instance $2 Rhizome HTTP server port number is known" [ -n "$_port" ]
if [ -n "$_var" ]; then if [ -n "$_var" ]; then
eval "$_var=\$_port" eval "$_var=\$_port"

View File

@ -328,7 +328,7 @@ setup_AddUpdateSameVersion() {
setup_AddDuplicate setup_AddDuplicate
cp file1.manifest file1_2.manifest cp file1.manifest file1_2.manifest
strip_signatures file1_2.manifest strip_signatures file1_2.manifest
sed -i -e '/^date=/d;/^filehash=/d;/^filesize=/d' file1_2.manifest $SED -i -e '/^date=/d;/^filehash=/d;/^filesize=/d' file1_2.manifest
tfw_cat -v file1_2.manifest tfw_cat -v file1_2.manifest
assertGrep --matches=0 file1_2.manifest '^filehash=' assertGrep --matches=0 file1_2.manifest '^filehash='
extract_manifest_version '' file1_2.manifest # asserts has version= line extract_manifest_version '' file1_2.manifest # asserts has version= line
@ -351,7 +351,7 @@ setup_AddUpdateNewVersion() {
setup_AddUpdateSameVersion setup_AddUpdateSameVersion
extract_manifest_version version file1_2.manifest extract_manifest_version version file1_2.manifest
let version=version+1 let version=version+1
sed -i -e "/^version=/s/=.*/=$version/" file1_2.manifest $SED -i -e "/^version=/s/=.*/=$version/" file1_2.manifest
assertGrep --matches=1 file1_2.manifest "^version=$version$" assertGrep --matches=1 file1_2.manifest "^version=$version$"
} }
test_AddUpdateNewVersion() { test_AddUpdateNewVersion() {
@ -367,7 +367,7 @@ test_AddUpdateNewVersion() {
doc_AddUpdateNoAuthor="Cannot add new payload to authorless manifest" doc_AddUpdateNoAuthor="Cannot add new payload to authorless manifest"
setup_AddUpdateNoAuthor() { setup_AddUpdateNoAuthor() {
setup_AddUpdateNewVersion setup_AddUpdateNewVersion
sed -i -e '/^BK=/d' file1_2.manifest $SED -i -e '/^BK=/d' file1_2.manifest
} }
test_AddUpdateNoAuthor() { test_AddUpdateNoAuthor() {
tfw_cat -v file1_2.manifest tfw_cat -v file1_2.manifest
@ -395,7 +395,7 @@ test_AddUpdateNoAuthorWithSecret() {
doc_AddUpdateAutoVersion="Add new payload to existing manifest with automatic version" doc_AddUpdateAutoVersion="Add new payload to existing manifest with automatic version"
setup_AddUpdateAutoVersion() { setup_AddUpdateAutoVersion() {
setup_AddUpdateSameVersion setup_AddUpdateSameVersion
sed -i -e '/^version=/d' file1_2.manifest $SED -i -e '/^version=/d' file1_2.manifest
assertGrep --matches=0 file1_2.manifest '^version=' assertGrep --matches=0 file1_2.manifest '^version='
} }
test_AddUpdateAutoVersion() { test_AddUpdateAutoVersion() {