Bug 781221

Summary: mhsh exception when running "sleep"
Product: [Retired] Matahari Reporter: Russell Bryant <rbryant>
Component: matahariAssignee: Zane Bitter <zbitter>
Status: CLOSED WONTFIX QA Contact: Dave Johnson <dajohnso>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.6CC: astokes, matahari-maint, rbryant, whayutin, zbitter
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-07-17 12:30:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Russell Bryant 2012-01-13 01:48:37 UTC
When running "sleep" without an argument, an exception is raised.  I realize that this is invalid input, but I wouldn't expect any Python exceptions to be shown in the shell.

mhsh> sleep 
% Missing arguments: 'FLOAT'

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/matahari/shell/interpreter/interpreter.py", line 81, in onecmd
    return cmd.Cmd.onecmd(self, line)
  File "/usr/lib64/python2.7/cmd.py", line 219, in onecmd
    return func(arg)
  File "/usr/lib/python2.7/site-packages/matahari/shell/interpreter/command.py", line 185, in __call__
    message="Missing arguments: '%s'" % miss)
InvalidCommandException: Missing arguments: 'FLOAT'

Comment 1 Russell Bryant 2012-01-13 01:50:00 UTC
The same goes for "list", "show", "select", "clear"

Comment 2 Zane Bitter 2012-01-13 09:26:49 UTC
This is expected behaviour as per the release announcement:
"Currently debugging is turned on, which means that all exceptions are printed out in full. This will eventually be disabled, but for now if you see one that you think is incorrect, please report it and include the stack trace."
https://fedorahosted.org/pipermail/matahari/2011-December/002270.html

However I posted a patch to the ML that makes this a selectable option from the command line instead:
https://fedorahosted.org/pipermail/matahari/2012-January/002284.html

Comment 3 Russell Bryant 2012-01-13 21:45:27 UTC
Ok, sounds good.  Feel free to close this out.  Thanks!