mirror of
https://github.com/inspec/inspec
synced 2024-11-14 00:47:10 +00:00
a1129f9efc
In https://github.com/inspec/inspec/issues/4936 the issue was reported that naming an input the same as a control caused an unexpected failure. In that particular case, the naming was a result of a pre-waivers workaround which is no longer necessary, but ultimately a breakage of that name clash is an unexpected occurrance. Due to how inputs are named and registered, `__apply_waivers` thinks that an object is a waiver that is not a waiver and tries to process it. On the micro level, it breaks when trying to pass a variable to a string as if it were a Hash. It is imperative that we preserve 100% of the current featureset, pass our tests, and fix this edge case along with new test coverage for the failure. This PR updates the code to do a slightly more elegant and small ‘waiver check’ to stop the namespace clash from breaking our code. Signed-off-by: Nick Schwaderer <nschwaderer@chef.io> |
||
|---|---|---|
| .. | ||
| fetchers_test.rb | ||
| filter_table_test.rb | ||
| git_fetcher_test.rb | ||
| helper.rb | ||
| inheritance_test.rb | ||
| inputs_test.rb | ||
| inspec_archive_test.rb | ||
| inspec_check_test.rb | ||
| inspec_detect_test.rb | ||
| inspec_exec_automate_test.rb | ||
| inspec_exec_json_test.rb | ||
| inspec_exec_jsonmin_test.rb | ||
| inspec_exec_junit_test.rb | ||
| inspec_exec_test.rb | ||
| inspec_json_profile_test.rb | ||
| inspec_report_test.rb | ||
| inspec_schema_test.rb | ||
| inspec_shell_test.rb | ||
| inspec_supermarket_test.rb | ||
| inspec_test.rb | ||
| inspec_vendor_test.rb | ||
| license_test.rb | ||
| logging_test.rb | ||
| plugins_test.rb | ||
| ui_test.rb | ||
| waivers_test.rb | ||