diff options
author | danielsdeleo <dan@chef.io> | 2016-11-03 20:52:38 -0700 |
---|---|---|
committer | danielsdeleo <dan@chef.io> | 2016-11-03 21:20:50 -0700 |
commit | 236d95796974420742ca9ffdc999130c3812c3ae (patch) | |
tree | d406031f132f51bef1fe3dc6e6077d3a0d2e7118 /acceptance | |
parent | a88e3068a9bd1755120663c8a4ff6ced70c7fd1d (diff) | |
download | chef-236d95796974420742ca9ffdc999130c3812c3ae.tar.gz |
Remove overly strict acceptance test
Implementations should be robust to extra fields for forwards
compatibility with future clients.
Signed-off-by: Daniel DeLeo <dan@chef.io>
Diffstat (limited to 'acceptance')
-rw-r--r-- | acceptance/data-collector/test/integration/default/serverspec/default_spec.rb | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/acceptance/data-collector/test/integration/default/serverspec/default_spec.rb b/acceptance/data-collector/test/integration/default/serverspec/default_spec.rb index f9d365ac58..25ac1c0a33 100644 --- a/acceptance/data-collector/test/integration/default/serverspec/default_spec.rb +++ b/acceptance/data-collector/test/integration/default/serverspec/default_spec.rb @@ -91,11 +91,6 @@ shared_examples_for "run_start payload check" do expect(missing_fields).to eq([]) end - it "does not have any extra fields" do - payload = JSON.load(command("curl http://localhost:9292/cache/run_start").stdout) - extra_fields = payload.keys.select { |key| !required_fields.include?(key) && !optional_fields.include?(key) } - expect(extra_fields).to eq([]) - end end end @@ -131,11 +126,6 @@ shared_examples_for "run_converge.success payload check" do expect(missing_fields).to eq([]) end - it "does not have any extra fields" do - payload = JSON.load(command("curl http://localhost:9292/cache/run_converge.success").stdout) - extra_fields = payload.keys.select { |key| !required_fields.include?(key) && !optional_fields.include?(key) } - expect(extra_fields).to eq([]) - end end end |