Skip to content

Commit

Permalink
Review changes on the grep_pipe function
Browse files Browse the repository at this point in the history
Just renaming and unification with the grep command.
  • Loading branch information
gbladislau committed Jun 13, 2024
1 parent c76b6c0 commit 2abe149
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions aexpect/ops_linux.py
Original file line number Diff line number Diff line change
Expand Up @@ -432,22 +432,22 @@ def grep(session, expr, path, check=False, flags="-a"):
return output


def grep_pipe(session, cmd, pattern, flags = "", check = False):
def grep_pipe(session, cmd, expr, check = False, flags ="-a"):
"""
Invoke ``grep`` remotely searching for an expression in a command output.
:param session: vm session
:param session: session to run the command on
:type session: ShellSession
:param str cmd: command that its output will be grepped
:param str pattern: pattern to grep
:param str flags: extra flags passed to ``grep`` on the command line
:param str cmd: command whose output will be grepped
:param str expr: search expression
:param bool check: whether to quietly run grep for a boolean check
:param str flags: extra flags passed to ``grep`` on the command line
:returns: whether there is a match or not or what ``grep`` emits on stdout
if the check mode is disabled
:rtype: bool or str
:raises: ShellCmdError if the check mode is disabled and status is nonzero
"""
grep_command = f"{cmd} | grep {flags} '{pattern}'"
grep_command = f"{cmd} | grep {flags} '{expr}'"
status, output = session.cmd_status_output(grep_command)
if check:
return status == 0
Expand Down

0 comments on commit 2abe149

Please sign in to comment.