Fixes #23722 - removed redundant info about OS from hostgroup

theforeman · Jun 20, 2018 · 58ab9e7 · 58ab9e7
1 parent 30bdd1f
commit 58ab9e7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/hammer_cli_foreman/hostgroup.rb b/lib/hammer_cli_foreman/hostgroup.rb
@@ -71,7 +71,13 @@ class ListCommand < HammerCLIForeman::ListCommand
 
     class InfoCommand < HammerCLIForeman::InfoCommand
 
-      output ListCommand.output_definition do
+      output do
+        field :id, _("Id")
+        field :name, _("Name")
+        field :title, _("Title")
+        field nil, _("Puppet Environment"), Fields::SingleReference, :key => :environment
+        field nil, _("Model"), Fields::SingleReference, :key => :model
+
         field :description, _("Description"), Fields::LongText, :hide_blank => true
         field nil, _("Parent"), Fields::SingleReference, :key => :parent, :hide_blank => true
         field nil, _("Puppet CA Proxy"), Fields::SingleReference, :key => :puppet_ca_proxy

diff --git a/test/unit/hostgroup_test.rb b/test/unit/hostgroup_test.rb
@@ -41,7 +41,7 @@
     context "output" do
       with_params ["--id=1"] do
         it_should_print_n_records 1
-        it_should_print_columns ["Id", "Name", "Title", "Operating System"]
+        it_should_print_columns ["Id", "Name", "Title"]
         it_should_print_columns ["Puppet Environment", "Puppetclasses", "Parent"]
         it_should_print_columns ["Puppet CA Proxy", "Puppet Master Proxy"]
         it_should_print_columns ["Parameters", "Description"]