Adds Shell / Moves fixtures#596
Merged
Merged
Conversation
This commit adds a shell that can be started by simply calling 'slcli' with no command. The shell can be exited using ctrl + d. The shell has autocomplete. The shell probably needs a way to set global options (--fixtures, --debug, --proxy, --config, --timings, etc). In the future, this mode can be used to prompt for user input. Fixtures were moved to a higher level to avoid needing test dependencies to use fixtures.
|
|
Contributor
Author
|
Example Usage: slcli shell
(0)> vs list
....
(0)>Currently, the global options passed in when using slcli --fixtures shell
(0)> vs list
:.....:..........:..............:.............:............:........:
: id : hostname : primary_ip : backend_ip : datacenter : action :
:.....:..........:..............:.............:............:........:
: 100 : vs-test1 : 172.16.240.2 : 10.45.19.37 : TEST00 : - :
: 104 : vs-test2 : 172.16.240.7 : 10.45.19.35 : TEST00 : - :
:.....:..........:..............:.............:............:........:
(0)> --no-fixtures vs list
... (non-fixture data)...The only place where this doesn't work as expected is the verbosity/debug level. |
|
|
|
|
Member
There was a problem hiding this comment.
This should be
os.makedirs(app_path)On mac I get the following.
os.path.dirname: /Users/christopher/Library/Application Support
app_path: /Users/christopher/Library/Application Support/softlayer
~/Code/softlayer-python $ ./slcli shell
Traceback (most recent call last):
File "./slcli", line 7, in <module>
load_entry_point('SoftLayer', 'console_scripts', 'slcli')()
File "/Users/christopher/Code/softlayer-python/SoftLayer/CLI/core.py", line 178, in main
cli.main(**kwargs)
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/core.py", line 590, in main
rv = self.invoke(ctx)
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/core.py", line 936, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/core.py", line 782, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/core.py", line 416, in invoke
return callback(*args, **kwargs)
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/decorators.py", line 64, in new_func
return ctx.invoke(f, obj, *args[1:], **kwargs)
File "/Library/Python/2.7/site-packages/click-3.3-py2.7.egg/click/core.py", line 416, in invoke
return callback(*args, **kwargs)
File "/Users/christopher/Code/softlayer-python/SoftLayer/shell/core.py", line 54, in cli
os.makedirs(os.path.dirname(app_path))
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.py", line 157, in makedirs
mkdir(name, mode)
OSError: [Errno 17] File exists: '/Users/christopher/Library/Application Support'
Contributor
Author
There was a problem hiding this comment.
Should be fixed now, thanks!
Member
|
Spot checked my favorite slcli calls and they all work great. |
underscorephil
added a commit
that referenced
this pull request
Aug 11, 2015
Adds Shell / Moves fixtures
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit adds a shell that can be started by simply calling 'slcli' with no command. The shell can be exited using ctrl + d. The shell has autocomplete. The shell probably needs a way to set global options (--fixtures, --debug, --proxy, --config, --timings, etc). In the future, this mode can be used to prompt for user input.
Fixtures were moved to a higher level to avoid needing test dependencies (nosetests, testtools, etc) to use fixtures.
This is still a bit of a WiP.