berkshelf 7.2.1 → 7.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4950e40e68a8fa20ad29aa5d589f27e4dd3fa0d7966abaedae2a9fc81398bc42
-  data.tar.gz: 706ca0ccbf4033186ac0adb6b1da988a55028fa1ccfae5c09a02c2ac414d31b4
+  metadata.gz: a3328d9ead777bfb150d7588648ca5376fa46fa5fafa8ded0afdb9cc710043b3
+  data.tar.gz: 94203ac8481a5a5155987e5f060d6fd1c78cc9f3073dd84efcb597d1a21ff5c6
 SHA512:
-  metadata.gz: a4207838b3b203d6446a6f4d529e2d74a67a4109dadb206011be3e9f34a7bd3ca5373df4e3ec5e07994da54571d6c9d91c7c245cd45b4afdbe5b6481c5db9aec
-  data.tar.gz: 4a57a2691d696385dbf46aa141b69b5dcd619a920f9abfb3a8b4ce0f5c3155591216b7c1267a804f3f80c8b805875253824b67686a1141df0107e772329fb3dc
+  metadata.gz: 337c5f8eb9d149f27e7fc40f6e1228f7fdbc2606a8c352dbe8c942ee5f6ab081e5db039fb9b46e41a9dbaff017ddf7d004d02c7d8d272d3042cf892e5e68396e
+  data.tar.gz: b9d8896f85f1c98924c3472ad97648071bc8301854a0ea49fe7dde2472faafe54f035b27140928462de7e099aa95983cda21d52d928ad93698ccf7e1404265e2

data/lib/berkshelf/api_client.rb

@@ -16,8 +16,8 @@ module Berkshelf
         Connection.new(*args)
       end
 
-      def chef_server(*args)
-        ChefServerConnection.new(*args)
+      def chef_server(**args)
+        ChefServerConnection.new(**args)
       end
     end
   end

data/lib/berkshelf/api_client/chef_server_connection.rb

@@ -5,15 +5,13 @@ module Berkshelf
     require_relative "errors"
 
     class ChefServerConnection
-      attr_reader :client
-
-      def initialize(*args)
-        @client = Berkshelf::RidleyCompat.new(*args)
-        @url = args[0][:server_url]
+      def initialize(**args)
+        @client = Berkshelf::RidleyCompat.new(**args)
+        @url = args[:server_url]
       end
 
       def universe
-        response = client.get("universe")
+        response = @client.get("universe")
 
         [].tap do |cookbooks|
           response.each do |name, versions|

data/lib/berkshelf/berksfile.rb

@@ -644,7 +644,7 @@ module Berkshelf
           FileUtils.mkdir_p(cookbook_destination)
 
           # Dir.glob does not support backslash as a File separator
-          src   = cookbook.path.to_s.tr('\\', "/")
+          src   = cookbook.path.to_s.tr("\\", "/")
           files = FileSyncer.glob(File.join(src, "**/*"))
 
           # strip directories

data/lib/berkshelf/installer.rb

@@ -19,13 +19,13 @@ module Berkshelf
     def build_universe
       berksfile.sources.collect do |source|
         Thread.new do
-          begin
-            Berkshelf.formatter.msg("Fetching cookbook index from #{source}...")
-            source.build_universe
-          rescue Berkshelf::APIClientError => ex
-            Berkshelf.formatter.warn "Error retrieving universe from source: #{source}"
-            Berkshelf.formatter.warn "  * [#{ex.class}] #{ex}"
-          end
+
+          Berkshelf.formatter.msg("Fetching cookbook index from #{source}...")
+          source.build_universe
+        rescue Berkshelf::APIClientError => ex
+          Berkshelf.formatter.warn "Error retrieving universe from source: #{source}"
+          Berkshelf.formatter.warn "  * [#{ex.class}] #{ex}"
+
         end
       end.map(&:join)
     end

data/lib/berkshelf/ssl_policies.rb

@@ -20,7 +20,7 @@ module Berkshelf
     end
 
     def trusted_certs_dir
-      config_dir = Berkshelf.config.chef.trusted_certs_dir.to_s.tr('\\', "/")
+      config_dir = Berkshelf.config.chef.trusted_certs_dir.to_s.tr("\\", "/")
       if config_dir.empty? || !::File.exist?(config_dir)
         File.join(ENV["HOME"], ".chef", "trusted_certs")
       else

data/lib/berkshelf/uploader.rb

@@ -59,41 +59,41 @@ module Berkshelf
         connection.get("users/#{Berkshelf.config.chef.node_name}") rescue nil
 
         cookbooks.map do |cookbook|
+
+          compiled_metadata = cookbook.compile_metadata
+          cookbook.reload if compiled_metadata
+          cookbook_version = cookbook.cookbook_version
+          Berkshelf.log.debug "  Uploading #{cookbook.cookbook_name}"
+          cookbook_version.freeze_version if options[:freeze]
+
+          # another two lines that are necessary for chef < 13.2 support (affects 11.x/12.x as well)
+          cookbook_version.metadata.maintainer "" if cookbook_version.metadata.maintainer.nil?
+          cookbook_version.metadata.maintainer_email "" if cookbook_version.metadata.maintainer_email.nil?
+
           begin
-            compiled_metadata = cookbook.compile_metadata
-            cookbook.reload if compiled_metadata
-            cookbook_version = cookbook.cookbook_version
-            Berkshelf.log.debug "  Uploading #{cookbook.cookbook_name}"
-            cookbook_version.freeze_version if options[:freeze]
-
-            # another two lines that are necessary for chef < 13.2 support (affects 11.x/12.x as well)
-            cookbook_version.metadata.maintainer "" if cookbook_version.metadata.maintainer.nil?
-            cookbook_version.metadata.maintainer_email "" if cookbook_version.metadata.maintainer_email.nil?
-
-            begin
-              Chef::CookbookUploader.new(
-                [ cookbook_version ],
-                force: options[:force],
-                concurrency: 1, # sadly
-                rest: connection,
-                skip_syntax_check: options[:skip_syntax_check]
-              ).upload_cookbooks
-              Berkshelf.formatter.uploaded(cookbook, connection)
-            rescue Chef::Exceptions::CookbookFrozen
-              if options[:halt_on_frozen]
-                raise FrozenCookbook.new(cookbook)
-              end
-
-              Berkshelf.formatter.skipping(cookbook, connection)
-            end
-          ensure
-            if compiled_metadata
-              # this is necessary on windows to clean up the ruby object that was pointing at the file
-              # so that we can reliably delete it.  windows is terrible.
-              GC.start
-              File.unlink(compiled_metadata)
+            Chef::CookbookUploader.new(
+              [ cookbook_version ],
+              force: options[:force],
+              concurrency: 1, # sadly
+              rest: connection,
+              skip_syntax_check: options[:skip_syntax_check]
+            ).upload_cookbooks
+            Berkshelf.formatter.uploaded(cookbook, connection)
+          rescue Chef::Exceptions::CookbookFrozen
+            if options[:halt_on_frozen]
+              raise FrozenCookbook.new(cookbook)
             end
+
+            Berkshelf.formatter.skipping(cookbook, connection)
+          end
+        ensure
+          if compiled_metadata
+            # this is necessary on windows to clean up the ruby object that was pointing at the file
+            # so that we can reliably delete it.  windows is terrible.
+            GC.start
+            File.unlink(compiled_metadata)
           end
+
         end
       end
     end

data/lib/berkshelf/version.rb

@@ -1,3 +1,3 @@
 module Berkshelf
-  VERSION = "7.2.1".freeze
+  VERSION = "7.2.2".freeze
 end

data/spec/unit/berkshelf/berkshelf/api_client/chef_server_connection_spec.rb

@@ -0,0 +1,65 @@
+require "spec_helper"
+
+describe Berkshelf::APIClient::ChefServerConnection do
+  let(:instance) do
+    described_class.new(
+      server_url: "https://chef.example.org/organizations/foobar",
+      timeout: 30,
+      open_timeout: 3,
+      ssl: {}
+    )
+  end
+
+  describe "#universe" do
+    before do
+      body_response = %q{{"ruby":{"1.2.3":{"endpoint_priority":0,"platforms":{},"dependencies":{"build-essential":">= 1.2.2"},"location_type":"supermarket","location_path":"https://supermarket.getchef.com/"},"2.0.0":{"endpoint_priority":0,"platforms":{},"dependencies":{"build-essential":">= 1.2.2"},"location_type":"supermarket","location_path":"https://supermarket.getchef.com/"}},"elixir":{"1.0.0":{"endpoint_priority":0,"platforms":{"CentOS":"= 6.0.0"},"dependencies":{},"location_type":"supermarket","location_path":"https://supermarket.getchef.com/"}}}}
+
+      stub_request(:get, "https://chef.example.org/organizations/foobar/universe")
+        .to_return(status: 200, body: body_response, headers: { "Content-Type" => "application/json; charset=utf-8" })
+    end
+
+    subject { instance.universe }
+
+    it "returns an array of APIClient::RemoteCookbook" do
+      expect(subject).to be_a(Array)
+
+      subject.each do |remote|
+        expect(remote).to be_a(Berkshelf::APIClient::RemoteCookbook)
+      end
+    end
+
+    it "contains a item for each dependency" do
+      expect(subject.size).to eq(3)
+      expect(subject[0].name).to eql("ruby")
+      expect(subject[0].version).to eql("1.2.3")
+      expect(subject[1].name).to eql("ruby")
+      expect(subject[1].version).to eql("2.0.0")
+      expect(subject[2].name).to eql("elixir")
+      expect(subject[2].version).to eql("1.0.0")
+    end
+
+    it "has the dependencies for each" do
+      expect(subject[0].dependencies).to include("build-essential" => ">= 1.2.2")
+      expect(subject[1].dependencies).to include("build-essential" => ">= 1.2.2")
+      expect(subject[2].dependencies).to be_empty
+    end
+
+    it "has the platforms for each" do
+      expect(subject[0].platforms).to be_empty
+      expect(subject[1].platforms).to be_empty
+      expect(subject[2].platforms).to include("CentOS" => "= 6.0.0")
+    end
+
+    it "has a location_path for each" do
+      subject.each do |remote|
+        expect(remote.location_path).to_not be_nil
+      end
+    end
+
+    it "has a location_type for each" do
+      subject.each do |remote|
+        expect(remote.location_type).to_not be_nil
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: berkshelf
 version: !ruby/object:Gem::Version
-  version: 7.2.1
+  version: 7.2.2
 platform: ruby
 authors:
 - Jamie Winsor
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-15 00:00:00.000000000 Z
+date: 2021-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout
@@ -315,6 +315,7 @@ files:
 - spec/support/path_helpers.rb
 - spec/support/shared_examples/formatter.rb
 - spec/unit/berkshelf/berksfile_spec.rb
+- spec/unit/berkshelf/berkshelf/api_client/chef_server_connection_spec.rb
 - spec/unit/berkshelf/berkshelf/api_client/connection_spec.rb
 - spec/unit/berkshelf/berkshelf/api_client/remote_cookbook_spec.rb
 - spec/unit/berkshelf/berkshelf/api_client_spec.rb