Fixes #27508 - use LCE for package list (#689)

* Fixes #27508 - use LCE for package list

'hammer package list --environment=Library --organization-id=1'

This commit ensures that the --environment flag correctly points to
the lifecyle environment and not puppet env.

* Refs #27508 - Adds unit test

(cherry picked from commit 0b0b878)

lib/hammer_cli_katello/package.rb

@@ -3,6 +3,9 @@ class PackageCommand < HammerCLIKatello::Command
     resource :packages
 
     class ListCommand < HammerCLIKatello::ListCommand
+      extend_with(HammerCLIKatello::CommandExtensions::LifecycleEnvironment.new)
+      include LifecycleEnvironmentNameMapping
+
       output do
         field :id, _("ID")
         field :filename, _("Filename")
@@ -34,7 +37,6 @@ class ListCommand < HammerCLIKatello::ListCommand
               :option_environment_name).required
         end
       end
-
       build_options do |o|
         o.expand.including(:products, :content_views)
       end

test/functional/package/list_test.rb

@@ -2,11 +2,14 @@
 require_relative '../repository/repository_helpers'
 require_relative '../product/product_helpers'
 require_relative '../organization/organization_helpers'
+require_relative '../lifecycle_environment/lifecycle_environment_helpers'
 require 'hammer_cli_katello/package'
 
+# rubocop:disable ModuleLength
 module HammerCLIKatello
   describe PackageCommand::ListCommand do
     include OrganizationHelpers
+    include LifecycleEnvironmentHelpers
     include RepositoryHelpers
     include ProductHelpers
 
@@ -66,6 +69,28 @@ module HammerCLIKatello
       end
     end
 
+    describe 'environment options' do
+      it 'may be specified environment name' do
+        org_id = 2
+        env_name = "dev"
+        expected_id = 6
+        expect_lifecycle_environment_search(org_id, env_name, expected_id)
+
+        api_expects(:packages, :index).with_params("organization_id" => org_id,
+                                                   "environment_id" => expected_id,
+                                                   "page" => 1, "per_page" => 1000)
+
+        run_cmd(%W(package list --environment=#{env_name} --organization-id=#{org_id}))
+      end
+
+      it 'may be specified environment name no org fails' do
+        api_expects_no_call
+        r = run_cmd(%w(package list --environment Library))
+        expected_error = "--organization, --organization-label, --organization-id"
+        assert(r.err.include?(expected_error), "Invalid error message")
+      end
+    end
+
     describe 'product options' do
       it 'may be specified by ID' do
         api_expects(:repositories, :index)