chef-zero 4.8.0 → 4.9.0

data/spec/search_spec.rb

@@ -1,11 +1,11 @@
-require 'chef_zero/solr/solr_parser'
-require 'chef_zero/solr/solr_doc'
+require "chef_zero/solr/solr_parser"
+require "chef_zero/solr/solr_doc"
 
 describe ChefZero::Solr::SolrParser do
   let (:all_docs) do
     docs = []
-    [{'foo' => 'a'},
-     {'foo' => 'd'}].each_with_index do |h, i|
+    [{ "foo" => "a" },
+     { "foo" => "d" }].each_with_index do |h, i|
       docs.push ChefZero::Solr::SolrDoc.new(h, i)
     end
     docs
@@ -13,24 +13,24 @@ describe ChefZero::Solr::SolrParser do
 
   def search_for(query)
     q = ChefZero::Solr::SolrParser.new(query).parse
-    all_docs.select {|doc| q.matches_doc?(doc) }
+    all_docs.select { |doc| q.matches_doc?(doc) }
   end
 
   it "handles terms" do
-    search_for('foo:d').size.should eq(1)
+    search_for("foo:d").size.should eq(1)
   end
 
   it "handles ranges" do
-    search_for('foo:[a TO c]').size.should eq(1)
+    search_for("foo:[a TO c]").size.should eq(1)
   end
 
   it "handles -" do
-    search_for('-foo:a').size.should eq(1)
+    search_for("-foo:a").size.should eq(1)
   end
 
   it "handles wildcard ranges" do
-    search_for('foo:[* TO c]').size.should eq(1)
-    search_for('foo:[c TO *]').size.should eq(1)
-    search_for('foo:[* TO *]').size.should eq(2)
+    search_for("foo:[* TO c]").size.should eq(1)
+    search_for("foo:[c TO *]").size.should eq(1)
+    search_for("foo:[* TO *]").size.should eq(2)
   end
 end

data/spec/server_spec.rb

@@ -1,9 +1,9 @@
-require 'chef_zero/server'
-require 'net/http'
-require 'uri'
+require "chef_zero/server"
+require "net/http"
+require "uri"
 
 describe ChefZero::Server do
-  context 'with a server bound to port 8889' do
+  context "with a server bound to port 8889" do
     before :each do
       @server = ChefZero::Server.new(:port => 8889)
       @server.start_background
@@ -12,84 +12,84 @@ describe ChefZero::Server do
       @server.stop
     end
 
-    it 'a second server bound to port 8889 throws EADDRINUSE' do
+    it "a second server bound to port 8889 throws EADDRINUSE" do
       expect { ChefZero::Server.new(:port => 8889).start }.to raise_error Errno::EADDRINUSE
     end
 
-    it 'a server bound to range 8889-9999 binds to a port > 8889' do
+    it "a server bound to range 8889-9999 binds to a port > 8889" do
       server = ChefZero::Server.new(:port => 8889.upto(9999))
       server.start_background
       expect(server.port).to be > 8889
       expect(URI(server.url).port).to be > 8889
     end
 
-    it 'a server bound to range 8889-8889 throws an exception' do
+    it "a server bound to range 8889-8889 throws an exception" do
       expect { ChefZero::Server.new(:port => 8889.upto(8889)).start_background }.to raise_error Errno::EADDRINUSE
     end
 
-    it 'has a very patient request timeout' do
+    it "has a very patient request timeout" do
       expect(@server.server.config[:RequestTimeout]).to eq 300
     end
 
-    context 'accept headers' do
+    context "accept headers" do
       def get_nodes(accepts)
         uri = URI(@server.url)
         httpcall = Net::HTTP.new(uri.host, uri.port)
-        httpcall.get('/nodes', 'Accept' => accepts)
+        httpcall.get("/nodes", "Accept" => accepts)
       end
 
       def get_version
         uri = URI(@server.url)
         httpcall = Net::HTTP.new(uri.host, uri.port)
-        httpcall.get('/version', 'Accept' => 'text/plain, application/json')
+        httpcall.get("/version", "Accept" => "text/plain, application/json")
       end
 
-      it 'accepts requests with no accept header' do
-        request = Net::HTTP::Get.new('/nodes')
-        request.delete('Accept')
+      it "accepts requests with no accept header" do
+        request = Net::HTTP::Get.new("/nodes")
+        request.delete("Accept")
         uri = URI(@server.url)
         response = Net::HTTP.new(uri.host, uri.port).request(request)
-        expect(response.code).to eq '200'
+        expect(response.code).to eq "200"
       end
 
-      it 'accepts requests with accept: application/json' do
-        expect(get_nodes('application/json').code).to eq '200'
+      it "accepts requests with accept: application/json" do
+        expect(get_nodes("application/json").code).to eq "200"
       end
 
-      it 'accepts requests with accept: application/*' do
-        expect(get_nodes('application/*').code).to eq '200'
+      it "accepts requests with accept: application/*" do
+        expect(get_nodes("application/*").code).to eq "200"
       end
 
-      it 'accepts requests with accept: application/*' do
-        expect(get_nodes('*/*').code).to eq '200'
+      it "accepts requests with accept: application/*" do
+        expect(get_nodes("*/*").code).to eq "200"
       end
 
-      it 'denies requests with accept: application/blah' do
-        expect(get_nodes('application/blah').code).to eq '406'
+      it "denies requests with accept: application/blah" do
+        expect(get_nodes("application/blah").code).to eq "406"
       end
 
-      it 'denies requests with accept: blah/json' do
-        expect(get_nodes('blah/json').code).to eq '406'
+      it "denies requests with accept: blah/json" do
+        expect(get_nodes("blah/json").code).to eq "406"
       end
 
-      it 'denies requests with accept: blah/*' do
-        expect(get_nodes('blah/*').code).to eq '406'
+      it "denies requests with accept: blah/*" do
+        expect(get_nodes("blah/*").code).to eq "406"
       end
 
-      it 'denies requests with accept: blah/*' do
-        expect(get_nodes('blah/*').code).to eq '406'
+      it "denies requests with accept: blah/*" do
+        expect(get_nodes("blah/*").code).to eq "406"
       end
 
-      it 'denies requests with accept: <empty string>' do
-        expect(get_nodes('').code).to eq '406'
+      it "denies requests with accept: <empty string>" do
+        expect(get_nodes("").code).to eq "406"
       end
 
-      it 'accepts requests with accept: a/b;a=b;c=d, application/json;a=b, application/xml;a=b' do
-        expect(get_nodes('a/b;a=b;c=d, application/json;a=b, application/xml;a=b').code).to eq '200'
+      it "accepts requests with accept: a/b;a=b;c=d, application/json;a=b, application/xml;a=b" do
+        expect(get_nodes("a/b;a=b;c=d, application/json;a=b, application/xml;a=b").code).to eq "200"
       end
 
-      it 'accepts /version' do
-        expect(get_version.body.start_with?('chef-zero')).to be true
+      it "accepts /version" do
+        expect(get_version.body.start_with?("chef-zero")).to be true
       end
     end
   end

data/spec/socketless_server_map_spec.rb

@@ -1,5 +1,4 @@
-require 'chef_zero/socketless_server_map'
-
+require "chef_zero/socketless_server_map"
 
 describe "Socketless Mode" do
 
@@ -72,5 +71,4 @@ describe "Socketless Mode" do
     end
   end
 
-
 end

data/spec/support/oc_pedant.rb

@@ -21,7 +21,7 @@
 ################################################################################
 # You MUST specify the address of the server the API requests will be
 # sent to.  Only specify protocol, hostname, and port.
-chef_server 'http://127.0.0.1:8889'
+chef_server "http://127.0.0.1:8889"
 
 # If you are doing development testing, you can specify the address of
 # the Solr server.  The presence of this parameter will enable tests
@@ -54,7 +54,6 @@ search_server   chef_server
 search_commit_url "/dummy"
 search_url_fmt    "/dummy?fq=+X_CHEF_type_CHEF_X:%{type}&q=%{query}&wt=json"
 
-
 # We're starting to break tests up into groups based on different
 # criteria.  The proper API tests (the results of which are viewable
 # to OPC customers) should be the only ones run by Pedant embedded in
@@ -69,10 +68,10 @@ search_url_fmt    "/dummy?fq=+X_CHEF_type_CHEF_X:%{type}&q=%{query}&wt=json"
 # value.
 include_internal false
 
-key = 'spec/support/stickywicket.pem'
+key = "spec/support/stickywicket.pem"
 
 org(name: "pedant-testorg",
-    create_me: !ENV['CHEF_FS'],
+    create_me: !ENV["CHEF_FS"],
     validator_key: key)
 
 internal_account_url chef_server
@@ -85,15 +84,15 @@ delete_org true
 # are using pre-existing users, you must supply a ':key_file' key,
 # which should be the fully-qualified path /on the machine Pedant is
 # running on/ to a private key for that user.
-superuser_name 'pivotal'
+superuser_name "pivotal"
 superuser_key key
 webui_key key
 
 def cheffs_or_else_user(value)
-  ENV['CHEF_FS'] ? "pivotal" : value
+  ENV["CHEF_FS"] ? "pivotal" : value
 end
 
-keyfile_maybe = ENV['CHEF_FS'] ? { key_file: key } : { key_file: nil }
+keyfile_maybe = ENV["CHEF_FS"] ? { key_file: key } : { key_file: nil }
 
 requestors({
              :clients => {
@@ -102,45 +101,45 @@ requestors({
                  :name => "pedant_admin_client",
                  :create_me => true,
                  :create_knife => true,
-                 :admin => true
+                 :admin => true,
                },
                :non_admin => {
-                 :name => 'pedant_client',
+                 :name => "pedant_client",
                  :create_me => true,
                  :create_knife => true,
                },
                :bad => {
-                 :name => 'bad_client',
+                 :name => "bad_client",
                  :create_me => true,
                  :create_knife => true,
-                 :bogus => true
-               }
+                 :bogus => true,
+               },
              },
 
              :users => {
                # An administrator in the testing organization
                :admin => {
                  :name => cheffs_or_else_user("pedant_admin_user"),
-                 :create_me => !ENV['CHEF_FS'],
-                 :associate => !ENV['CHEF_FS'],
-                 :create_knife => true
+                 :create_me => !ENV["CHEF_FS"],
+                 :associate => !ENV["CHEF_FS"],
+                 :create_knife => true,
                }.merge(keyfile_maybe),
 
                :non_admin => {
                  :name => cheffs_or_else_user("pedant_user"),
-                 :create_me => !ENV['CHEF_FS'],
-                 :associate => !ENV['CHEF_FS'],
-                 :create_knife => true
+                 :create_me => !ENV["CHEF_FS"],
+                 :associate => !ENV["CHEF_FS"],
+                 :create_knife => true,
                }.merge(keyfile_maybe),
 
                # A user that is not a member of the testing organization
                :bad => {
                  :name => cheffs_or_else_user("pedant-nobody"),
-                 :create_me => !ENV['CHEF_FS'],
+                 :create_me => !ENV["CHEF_FS"],
                  :create_knife => true,
-                 :associate => false
+                 :associate => false,
                }.merge(keyfile_maybe),
-             }
+             },
            })
 
 self[:tags] = [:validation, :authentication, :authorization]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chef-zero
 version: !ruby/object:Gem::Version
-  version: 4.8.0
+  version: 4.9.0
 platform: ruby
 authors:
 - John Keiser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-25 00:00:00.000000000 Z
+date: 2016-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-log