From 5932ce17e128c096fcc56eb04b27e780da3cf255 Mon Sep 17 00:00:00 2001 From: Dave Barach Date: Wed, 9 Oct 2019 17:28:04 -0400 Subject: tests: add cli_return_response to vpp_papi_provider To improve gcov/lcov code coverage stats, it's necessary to send incorrect debug CLI commands; to force vpp into debug CLI error paths. cli_return_response() sends commands and returns the response object, so test vectors can handle failures. Type: feature Signed-off-by: Dave Barach Change-Id: I4fab591c9d2e30c996f016e18e4fd69b9c5bed06 --- test/vpp_papi_provider.py | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) (limited to 'test') diff --git a/test/vpp_papi_provider.py b/test/vpp_papi_provider.py index 8cb7c971758..12dcf1081ab 100644 --- a/test/vpp_papi_provider.py +++ b/test/vpp_papi_provider.py @@ -334,17 +334,28 @@ class VppPapiProvider(object): self.hook.after_api(api_fn.__name__, api_args) return reply - def cli(self, cli): + def cli_return_response(self, cli): """ Execute a CLI, calling the before/after hooks appropriately. + Return the reply without examining it :param cli: CLI to execute - :returns: CLI output + :returns: response object """ self.hook.before_cli(cli) cli += '\n' r = self.papi.cli_inband(cmd=cli) self.hook.after_cli(cli) + return r + + def cli(self, cli): + """ Execute a CLI, calling the before/after hooks appropriately. + + :param cli: CLI to execute + :returns: CLI output + + """ + r = self.cli_return_response(cli) if r.retval == -156: raise CliSyntaxError(r.reply) if r.retval != 0: -- cgit 1.2.3-korg