-
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
Feature/fix ability to pass in supermarket url #1595
Feature/fix ability to pass in supermarket url #1595
Conversation
It looks like this was originally supposed to work, but at some point the default value was put in the method body rather than in the method parameters. This change allows you to configure the supermarket_url in test kitchen like so: ``` verifier: inspec_tests: - name: linux-hardening supermarket: som3guy/apache-disa-stig supermarket_url: https://my.supermarket.com ``` Signed-off-by: Ryan Larson <ryan.mango.larson@gmail.com>
Signed-off-by: Ryan Larson <ryan.mango.larson@gmail.com>
Signed-off-by: Ryan Larson <ryan.mango.larson@gmail.com>
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.
Thank you @rylarson for finding and fixing that annoying bug
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.
@rylarson thanks for the fix! Can you please add addressable
to the gemspec for this change? Right now, we're bringing in addressable
via other dependencies. Since we're now depending on it directly, we should declare that for future-proofing purposes.
Signed-off-by: Ryan Larson <ryan.mango.larson@gmail.com>
18cd76e
to
170808b
Compare
@adamleff The gemspec has been updated with addressable. Sorry for missing that. |
No apologies necessary, @rylarson! I'll review this shortly. |
@rylarson this looks fantastic, and major props for adding tests for the We greatly appreciate your contribution! |
Enable customization of supermarket_url
It looks like this was originally supposed to work, but at some point
the default value was put in the method body rather than in the method
parameters.
This change allows you to configure the supermarket_url in test kitchen
like so:
I added tests for all of the methods on
Supermarket::API
to make sure we don't have any regressions going forward.