knife 18.3.0 → 18.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eee4d86bce76117b7bc25593ac766eea25bb2f3a6d46c861b578f04595c23e71
-  data.tar.gz: 92f611bca1afb09fffe920d91120c00ccb7e6bc5bca12af188ceecc1378bb216
+  metadata.gz: fdf20084b258549d98683a30f13e82c8c05ef96454c04108db37f1a2a5a71182
+  data.tar.gz: 0cae6131e1d8c5b05547b1f221942e84f501b0ce9685f162e308af8b868c16df
 SHA512:
-  metadata.gz: 597d3b7e1ff3d0743318324185b920318c36c56d247d0a32a212b23f565c0bfc26812932db787129e401614d2e518e5a5e7fb19c1aa3f5138cc0f05890ccab55
-  data.tar.gz: c918ef6cdb6999f1b2eb943fb1b0003a1b5c71acf2b2ef5206b40035fdd06558678a77b84f82815415e4c3695ceae4861ad1646663b853fbc6c6c2bff72b4354
+  metadata.gz: 5c1ff58d3324c77d44cd1168bad50e7fa94abf3be2eda9bdd8a39708b41e89f054a3b1113889c38f118adf8d85c75289f9443d3dba0675e2121ff530a645ba84
+  data.tar.gz: e3d0a3749aa738b7e1cb69793af8bff04a3d209442f58d0f4bdbacbcd4484b1222ce240eae440f45dbd537c73768d86bdcc0d22e4a4e0c36824382ee0b00000b

data/lib/chef/knife/core/gem_glob_loader.rb

@@ -78,7 +78,7 @@ class Chef
           if check_load_path
             files = $LOAD_PATH.map do |load_path|
               Dir["#{File.expand_path glob, ChefConfig::PathHelper.escape_glob_dir(load_path)}#{Gem.suffix_pattern}"]
-            end.flatten.select { |file| File.file? file.untaint }
+            end.flatten.select { |file| File.file? file }
 
           end
 
@@ -114,7 +114,7 @@ class Chef
 
           glob = File.join(ChefConfig::PathHelper.escape_glob_dir(spec.full_gem_path, dirs), glob)
 
-          Dir[glob].map(&:untaint)
+          Dir[glob]
         end
 
         def from_different_chef_version?(path)

data/lib/chef/knife/core/ui.rb

@@ -41,6 +41,7 @@ class Chef
       def_delegator :@presenter, :format_list_for_display
       def_delegator :@presenter, :format_for_display
       def_delegator :@presenter, :format_cookbook_list_for_display
+      def_delegator :@presenter, :attribute_field_separator
 
       def initialize(stdout, stderr, stdin, config)
         @stdout, @stderr, @stdin, @config = stdout, stderr, stdin, config

data/lib/chef/knife/search.rb

@@ -90,7 +90,7 @@ class Chef
         if config[:filter_result]
           search_args[:filter_result] = create_result_filter(config[:filter_result])
         elsif (not ui.config[:attribute].nil?) && (not ui.config[:attribute].empty?)
-          search_args[:filter_result] = create_result_filter_from_attributes(ui.config[:attribute])
+          search_args[:filter_result] = create_result_filter_from_attributes(ui.config[:attribute], ui.attribute_field_separator)
         elsif config[:id_only]
           search_args[:filter_result] = create_result_filter_from_attributes([])
         end
@@ -179,10 +179,10 @@ class Chef
         final_filter
       end
 
-      def create_result_filter_from_attributes(filter_array)
+      def create_result_filter_from_attributes(filter_array, separator = ".")
         final_filter = {}
         filter_array.each do |f|
-          final_filter[f] = f.split(".")
+          final_filter[f] = f.split(separator)
         end
         # adding magic filter so we can actually pull the name as before
         final_filter["__display_name"] = [ "name" ]

data/lib/chef/knife/version.rb

@@ -17,7 +17,7 @@
 class Chef
   class Knife
     KNIFE_ROOT = File.expand_path("../..", __dir__)
-    VERSION = "18.3.0".freeze
+    VERSION = "18.4.2".freeze
   end
 end
 

data/spec/unit/knife/search_spec.rb

@@ -0,0 +1,148 @@
+#
+# Author:: Ashique Saidalavi (<ashique.saidalavi@progress.com>)
+# Copyright:: Copyright (c) Chef Software Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require "knife_spec_helper"
+
+describe Chef::Knife::Search do
+  describe "node" do
+    let(:knife) { Chef::Knife::Search.new }
+    let(:query) { double("Chef::Search::Query") }
+    let(:stdout) { StringIO.new }
+    let(:stderr) { StringIO.new }
+
+    before(:each) do
+      node = Chef::Node.new.tap do |n|
+        n.automatic_attrs["fqdn"] = "foobar"
+        n.automatic_attrs["ohai_time"] = 1343845969
+        n.automatic_attrs["platform"] = "mac_os_x"
+        n.automatic_attrs["platform_version"] = "10.12.5"
+      end
+
+      allow(query).to receive(:search).and_yield(node)
+      allow(Chef::Search::Query).to receive(:new).and_return(query)
+
+      allow(knife).to receive(:output).and_return(true)
+      allow(knife.ui).to receive(:stdout).and_return(stdout)
+      allow(knife.ui).to receive(:stderr).and_return(stderr)
+    end
+
+    describe "run" do
+      it "should be successful" do
+        knife.name_args = ["node", ":"]
+        expect(query).to receive(:search).with("node", ":", { fuzz: true })
+        knife.run
+      end
+
+      context "read_cli_args" do
+        it "should be invoked" do
+          expect(knife).to receive(:read_cli_args)
+          knife.run
+        end
+
+        it "should raise error if query passed with argument as well as -q option" do
+          knife.name_args = ["node", ":"]
+          knife.config[:query] = ":"
+
+          expect { knife.run }.to raise_error(SystemExit)
+          expect(stderr.string).to match /Please specify query as an argument or an option via -q, not both/im
+        end
+
+        it "should fail if no query passed" do
+          knife.name_args = []
+
+          expect { knife.run }.to raise_error(SystemExit)
+          expect(stderr.string).to match /No query specified/im
+        end
+      end
+
+      context "filters" do
+        before do
+          knife.name_args = ["node", "packages:git"]
+          knife.config[:filter_result] = "env=chef_environment"
+        end
+
+        it "should invoke create_result_filter" do
+          expect(knife).to receive(:create_result_filter).with("env=chef_environment")
+          knife.run
+        end
+
+        it "should invoke search object with correct filter" do
+          expect(query).to receive(:search).with("node", "packages:git", { filter_result: { env: ["chef_environment"] }, fuzz: true })
+          knife.run
+        end
+      end
+
+      context "attributes" do
+        before do
+          knife.name_args = ["node", "packages:git"]
+          knife.ui.config[:attribute] = ["packages.git.version"]
+        end
+
+        it "should invoke create_result_filter_from_attributes method" do
+          expect(knife).to receive(:create_result_filter_from_attributes).with(["packages.git.version"], knife.ui.attribute_field_separator)
+          knife.run
+        end
+
+        it "should invoke search query with correct filter" do
+          filter_obj = {
+            filter_result: {
+              "__display_name" => ["name"],
+              "packages.git.version" => %w{packages git version} },
+            fuzz: true,
+          }
+          expect(query).to receive(:search).with("node", "packages:git", filter_obj)
+          knife.run
+        end
+
+        context "field_separator" do
+          it "should have dot as the default field_separator" do
+            expect(knife.ui.attribute_field_separator).to eq(".")
+          end
+
+          it "has the correct field_separator" do
+            knife.config[:field_separator] = ":"
+
+            expect(knife.ui.attribute_field_separator).to eq(":")
+          end
+
+          context "parsing" do
+            before do
+              knife.name_args = %w{node packages:git}
+            end
+
+            it "parses the attributes correctly for dot field_separator" do
+              knife.config[:field_separator] = "."
+              knife.ui.config[:attribute] = ["packages.git.version"]
+
+              filter_output = { "__display_name" => ["name"], "packages.git.version" => %w{packages git version} }
+              expect(knife.create_result_filter_from_attributes(["packages.git.version"], ".")).to eq(filter_output)
+            end
+
+            it "parses it for different field_separator" do
+              knife.config[:field_separator] = ":"
+              knife.ui.config[:attribute] = ["packages:git:version"]
+
+              filter_output = { "__display_name" => ["name"], "packages:git:version" => %w{packages git version} }
+              expect(knife.create_result_filter_from_attributes(["packages:git:version"], ":")).to eq(filter_output)
+            end
+          end
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knife
 version: !ruby/object:Gem::Version
-  version: 18.3.0
+  version: 18.4.2
 platform: ruby
 authors:
 - Adam Jacob
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-18 00:00:00.000000000 Z
+date: 2024-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef-config
@@ -1098,6 +1098,7 @@ files:
 - spec/unit/knife/role_run_list_replace_spec.rb
 - spec/unit/knife/role_run_list_set_spec.rb
 - spec/unit/knife/role_show_spec.rb
+- spec/unit/knife/search_spec.rb
 - spec/unit/knife/ssh_spec.rb
 - spec/unit/knife/ssl_check_spec.rb
 - spec/unit/knife/ssl_fetch_spec.rb
@@ -1144,7 +1145,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.2.33
 signing_key:
 specification_version: 4
 summary: The knife CLI for Chef Infra.