foreman_scc_manager 2.2.3 → 3.0.0

data/test/controllers/api/v2/scc_accounts_test.rb

@@ -13,7 +13,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -35,12 +35,12 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=TG9yZW0gaXBzdW0gZG9sb3Igc2l0IGFtZXQsIGNvbnNlY3RldHVyIGFkaXBp%0Ac2NpbmcgZWxpdCwgc2VkIGRvIGVpdXNtb2QgdGVtcG9yIGluY2l; path=/; secure',
-            'Uy#u~osh#oh3ahv.op0OII; Expires=Fri, 02-Mar-2029 15:07:20 GMT; Path=/'
+            'Uy#u~osh#oh3ahv.op0OII; Expires=Fri, 02-Mar-2029 15:07:20 GMT; Path=/',
           ],
           x_request_id: '67450237-e4aa-4994-a47d-ed3ce142555b',
           x_runtime: '0.144083',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
 
@@ -52,7 +52,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -74,12 +74,12 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=SEZSwgerzOHlW5vYxxcVxrRQ3J4lKMmwGAIXpDyuTyCtcgQqrkMDlSdlxhU6rQp71%2Fdic6jiKgVbJB6vCeAJ2A%3D%3D; path=/; secure',
-            'TbBx+jfg=v1XitvAA@@ncY; Expires=Thu, 04-Oct-2029 13:14:57 GMT; Path=/'
+            'TbBx+jfg=v1XitvAA@@ncY; Expires=Thu, 04-Oct-2029 13:14:57 GMT; Path=/',
           ],
           x_request_id: '2418bd69-efab-4d78-9a73-63570744d2cb',
           x_runtime: '0.645444',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
     ############
@@ -91,7 +91,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -114,12 +114,12 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=EABKsiefcpa7dMNEXRixmihKeUfIvXF4AwmNQt2wZG5Fm%2FPKvR0%2FMBDVV5lZJ3p4waUAcds2xWv42vbKg9GQhg%3D%3D; path=/; secure',
-            'TbBx+jfg=v1jitvAA@@UII; Expires=Thu, 08-Mar-2029 15:37:15 GMT; Path=/'
+            'TbBx+jfg=v1jitvAA@@UII; Expires=Thu, 08-Mar-2029 15:37:15 GMT; Path=/',
           ],
           x_request_id: 'd2797941-1aed-499c-8e06-b4cb52515443',
           x_runtime: '6.671012',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
     # products page2
@@ -128,7 +128,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -151,12 +151,12 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=z3bGc45lQxf%2FXq7qN7cwzJrK1zcw4e7uuskVCPejeN0zv3ExUcb8ev3jhGnDGJaSz3ZwV7Dk0SdLII%2FOcI2eEw%3D%3D; path=/; secure',
-            'TbBx+jfg=v1oytvAA@@I73; Expires=Thu, 08-Mar-2029 15:37:23 GMT; Path=/'
+            'TbBx+jfg=v1oytvAA@@I73; Expires=Thu, 08-Mar-2029 15:37:23 GMT; Path=/',
           ],
           x_request_id: '17e6707a-1134-403d-a49c-7344442446c1',
           x_runtime: '6.671012',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
   end
@@ -203,28 +203,28 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
 
   test 'should update scc_account' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :interval => 'weekly' } }
+    put :update, params: { id: account.id, scc_account: { :interval => 'weekly' } }
     assert_equal 'weekly', assigns(:scc_account).interval
     assert_response :ok
   end
 
   test 'should refuse update scc_account with invalid interval' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :interval => 'yearly' } }
+    put :update, params: { id: account.id, scc_account: { :interval => 'yearly' } }
     assert_equal 'yearly', assigns(:scc_account).interval
     assert_response :unprocessable_entity
   end
 
   test 'should refuse update scc_account with empty date' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :interval => 'weekly', :sync_date => '' } }
+    put :update, params: { id: account.id, scc_account: { :interval => 'weekly', :sync_date => '' } }
     assert_response :unprocessable_entity
     assert_error_message 'Sync date must be a valid datetime'
   end
 
   test 'should fail to update scc_account with interval set and invalid date' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :sync_date => 'invalid_date', :interval => 'weekly' } }
+    put :update, params: { id: account.id, scc_account: { :sync_date => 'invalid_date', :interval => 'weekly' } }
 
     assert_response :unprocessable_entity
     assert_error_message 'Sync date must be a valid datetime'
@@ -232,7 +232,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
 
   test 'should fail to update scc_account with empty name' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :name => '', :sync_date => Time.now, :interval => 'weekly' } }
+    put :update, params: { id: account.id, scc_account: { :name => '', :sync_date => Time.now, :interval => 'weekly' } }
 
     assert_response :unprocessable_entity
     assert_error_message "Name can't be blank"
@@ -240,7 +240,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
 
   test 'should fail to update scc_account with empty login' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :login => '', :sync_date => Time.now, :interval => 'weekly' } }
+    put :update, params: { id: account.id, scc_account: { :login => '', :sync_date => Time.now, :interval => 'weekly' } }
 
     assert_response :unprocessable_entity
     assert_error_message "Login can't be blank"
@@ -248,7 +248,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
 
   test 'should fail to update scc_account with empty base_url' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :base_url => '', :sync_date => Time.now, :interval => 'weekly' } }
+    put :update, params: { id: account.id, scc_account: { :base_url => '', :sync_date => Time.now, :interval => 'weekly' } }
 
     assert_response :unprocessable_entity
     assert_error_message "Base url can't be blank"
@@ -268,7 +268,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(status: 404)
     account = scc_accounts(:one)
@@ -294,7 +294,7 @@ class Api::V2::SccAccountsControllerTest < ActionController::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(status: 404)
     account = scc_accounts(:one)

data/test/controllers/scc_accounts_controller_test.rb

@@ -19,7 +19,7 @@ class SccAccountsControllerTest < ActionController::TestCase
           'Accept-Encoding' => 'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
           'Authorization' => auth,
           'Host' => host,
-          'User-Agent' => "rest-client/2.1.0 (linux x86_64) ruby/#{RUBY_VERSION}p#{RUBY_PATCHLEVEL}"
+          'User-Agent' => "rest-client/2.1.0 (linux x86_64) ruby/#{RUBY_VERSION}p#{RUBY_PATCHLEVEL}",
         }
       )
       .to_return(status: 200, body: '', headers: {})
@@ -74,14 +74,14 @@ class SccAccountsControllerTest < ActionController::TestCase
 
   test 'should update scc_account' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
     assert_redirected_to '/scc_accounts'
     assert_equal 'weekly', SccAccount.find(account.id).interval
   end
 
   test 'should update scc_account with empty date if interval not set' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :name => 'new_name', :sync_date => '', :interval => 'never' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :name => 'new_name', :sync_date => '', :interval => 'never' } }, session: set_session_user
 
     assert_equal 'new_name', SccAccount.find(account.id).name
   end
@@ -89,42 +89,42 @@ class SccAccountsControllerTest < ActionController::TestCase
   test 'updates scc_account even if the date is invalid' do
     # @todo reminder to fix this in the future
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :name => 'new_name', :sync_date => 'invalid_date', :interval => 'never' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :name => 'new_name', :sync_date => 'invalid_date', :interval => 'never' } }, session: set_session_user
 
     assert_not_equal account.name, SccAccount.find(account.id).name
   end
 
   test 'should fail to update scc_account with interval set and empty date' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :sync_date => '', :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :sync_date => '', :interval => 'weekly' } }, session: set_session_user
 
     assert_equal SccAccount.find(account.id).sync_date, account.sync_date
   end
 
   test 'should fail to update scc_account with interval set and invalid date' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :sync_date => 'invalid_date', :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :sync_date => 'invalid_date', :interval => 'weekly' } }, session: set_session_user
 
     assert_equal SccAccount.find(account.id).sync_date, account.sync_date
   end
 
   test 'should fail to update scc_account with empty name' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :name => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :name => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
 
     assert_equal account.name, SccAccount.find(account.id).name
   end
 
   test 'should fail to update scc_account with empty login' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :login => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :login => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
 
     assert_equal account.login, SccAccount.find(account.id).login
   end
 
   test 'should fail to update scc_account with empty base url' do
     account = scc_accounts(:two)
-    put :update, params: { id: account.id, :scc_account => { :base_url => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
+    put :update, params: { id: account.id, scc_account: { :base_url => '', :sync_date => Time.now, :interval => 'weekly' } }, session: set_session_user
 
     assert_equal account.base_url, SccAccount.find(account.id).base_url
   end

data/test/features/sync_test.rb

@@ -9,7 +9,7 @@ class SccAccountSyncTest < ActiveSupport::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -31,12 +31,12 @@ class SccAccountSyncTest < ActiveSupport::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=TG9yZW0gaXBzdW0gZG9sb3Igc2l0IGFtZXQsIGNvbnNlY3RldHVyIGFkaXBp%0Ac2NpbmcgZWxpdCwgc2VkIGRvIGVpdXNtb2QgdGVtcG9yIGluY2l; path=/; secure',
-            'Uy#u~osh#oh3ahv.op0OII; Expires=Fri, 02-Mar-2029 15:07:20 GMT; Path=/'
+            'Uy#u~osh#oh3ahv.op0OII; Expires=Fri, 02-Mar-2029 15:07:20 GMT; Path=/',
           ],
           x_request_id: '67450237-e4aa-4994-a47d-ed3ce142555b',
           x_runtime: '0.144083',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
     ############
@@ -48,7 +48,7 @@ class SccAccountSyncTest < ActiveSupport::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -71,12 +71,12 @@ class SccAccountSyncTest < ActiveSupport::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=EABKsiefcpa7dMNEXRixmihKeUfIvXF4AwmNQt2wZG5Fm%2FPKvR0%2FMBDVV5lZJ3p4waUAcds2xWv42vbKg9GQhg%3D%3D; path=/; secure',
-            'TbBx+jfg=v1jitvAA@@UII; Expires=Thu, 08-Mar-2029 15:37:15 GMT; Path=/'
+            'TbBx+jfg=v1jitvAA@@UII; Expires=Thu, 08-Mar-2029 15:37:15 GMT; Path=/',
           ],
           x_request_id: 'd2797941-1aed-499c-8e06-b4cb52515443',
           x_runtime: '6.671012',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
     # products page2
@@ -85,7 +85,7 @@ class SccAccountSyncTest < ActiveSupport::TestCase
         headers: {
           'Accept' => 'application/vnd.scc.suse.com.v4+json',
           'Authorization' => 'Basic b25ldXNlcjpvbmVwYXNz',
-          'Host' => 'scc.example.com'
+          'Host' => 'scc.example.com',
         }
       ).to_return(
         status: 200,
@@ -108,12 +108,12 @@ class SccAccountSyncTest < ActiveSupport::TestCase
           cache_control: 'max-age=0, private, must-revalidate',
           set_cookie: [
             'XSRF-TOKEN=z3bGc45lQxf%2FXq7qN7cwzJrK1zcw4e7uuskVCPejeN0zv3ExUcb8ev3jhGnDGJaSz3ZwV7Dk0SdLII%2FOcI2eEw%3D%3D; path=/; secure',
-            'TbBx+jfg=v1oytvAA@@I73; Expires=Thu, 08-Mar-2029 15:37:23 GMT; Path=/'
+            'TbBx+jfg=v1oytvAA@@I73; Expires=Thu, 08-Mar-2029 15:37:23 GMT; Path=/',
           ],
           x_request_id: '17e6707a-1134-403d-a49c-7344442446c1',
           x_runtime: '6.671012',
           strict_transport_security: 'max-age=15552000, max-age=300',
-          content_encoding: 'gzip'
+          content_encoding: 'gzip',
         }
       )
   end

data/test/fixtures/models/scc_repositories.yml

@@ -13,7 +13,7 @@ repo_<%= i %>:
   installer_updates: false
 <% end %>
 
-repo_9:
+repo9:
   id: 9
   scc_account_id: 1
   scc_id: 63

data/test/models/scc_account_test.rb

@@ -24,7 +24,7 @@ class SccAccountCreateTest < ActiveSupport::TestCase
       # base_url has a default value set in DB
       # base_url: 'https://scc.example.org',
       login: 'account1',
-      password: 'secret'
+      password: 'secret',
     }
 
     # for every key in hash try to create account without it set
@@ -114,8 +114,8 @@ end
 class SccAccountUpdateReposTest < ActiveSupport::TestCase
   def setup
     @account = scc_accounts(:one)
-    test_repo = scc_repositories(:repo_9)
-    @account.scc_repositories = SccProduct.where(name: 'repo_9')
+    test_repo = scc_repositories(:repo9)
+    @account.scc_repositories = SccProduct.where(name: 'repo9')
     # generate test data
     @repo_data = { 'name' => test_repo.name,
                    'id' => test_repo.scc_id,

data/test/models/scc_product_test.rb

@@ -22,7 +22,7 @@ class SccProductCreateTest < ActiveSupport::TestCase
 
   test 'pretty_description' do
     assert_equal @product_with_normal_description.description,
-                 @product_with_normal_description.pretty_description
+      @product_with_normal_description.pretty_description
     assert_equal @product.pretty_description, 'lorem ipsum dolor sit amet'
     assert_equal @product_with_ugly_description.pretty_description, 'lorem ipsum dolor sit amet'
   end

data/test/support/fixtures_support.rb

@@ -5,7 +5,7 @@ module ForemanSccManager
       scc_products: ::SccProduct,
       scc_repositories: ::SccRepository,
       katello_root_repositories: ::Katello::RootRepository,
-      scc_katello_repositories: SccKatelloRepository
+      scc_katello_repositories: SccKatelloRepository,
     }.freeze
 
     def self.set_fixture_classes(test_class)

data/test/test_plugin_helper.rb

@@ -19,17 +19,17 @@ module FixtureTestCase
 
     # Fixtures are copied into a separate path to combine with Foreman fixtures. This directory
     # is kept out of version control.
-    self.fixture_path = "#{Rails.root}/tmp/combined_fixtures/"
+    self.fixture_path = Rails.root.join('tmp/combined_fixtures/')
     FileUtils.rm_rf(self.fixture_path) if File.directory?(self.fixture_path)
     Dir.mkdir(self.fixture_path)
     FileUtils.cp(Dir.glob("#{ForemanSccManager::Engine.root}/test/fixtures/models/*"), self.fixture_path)
     FileUtils.cp(Dir.glob("#{ForemanSccManager::Engine.root}/test/fixtures/files/*"), self.fixture_path)
     FileUtils.cp(Dir.glob("#{ForemanSccManager::Engine.root}/test/fixtures/controllers/*"), self.fixture_path)
-    FileUtils.cp(Dir.glob("#{Rails.root}/test/fixtures/*"), self.fixture_path)
+    FileUtils.cp(Dir.glob(Rails.root.join('test/fixtures/*')), self.fixture_path)
     fixtures(:all)
-    FIXTURES = load_fixtures(ActiveRecord::Base)
+    fixtures_loaded = load_fixtures(ActiveRecord::Base)
 
-    User.current = ::User.unscoped.find(FIXTURES['users']['admin']['id'])
+    User.current = ::User.unscoped.find(fixtures_loaded['users']['admin']['id'])
   end
 end
 

data/webpack/components/SCCProductPage/components/SCCProductPicker/components/SCCTreePicker/components/SCCRepoPicker/index.js

@@ -8,7 +8,7 @@ import './styles.scss';
 const createRepoSelectOption = (repo, disableRepos) => (
   <SelectOption
     key={repo.id}
-    isDisabled={repo.katello_root_repository_id !== null || disableRepos}
+    isDisabled={repo.katello_repository_id !== null || disableRepos}
     value={repo.name}
   />
 );
@@ -30,14 +30,14 @@ const setRepoSelection = (
         (repo) =>
           (!repo.name.includes('Debug') &&
             !repo.name.includes('Source-Pool') &&
-            repo.katello_root_repository_id === null) ||
-          repo.katello_root_repository_id !== null
+            repo.katello_repository_id === null) ||
+          repo.katello_repository_id !== null
       );
     } else {
       res = sccRepos;
     }
   } else {
-    res = sccRepos.filter((repo) => repo.katello_root_repository_id !== null);
+    res = sccRepos.filter((repo) => repo.katello_repository_id !== null);
   }
   return res.map((repo) => repo.name);
 };
@@ -83,7 +83,7 @@ const SCCRepoPicker = ({
         .filter(
           (repo) =>
             selectedRepos.includes(repo.name) &&
-            repo.katello_root_repository_id === null
+            repo.katello_repository_id === null
         )
         .map((repo) => repo.id),
       sccRepos
@@ -91,7 +91,7 @@ const SCCRepoPicker = ({
         .filter(
           (repo) =>
             selectedRepos.includes(repo.name) &&
-            repo.katello_root_repository_id === null
+            repo.katello_repository_id === null
         )
         .map((repo) => repo.name)
     );
@@ -120,7 +120,7 @@ const SCCRepoPicker = ({
         .filter(
           (repo) =>
             selectedRepos.includes(repo.name) &&
-            repo.katello_root_repository_id === null
+            repo.katello_repository_id === null
         )
         .map((repo) => repo.id),
       sccRepos
@@ -128,7 +128,7 @@ const SCCRepoPicker = ({
         .filter(
           (repo) =>
             selectedRepos.includes(repo.name) &&
-            repo.katello_root_repository_id === null
+            repo.katello_repository_id === null
         )
         .map((repo) => repo.name)
     );

data/webpack/components/SCCProductPage/components/SCCProductPicker/components/SCCTreePicker/index.js

@@ -33,11 +33,11 @@ const addParentToTree = (tree, par) => {
 
 const addSCCRepoPickerToTree = (
   tree,
-  disableRepos,
   activateDebugFilter,
   setSelectedReposFromChild
 ) => {
-  tree.customBadgeContent[0] = (
+  tree.customBadgeContent = [];
+  tree.customBadgeContent.push(
     <Tooltip content={__('Filter repositories')}>
       <SCCRepoPicker
         sccRepos={tree.scc_repositories}
@@ -47,7 +47,7 @@ const addSCCRepoPickerToTree = (
         sccProductId={tree.id}
         sccProductName={tree.name}
         setSelectedReposFromChild={setSelectedReposFromChild}
-      />{' '}
+      />
     </Tooltip>
   );
   return tree;
@@ -59,14 +59,8 @@ const setupTreeViewListItem = (
   activateDebugFilter,
   setSelectedReposFromChild
 ) => {
-  tree.customBadgeContent = [];
   addCheckBoxToTree(tree);
-  addSCCRepoPickerToTree(
-    tree,
-    tree.product_id === null,
-    activateDebugFilter,
-    setSelectedReposFromChild
-  );
+  addSCCRepoPickerToTree(tree, activateDebugFilter, setSelectedReposFromChild);
   if ('children' in tree) {
     tree.children = tree.children.map((p) =>
       setupTreeViewListItem(
@@ -90,7 +84,6 @@ const checkAllParents = (
     tree.checkProps.checked = true;
     addSCCRepoPickerToTree(
       tree,
-      false,
       activateDebugFilter,
       setSelectedReposFromChild
     );
@@ -114,7 +107,6 @@ const uncheckAllChildren = (
     tree.checkProps.checked = false;
     addSCCRepoPickerToTree(
       tree,
-      true,
       activateDebugFilter,
       setSelectedReposFromChild
     );

data/webpack/components/SCCProductPage/components/SCCProductView/components/SCCRepoView/index.js

@@ -16,7 +16,7 @@ import './styles.scss';
 
 const createKatelloRepoLink = (repo, sccProductId) => {
   const url = foremanUrl(
-    `/products/${sccProductId}/repositories/${repo.katello_root_repository_id}`
+    `/products/${sccProductId}/repositories/${repo.katello_repository_id}`
   );
   return (
     <Tooltip content={__('Go to Repository page')}>
@@ -35,7 +35,7 @@ const createRepoDropDownItem = (repo, sccProductId) => (
     component="button"
     icon={
       repo.subscription_valid ? (
-        repo.katello_root_repository_id !== null ? (
+        repo.katello_repository_id !== null ? (
           <Icon name="check" type="fa" />
         ) : (
           <Tooltip content={__('Repository not imported')}>
@@ -49,7 +49,7 @@ const createRepoDropDownItem = (repo, sccProductId) => (
       )
     }
   >
-    {repo.katello_root_repository_id !== null
+    {repo.katello_repository_id !== null
       ? createKatelloRepoLink(repo, sccProductId)
       : repo.name}
   </DropdownItem>
@@ -88,7 +88,7 @@ const SCCRepoView = ({ sccRepos, sccProductId }) => {
         >
           {sprintf(
             __('Repositories (%s/%s)'),
-            sccRepos.filter((r) => r.katello_root_repository_id !== null)
+            sccRepos.filter((r) => r.katello_repository_id !== null)
               .length,
             sccRepos.length
           )}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_scc_manager
 version: !ruby/object:Gem::Version
-  version: 2.2.3
+  version: 3.0.0
 platform: ruby
 authors:
 - ATIX AG
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-16 00:00:00.000000000 Z
+date: 2024-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -31,33 +31,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 6.3.1
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: theforeman-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.89.0
+        version: 0.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.89.0
-- !ruby/object:Gem::Dependency
-  name: rubocop-rails
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.8.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 2.8.1
+        version: 0.1.0
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -97,6 +83,13 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- app/assets/javascripts/foreman_scc_manager/locale/de/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/el/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/en/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/fr/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/ja/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/ka/foreman_scc_manager.js
+- app/assets/javascripts/foreman_scc_manager/locale/zh_CN/foreman_scc_manager.js
 - app/assets/javascripts/foreman_scc_manager/scc_accounts.js.coffee
 - app/controllers/api/v2/scc_accounts_controller.rb
 - app/controllers/api/v2/scc_products_controller.rb
@@ -166,7 +159,6 @@ files:
 - lib/tasks/setup_authentication_token.rake
 - lib/tasks/test.rake
 - locale/Makefile
-- locale/action_names.rb
 - locale/de/LC_MESSAGES/foreman_scc_manager.mo
 - locale/de/foreman_scc_manager.po
 - locale/el/LC_MESSAGES/foreman_scc_manager.mo
@@ -174,6 +166,8 @@ files:
 - locale/en/LC_MESSAGES/foreman_scc_manager.mo
 - locale/en/foreman_scc_manager.po
 - locale/foreman_scc_manager.pot
+- locale/fr/LC_MESSAGES/foreman_scc_manager.mo
+- locale/fr/foreman_scc_manager.po
 - locale/gemspec.rb
 - locale/ja/LC_MESSAGES/foreman_scc_manager.mo
 - locale/ja/foreman_scc_manager.po