-
Notifications
You must be signed in to change notification settings - Fork 682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve inspec shell by having it evaluate describe/control blocks. #909
Conversation
Note - the "-c" arm of the shell already works (I have tested it locally). I'm trying to figure out how to make the shell work. |
056143f
to
30e1521
Compare
@@ -13,11 +13,11 @@ class Fetcher < Fetchers::Url | |||
priority 500 | |||
|
|||
def self.resolve(target, opts = {}) | |||
return nil unless URI(target).scheme == 'supermarket' | |||
return nil unless target.is_a?(String) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we still need to ensure that the scheme is supermarket
30e1521
to
d7e667d
Compare
d7e667d
to
3b02087
Compare
aa9f12f
to
e43cdf2
Compare
e43cdf2
to
0cc8fc5
Compare
Fix #240 |
bf7419f
to
82c3dda
Compare
else | ||
res = run_command(o) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should reflect the --format
option here and do the json dump if activated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
run_command should already print the json output. It's no longer using a mock runner - it's doing the real rspec thing which handles --format on its own.
444322c
to
2e09090
Compare
This fixes #240 |
2e09090
to
45dd726
Compare
45dd726
to
7843852
Compare
7843852
to
2d4c3e3
Compare
2d4c3e3
to
04c9b34
Compare
30ab290
to
b9a8e95
Compare
b9a8e95
to
fc66591
Compare
fc66591
to
33ae22d
Compare
@arlimus @chris-rock @jbussdieker @alexpop
This definitely has bugs in it but the is the general strategy I came up with last night after diving head-first into inspec and pry.