hammer_cli 3.14.0 → 3.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 682498506056a4d18876c29924250c7050dbc143878fcf03d8bec72140dcfdb4
-  data.tar.gz: 4c135d71447e67d75bf6b763d90fcc5b30e41f360683f4205066862854cc3fe3
+  metadata.gz: e59588fd8f86b76aee78fc150ea938650b2f5df37c6dba8e53be5d44a1117f8f
+  data.tar.gz: f5d84d4062dc8c9a8db3856b1e6569980c86597ae4b7d96a439c0128b832365a
 SHA512:
-  metadata.gz: 1451af2336736479bd7a630b1717d42a3563ae8e24261a778e705417e060d18c286c2524e696f723b46059e5e4077ac5b5fb5e16f445f6297a2b87db0dcdb0b1
-  data.tar.gz: '0937139414c830add628ce0042ec4575a4972ab9e0d860babac4ef3d3c2976e53f5819464bc8e1e8e781a5cf8c5c93d45d01fde3f810645f5f135c60e9f90a76'
+  metadata.gz: 332b4e7295ee014cbf01f7abf00f41a11a67291955ac118814a832c1cd169a9e53b60cb618385a50399488311837c8c3bce78234986d3a3487107e6d9f14af0b
+  data.tar.gz: 7aceb50f62e7ede2ab787e89af97cf49375175c054fa301a74602506529ea7d9f4f311df4a60513fd47894c3b9e03768de04e0ba49220020cf59c9bb20c0900f

data/doc/creating_commands.md

@@ -755,6 +755,35 @@ def adapter
 end
 ```
 
+#### Deprecating fields
+To deprecate a field, add `:deprecated => true` as an option for the field. This will print a warning message to stderr whenever the field is displayed. Consider removing this field from the default set so it is not displayed without a `--fields` param:
+
+```
+field :dep_fld, _("Deprecated field"), Fields::Field, :sets => ['ALL'], :deprecated => true
+```
+
+Example output:
+
+```
+$ hammer foo info --fields "Deprecated field"
+Warning: Field 'Deprecated field' is deprecated and may be removed in future versions.
+Deprecated field: bar
+```
+
+Additionally, a field may be 'replaced by' another field using `:replaced_by => "Path/To/New/Field"`. This will mark the field as deprecated and print a similar warning message to stderr whenever the field is displayed:
+
+```
+field :rep_fld, _("Replaced field"), Fields::Field, :sets => ['ALL'], :replaced_by => "Path/New field"
+```
+
+Example output:
+
+```
+$ hammer foo info --fields "Replaced field"
+Warning: Field 'Replaced field' is deprecated. Consider using 'Path/New field' instead.
+Replaced field: bar
+```
+
 #### Verbosity
 Currently Hammer [defines](https://github.com/theforeman/hammer-cli/blob/master/lib/hammer_cli/verbosity.rb) three basic verbose modes:
   * __QUIET__ - Prints nothing

data/doc/release_notes.md

@@ -1,5 +1,14 @@
 Release notes
 =============
+### 3.15.0 (2025-05-14)
+* Translate warning message ([PR #396](https://github.com/theforeman/hammer-cli/pull/396)), [#38410](http://projects.theforeman.org/issues/38410)
+* Add mechanism for deprecating displayed fields ([PR #395](https://github.com/theforeman/hammer-cli/pull/395)), [#38410](http://projects.theforeman.org/issues/38410)
+* Fix test using #inspect on a hash
+* Add base64 and csv dependencies for ruby 3.4 compatibility
+* Fix gemspec/ordereddependencies cop
+* Use actions/checkout@v4
+* Bump to 3.15.0-develop
+
 ### 3.14.0 (2025-02-20)
 * Check better for parent option desc injection, [#38126](http://projects.theforeman.org/issues/38126)
 * Bump apipie-bindings to 0.7.0

data/lib/hammer_cli/output/adapter/base.rb

@@ -42,6 +42,12 @@ module HammerCLI::Output::Adapter
     end
 
     def render_field(field, data, label_width)
+      if field.replaced_by
+        warn _("Warning: Field '%{field}' is deprecated. Consider using '%{replacement}' instead.") % { field: field.full_label, replacement: field.replaced_by }
+      elsif field.deprecated
+        warn _("Warning: Field '%{field}' is deprecated and may be removed in future versions.") % { field: field.full_label }
+      end
+
       if field.is_a? Fields::ContainerField
         output = ""
 

data/lib/hammer_cli/output/fields.rb

@@ -4,7 +4,7 @@ module Fields
   class Field
     attr_reader :path
     attr_writer :sets
-    attr_accessor :label, :parent
+    attr_accessor :label, :parent, :replaced_by, :deprecated
 
     def initialize(options={})
       @hide_blank = options[:hide_blank].nil? ? false : options[:hide_blank]
@@ -12,6 +12,8 @@ module Fields
       @path = options[:path] || []
       @label = options[:label]
       @sets = options[:sets]
+      @replaced_by = options[:replaced_by]
+      @deprecated = (options[:deprecated].nil?) ? !@replaced_by.nil? : options[:deprecated]
       @options = options
     end
 

data/lib/hammer_cli/version.rb

@@ -1,5 +1,5 @@
 module HammerCLI
   def self.version
-    @version ||= Gem::Version.new "3.14.0"
+    @version ||= Gem::Version.new "3.15.0"
   end
 end

data/test/unit/abstract_test.rb

@@ -185,7 +185,7 @@ describe HammerCLI::AbstractCommand do
       test_command_class.option(['--password'], 'PASSWORD', 'Password')
       test_command = test_command_class.new("")
       test_command.run ['--password=pass']
-      _(@log_output.readline.strip).must_equal "INFO  HammerCLI::AbstractCommand : Called with options: {\"option_password\"=>\"***\"}"
+      _(@log_output.readline.strip).must_match(%r{INFO  HammerCLI::AbstractCommand : Called with options: {\"option_password\"\s*=>\s*\"\*\*\*\"}})
     end
   end
 

data/test/unit/output/adapter/base_test.rb

@@ -28,6 +28,9 @@ describe HammerCLI::Output::Adapter::Base do
     let(:blank)             { Fields::Field.new(:path => [:blank], :label => "Blank", :hide_blank => true) }
     let(:login)             { Fields::Field.new(:path => [:login], :label => "Login") }
     let(:missing)           { Fields::Field.new(:path => [:login], :label => "Missing", :hide_missing => false) }
+    let (:deprecated_a) { Fields::Field.new(:path => [:deprecated_a], :label => "Deprecated", :deprecated => true) }
+    let (:new_field) { Fields::Field.new(:path => [:new_field], :label => "New field") }
+    let (:deprecated_b) { Fields::Field.new(:path => [:deprecated_b], :label => "Replaced by", :replaced_by => 'New field') }
 
     let(:data) { HammerCLI::Output::RecordCollection.new [{
       :id => 112,
@@ -55,7 +58,10 @@ describe HammerCLI::Output::Adapter::Base do
           :name => 'size',
           :value => '32'
         }
-      ]
+      ],
+      deprecated_a: 'deprecated_a',
+      deprecated_b: 'deprecated_b',
+      new_field: 'new_field'
     }]}
 
     it "should print one field" do
@@ -199,6 +205,31 @@ describe HammerCLI::Output::Adapter::Base do
       _{ adapter.print_collection(fields, data) }.must_output(expected_output)
     end
 
+    it "should warn about deprecated fields" do
+      fields = [deprecated_a]
+
+      expected_stdout= [
+        "Deprecated: deprecated_a",
+        "\n"
+      ].join("\n")
+      expected_stderr = "Warning: Field 'Deprecated' is deprecated and may be removed in future versions.\n"
+
+      _{ adapter.print_collection(fields, data) }.must_output(stdout=expected_stdout, stderr=expected_stderr)
+    end
+
+    it "should warn about replaced fields" do
+      fields = [new_field, deprecated_b]
+
+      expected_stdout= [
+        "New field:   new_field",
+        "Replaced by: deprecated_b",
+        "\n"
+      ].join("\n")
+      expected_stderr = "Warning: Field 'Replaced by' is deprecated. Consider using 'New field' instead.\n"
+
+      _{ adapter.print_collection(fields, data) }.must_output(stdout=expected_stdout, stderr=expected_stderr)
+    end
+
     context 'printing by chunks' do
       let(:context) { { show_ids: true } }
       let(:collection_count) { 30 }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli
 version: !ruby/object:Gem::Version
-  version: 3.14.0
+  version: 3.15.0
 platform: ruby
 authors:
 - Martin Bačovský
@@ -9,30 +9,38 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-20 00:00:00.000000000 Z
+date: 2025-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: clamp
+  name: amazing_print
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.1
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.1
-    - - "<"
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: apipie-bindings
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 0.7.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
-  name: logging
+  name: base64
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -46,21 +54,27 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: unicode-display_width
+  name: clamp
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.3.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.3.1
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
-  name: amazing_print
+  name: csv
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -74,7 +88,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: highline
+  name: fast_gettext
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -88,7 +102,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: fast_gettext
+  name: highline
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -116,19 +130,33 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 2.0.6
 - !ruby/object:Gem::Dependency
-  name: apipie-bindings
+  name: logging
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: unicode-display_width
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 'Hammer cli provides universal extendable CLI interface for ruby apps
 
   '