katello 4.4.0.2 → 4.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9e2cbec78c4d297f09f1517b095e566c055bd14e2b3d4a534a5f904f8e9d43bc
-  data.tar.gz: 5f96a7ce828a6ac081883643f8f79d55088ded4a5a06d86c4bfc69fababc315e
+  metadata.gz: 5ba77185c8a7b69f5804f90eca22f14863b4eaf37311fbbfcdd30e95bff824cf
+  data.tar.gz: 024c9250e18577f8897698c4af3d21e432c3986a6b84599876412a715fe2bbc5
 SHA512:
-  metadata.gz: c109e2ed4481c10d8bc475b706aa00498f3cfce65fb2f9ec6cf589be8da2a47344561b31d45a5721aaab9743ccdbf751ec665663b858c26eb433803db59bbc6c
-  data.tar.gz: 87d9b9ced044f3b34df8045a1ca1a02ca5ba695e4048999290fc0594f21186065c63036b93d4b1beef09162ddf9bb60c06dbd7cde7057df8a456a5701b8e9e3d
+  metadata.gz: 00ba2a8b933f5069920529d0780c48c290a157ff62123a0f530332497c5c34e5901d20eb7b4b832a2f7fb1ba3ed6a7dd591b3131e1f20e5b1f75335d67d487a0
+  data.tar.gz: 31d37614299b5877a287ef408d3f3a37b46f8b2c9111860ce438c60e69b7fbab11c3e7531e433d0d54eea7f18e73f25a4d117b3364315dd3103102df082f788d

data/app/controllers/katello/api/v2/errata_controller.rb

@@ -35,7 +35,9 @@ module Katello
     end
 
     def all_for_content_view_filter(filter, _collection)
-      Erratum.joins(:repositories).merge(filter.applicable_repos)
+      available_ids = Erratum.joins(:repositories).merge(filter.applicable_repos)&.pluck(:errata_id) || []
+      added_ids = filter&.erratum_rules&.pluck(:errata_id) || []
+      Erratum.where(errata_id: available_ids + added_ids)
     end
 
     def custom_index_relation(collection)

data/app/controllers/katello/api/v2/module_streams_controller.rb

@@ -36,7 +36,9 @@ module Katello
     end
 
     def all_for_content_view_filter(filter, _collection)
-      ModuleStream.joins(:repositories).merge(filter.applicable_repos)
+      available_ids = ModuleStream.joins(:repositories).merge(filter.applicable_repos)&.pluck(:id) || []
+      added_ids = filter&.module_stream_rules&.pluck(:module_stream_id) || []
+      ModuleStream.where(id: available_ids + added_ids)
     end
 
     def available_for_content_view_filter(filter, _collection)

data/app/controllers/katello/api/v2/package_groups_controller.rb

@@ -67,7 +67,9 @@ module Katello
     end
 
     def all_for_content_view_filter(filter, _collection)
-      PackageGroup.joins(:repositories).merge(filter.applicable_repos)
+      available_ids = PackageGroup.joins(:repositories).merge(filter.applicable_repos)&.pluck(:pulp_id) || []
+      added_ids = filter&.package_group_rules&.pluck(:uuid) || []
+      PackageGroup.where(pulp_id: available_ids + added_ids)
     end
 
     def default_sort

data/app/lib/actions/pulp3/repository/refresh_distribution.rb

@@ -3,7 +3,6 @@ module Actions
     module Repository
       class RefreshDistribution < Pulp3::AbstractAsyncTask
         include Helpers::Presenter
-        middleware.use Actions::Middleware::ExecuteIfContentsChanged
 
         def plan(repository, smart_proxy, options = {})
           smart_proxy = SmartProxy.unscoped.find_by(id: smart_proxy) #support bulk actions
@@ -16,11 +15,9 @@ module Actions
               :repository_id => repository.id,
               :smart_proxy_id => smart_proxy.id
             }
-            refresh_options[:contents_changed] if options.key?(:contents_changed)
             action = plan_self(refresh_options)
 
-            plan_action(SaveDistributionReferences, repository, smart_proxy,
-                        action.output, :contents_changed => options[:contents_changed])
+            plan_action(SaveDistributionReferences, repository, smart_proxy, action.output)
           end
         end
 

data/app/lib/actions/pulp3/repository/save_distribution_references.rb

@@ -2,8 +2,6 @@ module Actions
   module Pulp3
     module Repository
       class SaveDistributionReferences < Pulp3::Abstract
-        middleware.use Actions::Middleware::ExecuteIfContentsChanged
-
         def plan(repository, smart_proxy, tasks, options = {})
           plan_self(repository_id: repository.id, smart_proxy_id: smart_proxy.id, tasks: tasks, contents_changed: options[:contents_changed])
         end

data/app/services/katello/content_unit_indexer.rb

@@ -22,7 +22,6 @@ module Katello
     end
 
     def import_all(filtered_indexing = false)
-      additive = filtered_indexing || (@repository&.mirroring_policy == 'additive')
       association_tracker = RepoAssociationTracker.new(@content_type, @service_class, @repository)
       units_from_pulp.each do |units|
         units.each do |unit|
@@ -52,7 +51,7 @@ module Katello
       end
 
       if @model_class.many_repository_associations && @repository
-        sync_repository_associations(association_tracker, additive: additive)
+        sync_repository_associations(association_tracker, additive: filtered_indexing)
       end
     end
 

data/app/services/katello/pulp3/erratum.rb

@@ -119,7 +119,7 @@ module Katello
         date.to_i.to_s == date ? epoch_to_date(date) : date
       end
 
-      def epoch_to_date(epoch)
+      def self.epoch_to_date(epoch)
         Time.at(epoch.to_i).to_s
       end
     end

data/app/services/katello/pulp3/repository.rb

@@ -285,11 +285,17 @@ module Katello
           return update_distribution
         end
 
-        if dist_ref
+        if dist && dist_ref
+          # If the saved distribution reference is wrong, delete it and use the existing distribution
+          if dist.pulp_href != dist_ref.href
+            dist_ref.destroy
+            save_distribution_references([dist.pulp_href])
+          end
           return update_distribution
         end
 
         # Since we got this far, we need to create a new distribution
+        # Note: the distribution reference can't be saved yet because distribution creation is async
         begin
           create_distribution(relative_path)
         rescue api.client_module::ApiError => e
@@ -307,6 +313,9 @@ module Katello
 
       def create_distribution(path)
         distribution_data = api.distribution_class.new(secure_distribution_options(path))
+        unless ::Katello::RepositoryTypeManager.find(repo.content_type).pulp3_skip_publication
+          fail_missing_publication(distribution_data.publication)
+        end
         api.distributions_api.create(distribution_data)
       end
 
@@ -314,9 +323,16 @@ module Katello
         api.distributions_api.list(args).results
       end
 
+      def read_distribution(href = distribution_reference.href)
+        ignore_404_exception { api.distributions_api.read(href) }
+      end
+
       def update_distribution
         if distribution_reference
           options = secure_distribution_options(relative_path).except(:name)
+          unless ::Katello::RepositoryTypeManager.find(repo.content_type).pulp3_skip_publication
+            fail_missing_publication(options[:publication])
+          end
           distribution_reference.update(:content_guard_href => options[:content_guard])
           api.distributions_api.partial_update(distribution_reference.href, options)
         end
@@ -375,6 +391,12 @@ module Katello
         hrefs.each do |href|
           pulp3_distribution_data = api.get_distribution(href)
           path, content_guard_href = pulp3_distribution_data&.base_path, pulp3_distribution_data&.content_guard
+          if distribution_reference
+            found_distribution = read_distribution(distribution_reference.href)
+            unless found_distribution
+              distribution_reference.destroy
+            end
+          end
           unless distribution_reference
             # Ensure that duplicates won't be created in the case of a race condition
             DistributionReference.where(path: path, href: href, repository_id: repo.id, content_guard_href: content_guard_href).first_or_create!
@@ -529,6 +551,12 @@ module Katello
         return 0 if root.retain_package_versions_count.nil? || root.using_mirrored_content?
         root.retain_package_versions_count.to_i
       end
+
+      def fail_missing_publication(publication_href)
+        unless lookup_publication(publication_href)
+          fail _("The repository's publication is missing. Please run a 'complete sync' on %s." % repo.name)
+        end
+      end
     end
   end
 end

data/app/views/foreman/smart_proxies/show.html.erb

@@ -6,4 +6,6 @@
   <div ng-controller="PulpPrimaryController">
     <%= render :file => 'smart_proxies/show' %>
   </div>
+<% else -%>
+  <%= render :file => 'smart_proxies/show' %>
 <% end -%>

data/app/views/katello/api/v2/module_streams/show.json.rabl

@@ -6,13 +6,6 @@ child :artifacts => :artifacts do
   attributes :id, :name
 end
 
-child :profiles => :profiles do
-  attributes :id, :name
-  child :rpms => :rpms do
-    attributes :id, :name
-  end
-end
-
 child :library_repositories => :repositories do
   attributes :id, :name
   glue :product do

data/db/migrate/20211220185935_clean_duplicate_content_units.rb

@@ -20,7 +20,7 @@ class CleanDuplicateContentUnits < ActiveRecord::Migration[6.0]
     to_delete = []
     to_update = []
     duplicate_models.each do |duplicate|
-      query = {new_id_field => new_id}
+      query = { new_id_field => new_id }
       unique_fields.each do |field|
         query[field] = duplicate.send(field)
       end
@@ -52,28 +52,25 @@ class CleanDuplicateContentUnits < ActiveRecord::Migration[6.0]
   def up
     handle_null_pulp_ids
     add_foreign_keys
+    delete_deprecated_models
 
     handle_duplicate(Katello::ModuleStream,
                      'module_stream_id',
                      [:pulp_id],
-                     associated_models: {Katello::RepositoryModuleStream => :repository_id},
+                     associated_models: { Katello::RepositoryModuleStream => :repository_id },
                      child_models: [Katello::ContentViewModuleStreamFilterRule,
                                     Katello::ContentFacetApplicableModuleStream,
                                     Katello::ModuleProfile,
                                     Katello::ModuleStreamArtifact,
                                     Katello::ModuleStreamErratumPackage,
                                     Katello::ModuleStreamRpm])
-    add_index :katello_module_streams, :pulp_id, :unique => true
 
-    handle_duplicate(Katello::ModuleProfile, 'module_profile_id', [:module_stream_id, :name],
-                     child_models: [Katello::ModuleProfileRpm])
+    add_index :katello_module_streams, :pulp_id, :unique => true
     add_index :katello_module_profiles, [:module_stream_id, :name], :unique => true
-
-    handle_duplicate(Katello::ModuleProfileRpm, 'module_profile_rpm_id', [:module_profile_id, :name])
     add_index :katello_module_profile_rpms, [:module_profile_id, :name], :unique => true
 
     handle_duplicate(Katello::AnsibleTag, 'ansible_tag_id', [:name],
-                     associated_models: {Katello::AnsibleCollectionTag => :ansible_collection_id})
+                     associated_models: { Katello::AnsibleCollectionTag => :ansible_collection_id })
     add_index :katello_ansible_tags, [:name], :unique => true
 
     handle_duplicate(Katello::AnsibleCollectionTag, 'ansible_collection_tag_id', [:ansible_collection_id, :ansible_tag_id])
@@ -83,7 +80,7 @@ class CleanDuplicateContentUnits < ActiveRecord::Migration[6.0]
     handle_duplicate(Katello::GenericContentUnit,
                      'generic_content_unit_id',
                      [:pulp_id],
-                     associated_models: {Katello::RepositoryGenericContentUnit => :repository_id})
+                     associated_models: { Katello::RepositoryGenericContentUnit => :repository_id })
     add_index :katello_generic_content_units, :pulp_id, :unique => true
 
     handle_duplicate(Katello::DockerManifestList, 'docker_manifest_list_id', [:pulp_id],
@@ -112,6 +109,11 @@ class CleanDuplicateContentUnits < ActiveRecord::Migration[6.0]
     change_column :katello_docker_manifests, :pulp_id, :string, :null => false
   end
 
+  def delete_deprecated_models
+    Katello::ModuleProfileRpm.delete_all
+    Katello::ModuleProfile.delete_all
+  end
+
   def add_foreign_keys
     Katello::DockerManifestListManifest.where.not(docker_manifest_list_id: Katello::DockerManifestList.pluck(:id)).delete_all
     add_foreign_key :katello_docker_manifest_list_manifests, :katello_docker_manifest_lists, column: :docker_manifest_list_id

data/db/migrate/20220303160220_remove_duplicate_errata.rb

@@ -1,4 +1,8 @@
 class RemoveDuplicateErrata < ActiveRecord::Migration[6.0]
+  # rubocop:disable Metrics/MethodLength
+  # rubocop:disable Metrics/AbcSize
+  # rubocop:disable Metrics/CyclomaticComplexity
+  # rubocop:disable Metrics/PerceivedComplexity
   def up
     #Update all unique errata records to have pulp_id = errata_id
     ::Katello::Erratum.group(:errata_id).having("count(errata_id) = 1").pluck(:errata_id).each do |original_errata_id|
@@ -24,9 +28,36 @@ class RemoveDuplicateErrata < ActiveRecord::Migration[6.0]
           repo_erratum.update(erratum_id: errata_to_keep.id)
         end
       end
+      ::Katello::ContentFacetErratum.where(erratum_id: dup_errata&.map(&:id)).each do |host_erratum|
+        if ::Katello::ContentFacetErratum.find_by(content_facet_id: host_erratum.content_facet_id, erratum_id: errata_to_keep.id)
+          host_erratum.delete
+        else
+          host_erratum.update(erratum_id: errata_to_keep.id)
+        end
+      end
+      dup_errata_ids = dup_errata&.pluck(:id)
+
+      erratum_packages = ::Katello::ErratumPackage.where(:erratum_id => dup_errata_ids)
+      erratum_packages.each do |dup_err_package|
+        erratum_package_to_keep = ::Katello::ErratumPackage.find_by(erratum_id: errata_to_keep.id, nvrea: dup_err_package.nvrea)
+        ::Katello::ModuleStreamErratumPackage.where(erratum_package_id: dup_err_package).each do |dup_mod_errata_package|
+          if ::Katello::ModuleStreamErratumPackage.find_by(module_stream_id: dup_mod_errata_package.module_stream_id, erratum_package_id: erratum_package_to_keep&.id)
+            dup_mod_errata_package.delete
+          else
+            begin
+              dup_mod_errata_package.update(erratum_package_id: erratum_package_to_keep&.id)
+            rescue
+              dup_mod_errata_package.delete
+            end
+          end
+        end
+      end
+
       dup_errata_ids = dup_errata&.pluck(:id)
       if dup_errata_ids&.present?
-        ::Katello::ErratumPackage.where(:erratum_id => dup_errata_ids).delete_all
+        erratum_packages = ::Katello::ErratumPackage.where(:erratum_id => dup_errata_ids)
+        ::Katello::ModuleStreamErratumPackage.where(erratum_package_id: erratum_packages).delete_all
+        erratum_packages.delete_all
         ::Katello::ErratumBugzilla.where(:erratum_id => dup_errata_ids).delete_all
         ::Katello::ErratumCve.where(:erratum_id => dup_errata_ids).delete_all
         ::Katello::Erratum.where(:id => dup_errata_ids).delete_all
@@ -34,7 +65,11 @@ class RemoveDuplicateErrata < ActiveRecord::Migration[6.0]
     end
   end
 
+  # rubocop:enable Metrics/MethodLength
+  # rubocop:enable Metrics/AbcSize
+  # rubocop:enable Metrics/CyclomaticComplexity
+  # rubocop:enable Metrics/PerceivedComplexity
   def down
-    fail ActiveRecord::IrreversibleMigration
+    #Don't do anything on reverse
   end
 end

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/products/details/repositories/mirroring-policy.service.js

@@ -10,7 +10,7 @@
 angular.module('Bastion.repositories').service('MirroringPolicy',
     ['translate', function (translate) {
 
-        this.defaultMirroringPolicy = 'additive';
+        this.defaultMirroringPolicy = 'mirror_content_only';
 
         this.mirroringPolicies = function(repoType) {
             var policies = {

data/lib/katello/version.rb

@@ -1,3 +1,3 @@
 module Katello
-  VERSION = "4.4.0.2".freeze
+  VERSION = "4.4.2".freeze
 end

data/webpack/components/Table/TableHooks.js

@@ -139,6 +139,7 @@ export const useBulkSelect = ({
   initialArry = [],
   initialSearchQuery = '',
   idColumn = 'id',
+  filtersQuery = '',
   isSelectable,
 }) => {
   const { selectionSet: inclusionSet, ...selectOptions } =
@@ -200,16 +201,16 @@ export const useBulkSelect = ({
     }
   };
 
-  const fetchBulkParams = () => {
+  const fetchBulkParams = (idColumnName = idColumn) => {
     const searchQueryWithExclusionSet = () => {
-      const query = [searchQuery,
-        !isEmpty(exclusionSet) && `${idColumn} !^ (${[...exclusionSet].join(',')})`];
+      const query = [searchQuery, filtersQuery,
+        !isEmpty(exclusionSet) && `${idColumnName} !^ (${[...exclusionSet].join(',')})`];
       return query.filter(item => item).join(' and ');
     };
 
     const searchQueryWithInclusionSet = () => {
       if (isEmpty(inclusionSet)) throw new Error('Cannot build a search query with no items selected');
-      return `${idColumn} ^ (${[...inclusionSet].join(',')})`;
+      return `${idColumnName} ^ (${[...inclusionSet].join(',')})`;
     };
 
     return selectAllMode ? searchQueryWithExclusionSet() : searchQueryWithInclusionSet();

data/webpack/components/Table/__test__/useBulkSelect.test.js

@@ -0,0 +1,99 @@
+import { act, renderHook } from '@testing-library/react-hooks';
+import { useBulkSelect } from '../TableHooks';
+
+const isSelectable = () => true;
+const idColumn = 'errata_id';
+const metadata = {
+  error: null, selectable: 2, subtotal: 2, total: 2,
+};
+const results = [
+  {
+    errata_id: 'RHSA-2022:2031',
+    id: 311,
+    severity: 'Low',
+    type: 'security',
+  },
+  {
+    errata_id: 'RHSA-2022:2110',
+    id: 17,
+    severity: 'Low',
+    type: 'security',
+  },
+];
+
+it('returns a scoped search string based on inclusionSet', () => {
+  const { result } = renderHook(() => useBulkSelect({
+    results,
+    metadata,
+    idColumn,
+    isSelectable,
+  }));
+
+  act(() => {
+    result.current.selectOne(true, 'RHSA-2022:2031');
+  });
+
+  expect(result.current.fetchBulkParams()).toBe('errata_id ^ (RHSA-2022:2031)');
+});
+
+it('returns a scoped search string based on exclusionSet', () => {
+  const { result } = renderHook(() => useBulkSelect({
+    results,
+    metadata,
+    idColumn,
+    isSelectable,
+  }));
+
+  act(() => {
+    result.current.selectAll(true);
+  });
+
+  act(() => {
+    result.current.selectOne(false, 'RHSA-2022:2031');
+  });
+
+  expect(result.current.fetchBulkParams()).toBe('errata_id !^ (RHSA-2022:2031)');
+});
+
+it('adds search query to scoped search string based on exclusionSet', () => {
+  const { result } = renderHook(() => useBulkSelect({
+    results,
+    metadata,
+    idColumn,
+    isSelectable,
+  }));
+
+  act(() => {
+    result.current.updateSearchQuery('type=security');
+  });
+
+  act(() => {
+    result.current.selectAll(true);
+  });
+
+  act(() => {
+    result.current.selectOne(false, 'RHSA-2022:2031');
+  });
+
+  expect(result.current.fetchBulkParams()).toBe('type=security and errata_id !^ (RHSA-2022:2031)');
+});
+
+it('adds filter dropdown query to scoped search string', () => {
+  const { result } = renderHook(() => useBulkSelect({
+    results,
+    metadata,
+    idColumn,
+    isSelectable,
+    filtersQuery: 'severity=Low',
+  }));
+
+  act(() => {
+    result.current.selectAll(true);
+  });
+
+  act(() => {
+    result.current.selectOne(false, 'RHSA-2022:2031');
+  });
+
+  expect(result.current.fetchBulkParams()).toBe('severity=Low and errata_id !^ (RHSA-2022:2031)');
+});

data/webpack/components/extensions/HostDetails/Tabs/ErrataTab/ErrataTab.js

@@ -79,6 +79,17 @@ export const ErrataTab = () => {
     __('Published date'),
   ];
 
+  const filtersQuery = () => {
+    const query = [];
+    if (errataTypeSelected !== ERRATA_TYPE) {
+      query.push(`type=${TYPES_TO_PARAM[errataTypeSelected]}`);
+    }
+    if (errataSeveritySelected !== ERRATA_SEVERITY) {
+      query.push(`severity=${SEVERITIES_TO_PARAM[errataSeveritySelected]}`);
+    }
+    return query.join(' and ');
+  };
+
   const fetchItems = useCallback(
     (params) => {
       if (!hostId) return hostIdNotReady;
@@ -112,6 +123,7 @@ export const ErrataTab = () => {
     results,
     metadata,
     idColumn: 'errata_id',
+    filtersQuery: filtersQuery(),
     isSelectable: result => result.installable,
     initialSearchQuery: searchParam || '',
   });
@@ -248,7 +260,7 @@ export const ErrataTab = () => {
   );
 
   const hostIsNonLibrary = (
-    contentFacet?.contentViewDefault === false && contentFacet.lifecycleEnvironmentLibrary === false
+    contentFacet?.contentViewDefault === false || contentFacet.lifecycleEnvironmentLibrary === false
   );
   const toggleGroup = (
     <Split hasGutter>

data/webpack/components/extensions/HostDetails/Tabs/RepositorySetsTab/RepositorySetsTab.js

@@ -75,7 +75,7 @@ const RepositorySetsTab = () => {
     contentViewName,
     lifecycleEnvironmentName,
   } = contentFacet;
-  const nonLibraryHost = contentViewDefault === false &&
+  const nonLibraryHost = contentViewDefault === false ||
     lifecycleEnvironmentLibrary === false;
   const simpleContentAccess = (Number(subscriptionStatus) === 5);
   const dispatch = useDispatch();

data/webpack/components/extensions/HostDetails/Tabs/__tests__/errataTab.test.js

@@ -558,7 +558,7 @@ test('Toggle Group shows if it\'s not the default content view or library enviro
   assertNockRequest(scope, done); // Pass jest callback to confirm test is done
 });
 
-test('Toggle Group does not show if it\'s the default content view ', async (done) => {
+test('Toggle Group shows if it\'s the default content view but non-library environment', async (done) => {
   const options = renderOptions({
     ...contentFacetAttributes,
     content_view_default: true,
@@ -579,14 +579,41 @@ test('Toggle Group does not show if it\'s the default content view ', async (don
 
   // Assert that the errata are now showing on the screen, but wait for them to appear.
   await patientlyWaitFor(() => expect(getAllByText('Important')[0]).toBeInTheDocument());
-  expect(queryByLabelText('Installable Errata')).not.toBeInTheDocument();
+  expect(queryByLabelText('Installable Errata')).toBeInTheDocument();
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(scope, done); // Pass jest callback to confirm test is done
+});
+
+test('Toggle Group shows if it\'s the library environment but non-default content view', async (done) => {
+  const options = renderOptions({
+    ...contentFacetAttributes,
+    lifecycle_environment_library: true,
+  });
+  // Setup autocomplete with mockForemanAutoComplete since we aren't adding /katello
+  const autocompleteScope = mockForemanAutocomplete(nockInstance, autocompleteUrl);
+  const mockErrata = makeMockErrata({});
+  // return errata data results when we look for errata
+  const scope = nockInstance
+    .get(hostErrata)
+    .query(defaultQuery)
+    .reply(200, mockErrata);
+
+  const {
+    queryByLabelText,
+    getAllByText,
+  } = renderWithRedux(<ErrataTab />, options);
+
+  // Assert that the errata are now showing on the screen, but wait for them to appear.
+  await patientlyWaitFor(() => expect(getAllByText('Important')[0]).toBeInTheDocument());
+  expect(queryByLabelText('Installable Errata')).toBeInTheDocument();
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope, done); // Pass jest callback to confirm test is done
 });
 
-test('Toggle Group does not show if it\'s the  library environment', async (done) => {
+test('Toggle Group does not show if it\'s the default content view and library environment', async (done) => {
   const options = renderOptions({
     ...contentFacetAttributes,
+    content_view_default: true,
     lifecycle_environment_library: true,
   });
   // Setup autocomplete with mockForemanAutoComplete since we aren't adding /katello

data/webpack/components/extensions/HostDetails/Tabs/__tests__/repositorySetsTab.test.js

@@ -36,7 +36,7 @@ const autocompleteUrl = '/repository_sets/auto_complete_search';
 const repositorySetBookmarks = foremanApi.getApiUrl('/bookmarks?search=controller%3Dkatello_product_contents');
 const contentOverride = foremanApi.getApiUrl('/hosts/1/subscriptions/content_override');
 
-const defaultQuery = {
+const limitToEnvQuery = {
   content_access_mode_env: true,
   content_access_mode_all: true,
   host_id: 1,
@@ -44,8 +44,8 @@ const defaultQuery = {
   page: 1,
   search: '',
 };
-const libraryQuery = {
-  ...defaultQuery,
+const showAllQuery = {
+  ...limitToEnvQuery,
   content_access_mode_env: false,
 };
 
@@ -73,7 +73,7 @@ test('Can call API for repository sets and show basic table', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
 
   const { getByText } = renderWithRedux(<RepositorySetsTab />, renderOptions());
@@ -97,7 +97,7 @@ test('Can handle no repository sets being present', async (done) => {
 
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, noResults);
 
   const { queryByText } = renderWithRedux(<RepositorySetsTab />, renderOptions());
@@ -113,7 +113,7 @@ test('Toggle Group shows if it\'s not the default content view or library enviro
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
 
   const {
@@ -128,7 +128,7 @@ test('Toggle Group shows if it\'s not the default content view or library enviro
   assertNockRequest(scope, done); // Pass jest callback to confirm test is done
 });
 
-test('Toggle Group does not show if it\'s the default content view ', async (done) => {
+test('Toggle Group shows if it\'s the default content view but non-library environment', async (done) => {
   const options = renderOptions({
     ...contentFacetAttributes,
     content_view_default: true,
@@ -137,7 +137,7 @@ test('Toggle Group does not show if it\'s the default content view ', async (don
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(libraryQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
 
   const {
@@ -147,12 +147,12 @@ test('Toggle Group does not show if it\'s the default content view ', async (don
 
   // Assert that the errata are now showing on the screen, but wait for them to appear.
   await patientlyWaitFor(() => expect(getByText(firstRepoSet.contentUrl)).toBeInTheDocument());
-  expect(queryByLabelText('Limit to environment')).not.toBeInTheDocument();
+  expect(queryByLabelText('Limit to environment')).toBeInTheDocument();
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope, done); // Pass jest callback to confirm test is done
 });
 
-test('Toggle Group does not show if it\'s the library environment', async (done) => {
+test('Toggle Group shows if it\'s the library environment but a non-default content view', async (done) => {
   const options = renderOptions({
     ...contentFacetAttributes,
     lifecycle_environment_library: true,
@@ -161,7 +161,32 @@ test('Toggle Group does not show if it\'s the library environment', async (done)
   // return errata data results when we look for errata
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(libraryQuery)
+    .query(limitToEnvQuery)
+    .reply(200, mockRepoSetData);
+
+  const {
+    queryByLabelText,
+    getByText,
+  } = renderWithRedux(<RepositorySetsTab />, options);
+
+  // Assert that the errata are now showing on the screen, but wait for them to appear.
+  await patientlyWaitFor(() => expect(getByText(firstRepoSet.contentUrl)).toBeInTheDocument());
+  expect(queryByLabelText('Limit to environment')).toBeInTheDocument();
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(scope, done); // Pass jest callback to confirm test is done
+});
+
+test('Toggle Group does not show if it\'s the library environment and default content view', async (done) => {
+  const options = renderOptions({
+    ...contentFacetAttributes,
+    lifecycle_environment_library: true,
+    content_view_default: true,
+  });
+  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
+  // return errata data results when we look for errata
+  const scope = nockInstance
+    .get(hostRepositorySets)
+    .query(showAllQuery)
     .reply(200, mockRepoSetData);
 
   const {
@@ -182,7 +207,7 @@ test('Can toggle with the Toggle Group ', async (done) => {
   // return errata data results when we look for errata
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
 
   const {
@@ -202,7 +227,7 @@ test('Can override to disabled', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
   const contentOverrideScope = nockInstance
     .put(contentOverride)
@@ -238,7 +263,7 @@ test('Can override to enabled', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
   const contentOverrideScope = nockInstance
     .put(contentOverride)
@@ -274,7 +299,7 @@ test('Can reset to default', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(hostRepositorySets)
-    .query(defaultQuery)
+    .query(limitToEnvQuery)
     .reply(200, mockRepoSetData);
   const contentOverrideScope = nockInstance
     .put(contentOverride)

data/webpack/scenes/ContentViews/Create/CreateContentViewForm.js

@@ -115,7 +115,7 @@ const CreateContentViewForm = ({ setModalOpen }) => {
               aria-label="component_tile"
               icon={<ContentViewIcon composite={false} />}
               id="component"
-              title={__('Component content view')}
+              title={__('Content view')}
               onClick={() => { setComponent(true); setComposite(false); }}
               isSelected={component}
             >
@@ -133,7 +133,7 @@ const CreateContentViewForm = ({ setModalOpen }) => {
               onClick={() => { setComposite(true); setComponent(false); }}
               isSelected={composite}
             >
-              {__('Consisting of multiple component content views')}
+              {__('Consisting of multiple content views')}
             </Tile>
           </GridItem>
         </Grid>

data/webpack/scenes/ContentViews/Create/__tests__/createContentView.test.js

@@ -68,7 +68,7 @@ test('Displays dependent fields correctly', () => {
   expect(getByText('Name')).toBeInTheDocument();
   expect(getByText('Label')).toBeInTheDocument();
   expect(getByText('Composite content view')).toBeInTheDocument();
-  expect(getByText('Component content view')).toBeInTheDocument();
+  expect(getByText('Content view')).toBeInTheDocument();
   expect(getByText('Solve dependencies')).toBeInTheDocument();
   expect(queryByText('Auto publish')).not.toBeInTheDocument();
   expect(getByText('Import only')).toBeInTheDocument();

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentContentViewAddModal.js

@@ -87,7 +87,7 @@ const ComponentContentViewAddModal = ({
 
   return (
     <Modal
-      title={componentId ? __('Update version') : __('Add component')}
+      title={componentId ? __('Update version') : __('Add content view')}
       variant={ModalVariant.small}
       isOpen={show}
       description={__(`Select available version of ${cvName} to use`)}

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentContentViewBulkAddModal.js

@@ -51,10 +51,10 @@ const ComponentContentViewBulkAddModal = ({ cvId, rowsToAdd, onClose }) => {
 
   return (
     <Modal
-      title={__('Add component content views')}
+      title={__('Add content views')}
       variant={ModalVariant.large}
       isOpen
-      description={__('Select available version of components to use')}
+      description={__('Select available version of content views to use')}
       onClose={onClose}
       appendTo={document.body}
     >

data/webpack/scenes/ContentViews/Details/ComponentContentViews/__tests__/contentViewComponents.test.js

@@ -212,7 +212,7 @@ test('Can add published component views to content view with modal', async (done
   });
   fireEvent.click(getByText('Add'));
   await patientlyWaitFor(() => {
-    expect(getByText('Add component')).toBeInTheDocument();
+    expect(getByText('Add content view')).toBeInTheDocument();
   });
   fireEvent.click(getByLabelText('add_component'));
   await patientlyWaitFor(() => {
@@ -317,7 +317,7 @@ test('Can bulk add component views to content view with modal', async (done) =>
     .reply(200, {});
 
   const {
-    getByText, getByLabelText, queryByText,
+    getAllByText, getByLabelText, queryByText,
   } = renderWithRedux(
     <ContentViewComponents cvId={4} details={cvDetails} />,
     renderOptions,
@@ -333,14 +333,14 @@ test('Can bulk add component views to content view with modal', async (done) =>
   });
   fireEvent.click(getByLabelText('bulk_add_components'));
   await patientlyWaitFor(() => {
-    expect(getByText('Add component content views')).toBeInTheDocument();
+    expect(getAllByText('Add content views')[1]).toBeInTheDocument();
   });
   fireEvent.click(getByLabelText('version-select-cv-10'));
   fireEvent.click(getByLabelText('cv-10-3.0'));
 
   fireEvent.click(getByLabelText('add_components'));
   await patientlyWaitFor(() => {
-    expect(queryByText('Add component content views')).not.toBeInTheDocument();
+    expect(queryByText('Select available version of content views to use')).not.toBeInTheDocument();
     expect(getByLabelText('bulk_add_components')).toHaveAttribute('aria-disabled', 'false');
   });
 

data/webpack/scenes/ContentViews/Details/ContentViewInfo.js

@@ -70,7 +70,7 @@ const ContentViewInfo = ({ cvId, details }) => {
         <TextListItem component={TextListItemVariants.dd} className="foreman-spaced-list">
           <Flex>
             <FlexItem spacer={{ default: 'spacerXs' }}>
-              <ContentViewIcon composite={composite} description={composite ? __('Composite') : __('Component')} />
+              <ContentViewIcon composite={composite} description={composite ? __('Composite') : __('Content view')} />
             </FlexItem>
           </Flex>
         </TextListItem>

data/webpack/scenes/ContentViews/__tests__/contentViewPage.test.js

@@ -43,14 +43,14 @@ test('Can call API for CVs and show on screen on page load', async (done) => {
     .query(true)
     .reply(200, cvIndexData);
 
-  const { queryByText } = renderWithRedux(<ContentViewsPage />, renderOptions);
+  const { queryByText, queryAllByText } = renderWithRedux(<ContentViewsPage />, renderOptions);
 
   expect(queryByText(firstCV.name)).toBeNull();
 
   // Assert that the CV name is now showing on the screen, but wait for it to appear.
   await patientlyWaitFor(() => {
     expect(queryByText(firstCV.name)).toBeInTheDocument();
-    expect(queryByText('Component content views')).toBeInTheDocument();
+    expect(queryAllByText('Content views')[0]).toBeInTheDocument();
     expect(queryByText('Composite content views')).toBeInTheDocument();
   });
 
@@ -354,7 +354,7 @@ test('Displays Create Content View and opens modal with Form', async () => {
   expect(queryByText('Name')).not.toBeInTheDocument();
   expect(queryByText('Label')).not.toBeInTheDocument();
   expect(queryByText('Composite content view')).not.toBeInTheDocument();
-  expect(queryByText('Component content view')).not.toBeInTheDocument();
+  expect(queryByText('Content view')).not.toBeInTheDocument();
   expect(queryByText('Solve dependencies')).not.toBeInTheDocument();
   expect(queryByText('Auto publish')).not.toBeInTheDocument();
   expect(queryByText('Import only')).not.toBeInTheDocument();
@@ -365,7 +365,7 @@ test('Displays Create Content View and opens modal with Form', async () => {
   expect(getByText('Name')).toBeInTheDocument();
   expect(getByText('Label')).toBeInTheDocument();
   expect(getByText('Composite content view')).toBeInTheDocument();
-  expect(getByText('Component content view')).toBeInTheDocument();
+  expect(getByText('Content view')).toBeInTheDocument();
   expect(getByText('Solve dependencies')).toBeInTheDocument();
   expect(queryByText('Auto publish')).not.toBeInTheDocument();
   expect(getByText('Import only')).toBeInTheDocument();

data/webpack/scenes/ContentViews/components/ContentViewsCounter.js

@@ -18,7 +18,7 @@ const ContentViewsCounter = () => {
         <b>
           <Flex>
             <FlexItem spacer={{ default: 'spacerXs' }}>
-              <ContentViewIcon composite={false} description={__('Component content views')} count={(component || component === 0) ? component : <InProgressIcon />} />
+              <ContentViewIcon composite={false} description={__('Content views')} count={(component || component === 0) ? component : <InProgressIcon />} />
             </FlexItem>
             <FlexItem>
               <Tooltip

data/webpack/scenes/ContentViews/expansions/DetailsExpansion.js

@@ -14,14 +14,14 @@ const DetailsExpansion = ({
     if (cvComposite) {
       return (
         <>
-          {__('Related component cvs: ')}
+          {__('Related content views: ')}
           <RelatedContentViewComponentsModal key="cvId" {...{ cvName, cvId, relatedCVCount }} />
         </>
       );
     }
     return (
       <>
-        {__('Related composite cvs: ')}
+        {__('Related composite content views: ')}
         <RelatedCompositeContentViewsModal
           key={cvId}
           {...{

data/webpack/scenes/ContentViews/expansions/RelatedContentViewComponentsModal.js

@@ -29,7 +29,7 @@ const RelatedContentViewsModal = ({ cvName, cvId, relatedCVCount }) => {
       <FlexItem>
         <RegistryIcon />
         <b>{` ${cvName}`}</b>
-        {__(' content view is used in listed component content views. For more information, ')}
+        {__(' content view is used in listed content views. For more information, ')}
         <Link to={urlBuilder(`content_views/${cvId}#/contentviews`, '')}>
           {__('view content view tabs.')}
         </Link>
@@ -49,7 +49,7 @@ const RelatedContentViewsModal = ({ cvName, cvId, relatedCVCount }) => {
       <Grid>
         <GridItem span={12}>
           <Modal
-            title={__('Related component content views')}
+            title={__('Related content views')}
             variant={ModalVariant.medium}
             isOpen={isOpen}
             description={description()}

data/webpack/scenes/ContentViews/expansions/__tests__/contentViewComponentsModal.test.js

@@ -30,7 +30,7 @@ test('Can call API and show Related Content Views Components Modal', async (done
 
   await patientlyWaitFor(() => expect(getByLabelText(`button_${cvId}`)).toBeInTheDocument());
   fireEvent.click(getByLabelText(`button_${cvId}`));
-  await patientlyWaitFor(() => expect(getByText('Related component content views')).toBeInTheDocument());
+  await patientlyWaitFor(() => expect(getByText('Related content views')).toBeInTheDocument());
 
   assertNockRequest(scope, done);
 });

data/webpack/scenes/ModuleStreams/Details/ModuleDetailsSchema.js

@@ -1,7 +1,6 @@
 import React from 'react';
 import { translate as __ } from 'foremanReact/common/I18n';
 import ModuleStreamDetailArtifacts from './ModuleStreamDetailArtifacts';
-import ModuleStreamDetailProfiles from './Profiles/ModuleStreamDetailProfiles';
 import ContentDetailInfo from '../../../components/Content/Details/ContentDetailInfo';
 import ContentDetailRepositories from '../../../components/Content/Details/ContentDetailRepositories';
 
@@ -17,7 +16,7 @@ export const displayMap = new Map([
 ]);
 
 export default (detailInfo) => {
-  const { repositories, profiles, artifacts } = detailInfo;
+  const { repositories, artifacts } = detailInfo;
 
   return [
     {
@@ -37,14 +36,6 @@ export default (detailInfo) => {
     },
     {
       key: 3,
-      tabHeader: __('Profiles'),
-      tabContent: (profiles && profiles.length ?
-        <ModuleStreamDetailProfiles profiles={profiles} /> :
-        __('No profiles to show')
-      ),
-    },
-    {
-      key: 4,
       tabHeader: __('Artifacts'),
       tabContent: (artifacts && artifacts.length ?
         <ModuleStreamDetailArtifacts artifacts={artifacts} /> :

data/webpack/scenes/ModuleStreams/Details/ModuleStreamDetails.js

@@ -44,21 +44,20 @@ class ModuleStreamDetails extends Component {
     return (
       <div>
         {!loading && <BreadcrumbsBar
+          isLoadingResources={loading}
           onSwitcherItemClick={(e, url) => this.handleBreadcrumbSwitcherItem(e, url)}
-          data={{
-            isSwitchable: true,
-            breadcrumbItems: [
-              {
-                caption: __('Module Streams'),
-                onClick: () =>
-                  this.props.history.push('/module_streams'),
-              },
-              {
-                caption: `${name} ${stream}`,
-              },
-            ],
-            resource,
-          }}
+          isSwitchable
+          breadcrumbItems={[
+            {
+              caption: __('Module Streams'),
+              onClick: () =>
+                this.props.history.push('/module_streams'),
+            },
+            {
+              caption: `${name} ${stream}`,
+            },
+          ]}
+          resource={resource}
         />}
         <ContentDetails
           contentDetails={moduleStreamDetails}

data/webpack/scenes/ModuleStreams/Details/__tests__/__snapshots__/ModuleStreamDetails.test.js.snap

@@ -76,11 +76,6 @@ exports[`Module stream details page rendering renders with loading state 1`] = `
         },
         Object {
           "key": 3,
-          "tabContent": "No profiles to show",
-          "tabHeader": "Profiles",
-        },
-        Object {
-          "key": 4,
           "tabContent": "No artifacts to show",
           "tabHeader": "Artifacts",
         },
@@ -93,26 +88,26 @@ exports[`Module stream details page rendering renders with loading state 1`] = `
 exports[`Module stream details page rendering renders with module stream provided 1`] = `
 <div>
   <BreadcrumbsBar
-    data={
-      Object {
-        "breadcrumbItems": Array [
-          Object {
-            "caption": "Module Streams",
-            "onClick": [Function],
-          },
-          Object {
-            "caption": "avocado latest",
-          },
-        ],
-        "isSwitchable": true,
-        "resource": Object {
-          "nameField": "name",
-          "resourceUrl": "/katello/api/v2/module_streams",
-          "switcherItemUrl": "/module_streams/:id",
+    breadcrumbItems={
+      Array [
+        Object {
+          "caption": "Module Streams",
+          "onClick": [Function],
         },
-      }
+        Object {
+          "caption": "avocado latest",
+        },
+      ]
     }
+    isSwitchable={true}
     onSwitcherItemClick={[Function]}
+    resource={
+      Object {
+        "nameField": "name",
+        "resourceUrl": "/katello/api/v2/module_streams",
+        "switcherItemUrl": "/module_streams/:id",
+      }
+    }
   />
   <ContentDetails
     contentDetails={
@@ -397,84 +392,6 @@ exports[`Module stream details page rendering renders with module stream provide
         },
         Object {
           "key": 3,
-          "tabContent": <ModuleStreamDetailProfiles
-            profiles={
-              Array [
-                Object {
-                  "id": 37,
-                  "name": "default",
-                  "rpms": Array [
-                    Object {
-                      "id": 108,
-                      "name": "perl",
-                    },
-                    Object {
-                      "id": 110,
-                      "name": "foo",
-                    },
-                    Object {
-                      "id": 111,
-                      "name": "rpm_0",
-                    },
-                    Object {
-                      "id": 112,
-                      "name": "rpm_1",
-                    },
-                    Object {
-                      "id": 113,
-                      "name": "rpm_2",
-                    },
-                    Object {
-                      "id": 114,
-                      "name": "rpm_3",
-                    },
-                    Object {
-                      "id": 115,
-                      "name": "rpm_4",
-                    },
-                    Object {
-                      "id": 116,
-                      "name": "rpm_5",
-                    },
-                    Object {
-                      "id": 117,
-                      "name": "rpm_6",
-                    },
-                    Object {
-                      "id": 118,
-                      "name": "rpm_7",
-                    },
-                    Object {
-                      "id": 119,
-                      "name": "rpm_8",
-                    },
-                    Object {
-                      "id": 120,
-                      "name": "rpm_9",
-                    },
-                    Object {
-                      "id": 121,
-                      "name": "rpm_10",
-                    },
-                  ],
-                },
-                Object {
-                  "id": 38,
-                  "name": "minimal",
-                  "rpms": Array [
-                    Object {
-                      "id": 84,
-                      "name": "python2-avocado",
-                    },
-                  ],
-                },
-              ]
-            }
-          />,
-          "tabHeader": "Profiles",
-        },
-        Object {
-          "key": 4,
           "tabContent": <ModuleStreamDetailArtifacts
             artifacts={
               Array [

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: katello
 version: !ruby/object:Gem::Version
-  version: 4.4.0.2
+  version: 4.4.2
 platform: ruby
 authors:
 - N/A
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-04-11 00:00:00.000000000 Z
+date: 2022-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -87,6 +87,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -94,6 +97,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7.0'
 - !ruby/object:Gem::Dependency
   name: foreman_remote_execution
   requirement: !ruby/object:Gem::Requirement
@@ -4690,6 +4696,7 @@ files:
 - webpack/components/Table/PageControls.js
 - webpack/components/Table/TableHooks.js
 - webpack/components/Table/TableWrapper.js
+- webpack/components/Table/__test__/useBulkSelect.test.js
 - webpack/components/Table/helpers.js
 - webpack/components/TooltipButton/TooltipButton.js
 - webpack/components/TooltipButton/TooltipButton.scss
@@ -5388,7 +5395,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.2.33
 signing_key:
 specification_version: 4
 summary: Content and Subscription Management plugin for Foreman