Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.2] Fix db modes test #507

Merged
merged 8 commits into from
Jun 23, 2022
Merged

[3.2] Fix db modes test #507

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d93a276
Allow the test script go ahead in case of $NODEOS_PID already shutdow…
softprofe Apr 11, 2021
3cfeabc
Add check to wait return code, if return code is 0 or 127 just let it…
softprofe Apr 13, 2021
9cce035
Return a return code in any case for the function run_expect_success()
softprofe Apr 13, 2021
b92a6a1
As the wait return code is likely to be $NODEOS_PID (stopped child re…
softprofe May 3, 2021
cd64609
label db_modes_test as nonparallelizable_tests to fix Mac issue
praphael Aug 27, 2021
170217b
remove temp file
praphael Aug 27, 2021
9bd1770
Revert "label db_modes_test as nonparallelizable_tests to fix Mac issue"
linh2931 Jun 23, 2022
1e528a8
Merge branch 'main' into fix_db_modes_test
linh2931 Jun 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion tests/db_modes_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,12 @@ run_expect_success() {
local NODEOS_PID=$!
sleep 10
kill $NODEOS_PID
wait $NODEOS_PID
rc=0
wait $NODEOS_PID && rc=$? || rc=$?
if [[ $rc -eq 127 || $rc -eq $NODEOS_PID ]]; then
rc=0
fi
return $rc
}

run_and_kill() {
Expand Down