autotest: Fix kernel module unit test teardown
authorChristoph Jaeger <cj@linux.com>
Wed, 28 Jan 2015 19:02:42 +0000 (14:02 -0500)
committerAndy Zhou <azhou@nicira.com>
Thu, 29 Jan 2015 05:26:06 +0000 (21:26 -0800)
commitebdabd05e9465dd6e50adedb4a0bcb5678e563de
tree9e8fa47362502ffa990cea57aad34ca15a2ce690
parentd69d61c7c17504dd59a851247d3fa1b7e88f55c1
autotest: Fix kernel module unit test teardown

Due to a misnaming, the macro calls

  OVS_SWITCHD_STOP([$1])
  AT_CHECK([modprobe -r openvswitch])

in OVS_KMOD_VSWITCHD_STOP make up a syntactically correct function definition
(OVS_SWITCHD_STOP does not exist, and therefore the call does not expand):

  OVS_SWITCHD_STOP()
  { set +x
  $as_echo "$at_srcdir/kmod-traffic.at:15: modprobe -r openvswitch"
  ...
  $at_traceon; }

Consequently, neither of the calls has the intended effect, i.e., stopping
ovs-vswitchd and ovsdb-server, checking their log files, and unloading the
datapath kernel module. Fix the misnaming, so all calls expand properly.

Fixes: 69c2bdfef9 ("autotest: add autotest framework for adding kernel module unit tests")
Signed-off-by: Christoph Jaeger <cj@linux.com>
Signed-off-by: Andy Zhou <azhou@nicira.com>
Author: add Christoph Jaeger
AUTHORS
tests/kmod-macros.at