Skip to content

Commit

Permalink
ctdb-tests: Fix nonsense arguments to ps stub
Browse files Browse the repository at this point in the history
These were fine (though still lazy) when these tests were the only
user of this stub.  However, the ps stub is about to be enhanced, so
fix these uses of it to represent the intended usage.

Signed-off-by: Martin Schwenke <[email protected]>
Reviewed-by: Amitay Isaacs <[email protected]>
  • Loading branch information
martin-schwenke authored and Amitay Isaacs committed May 28, 2021
1 parent ffb56c9 commit a3e7fd9
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion ctdb/tests/UNIT/eventscripts/05.system.monitor.015.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ set_mem_usage 90 90
required_result 1 <<EOF
ERROR: System memory utilization 90% >= threshold 85%
$FAKE_PROC_MEMINFO
$(ps foobar)
$(ps auxfww)
EOF

simple_test
2 changes: 1 addition & 1 deletion ctdb/tests/UNIT/eventscripts/05.system.monitor.017.sh
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ set_mem_usage 87 87
required_result 1 <<EOF
ERROR: System memory utilization 87% >= threshold 80%
$FAKE_PROC_MEMINFO
$(ps foobar)
$(ps auxfww)
EOF

simple_test
4 changes: 2 additions & 2 deletions ctdb/tests/UNIT/eventscripts/05.system.monitor.018.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ set_mem_usage 80 80
required_result 1 <<EOF
ERROR: System memory utilization 80% >= threshold 80%
$FAKE_PROC_MEMINFO
$(ps foobar)
$(ps auxfww)
EOF
simple_test

Expand Down Expand Up @@ -72,7 +72,7 @@ set_mem_usage 81 81
required_result 1 <<EOF
ERROR: System memory utilization 81% >= threshold 80%
$FAKE_PROC_MEMINFO
$(ps foobar)
$(ps auxfww)
EOF
simple_test

Expand Down

0 comments on commit a3e7fd9

Please sign in to comment.