summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authordanielsdeleo <dan@chef.io>2016-11-03 20:52:38 -0700
committerdanielsdeleo <dan@chef.io>2016-11-03 21:20:50 -0700
commit236d95796974420742ca9ffdc999130c3812c3ae (patch)
treed406031f132f51bef1fe3dc6e6077d3a0d2e7118
parenta88e3068a9bd1755120663c8a4ff6ced70c7fd1d (diff)
downloadchef-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>
-rw-r--r--acceptance/data-collector/test/integration/default/serverspec/default_spec.rb10
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