hammer_cli_katello 0.11.1 → 0.11.2

data/test/functional/repository/info_test.rb

@@ -18,7 +18,7 @@ describe "get repository info" do
     params = ["--id=#{repo_id}"]
 
     ex = api_expects(:repositories, :show, "Get info") do |par|
-      par["id"] == repo_id.to_s
+      par["id"] == repo_id
     end
 
     ex.returns({})

data/test/functional/repository/remove_content_test.rb

@@ -8,7 +8,7 @@ module HammerCLIKatello
 
     it 'allows minimal options' do
       api_expects(:repositories, :remove_content) do |p|
-        p['id'] == '1' && p['ids'] == %w(20 21 22)
+        p['id'] == 1 && p['ids'] == %w(20 21 22)
       end
 
       run_cmd(%w(repository remove-content --id 1 --ids 20,21,22))

data/test/functional/repository/synchronize_test.rb

@@ -25,12 +25,12 @@ describe 'Synchronize a repository' do
     params = ["--id=#{repo_id}"]
 
     ex = api_expects(:repositories, :sync, 'Repository is synced') do |par|
-      par['id'] == repo_id.to_s
+      par['id'] == repo_id
     end
 
     ex.returns(sync_response)
 
-    expect_foreman_task('3')
+    expect_foreman_task(3)
 
     result = run_cmd(@cmd + params)
     assert_equal(result.exit_code, 0)
@@ -49,7 +49,7 @@ describe 'Synchronize a repository' do
 
     ex.returns(sync_response)
 
-    expect_foreman_task('3')
+    expect_foreman_task(3)
 
     result = run_cmd(@cmd + params)
     assert_equal(result.exit_code, 0)

data/test/functional/repository/update_test.rb

@@ -8,7 +8,7 @@ module HammerCLIKatello
 
     it 'allows minimal options' do
       api_expects(:repositories, :update) do |p|
-        p['id'] == '1' && p['name'] == 'rep1'
+        p['id'] == 1 && p['name'] == 'rep1'
       end
 
       run_cmd(%w(repository update --id 1 --new-name rep1))

data/test/functional/repository/upload_test.rb

@@ -30,7 +30,7 @@ describe 'upload repository' do
     params = ["--id=#{repo_id}", "--path=#{path}"]
 
     ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
-      par[:repository_id] == repo_id.to_s
+      par[:repository_id] == repo_id
     end
 
     ex.returns(upload_response)
@@ -42,14 +42,14 @@ describe 'upload repository' do
         :size => 0,
         :checksum => 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855'
       }
-      par[:id] == repo_id.to_s && par[:uploads] == [upload] && par[:sync_capsule] == true &&
+      par[:id] == repo_id && par[:uploads] == [upload] && par[:sync_capsule] == true &&
         par[:publish_repository] == true
     end
 
     ex2.returns("")
 
     ex3 = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
-      par[:id] == upload_id && par[:repository_id] == repo_id.to_s
+      par[:id] == upload_id && par[:repository_id] == repo_id
     end
 
     ex3.returns("")
@@ -105,7 +105,7 @@ describe 'upload repository' do
     params = ["--id=#{repo_id}", "--path={test}.[r{1}]pm"]
 
     ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
-      par[:repository_id] == repo_id.to_s
+      par[:repository_id] == repo_id
     end
 
     ex.returns(upload_response)
@@ -117,14 +117,14 @@ describe 'upload repository' do
         :size => 0,
         :checksum => 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855'
       }
-      par[:id] == repo_id.to_s && par[:uploads] == [upload] && par[:sync_capsule] == true &&
+      par[:id] == repo_id && par[:uploads] == [upload] && par[:sync_capsule] == true &&
         par[:publish_repository] == true
     end
 
     ex2.returns("")
 
     ex3 = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
-      par[:id] == upload_id && par[:repository_id] == repo_id.to_s
+      par[:id] == upload_id && par[:repository_id] == repo_id
     end
 
     ex3.returns("")
@@ -143,7 +143,7 @@ describe 'upload repository' do
     # Begin first upload cycle
 
     ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
-      par[:repository_id] == repo_id.to_s
+      par[:repository_id] == repo_id
     end
 
     ex.returns(upload_response)
@@ -155,14 +155,14 @@ describe 'upload repository' do
         :size => 0,
         :checksum => 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855'
       }
-      par[:id] == repo_id.to_s && par[:uploads] == [upload] && par[:sync_capsule] == false &&
+      par[:id] == repo_id && par[:uploads] == [upload] && par[:sync_capsule] == false &&
         par[:publish_repository] == false
     end
 
     ex.returns("")
 
     ex = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
-      par[:id] == upload_id && par[:repository_id] == repo_id.to_s
+      par[:id] == upload_id && par[:repository_id] == repo_id
     end
 
     ex.returns("")
@@ -170,7 +170,7 @@ describe 'upload repository' do
     # Begin second upload cycle
 
     ex = api_expects(:content_uploads, :create, "Create upload for content") do |par|
-      par[:repository_id] == repo_id.to_s
+      par[:repository_id] == repo_id
     end
 
     ex.returns(upload_response)
@@ -182,14 +182,14 @@ describe 'upload repository' do
         :size => 0,
         :checksum => 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855'
       }
-      par[:id] == repo_id.to_s && par[:uploads] == [upload] && par[:sync_capsule] == true &&
+      par[:id] == repo_id && par[:uploads] == [upload] && par[:sync_capsule] == true &&
         par[:publish_repository] == true
     end
 
     ex.returns("")
 
     ex = api_expects(:content_uploads, :destroy, "Delete the upload") do |par|
-      par[:id] == upload_id && par[:repository_id] == repo_id.to_s
+      par[:id] == upload_id && par[:repository_id] == repo_id
     end
 
     ex.returns("")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_katello
 version: !ruby/object:Gem::Version
-  version: 0.11.1
+  version: 0.11.2
 platform: ruby
 authors:
 - Adam Price
@@ -34,7 +34,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-29 00:00:00.000000000 Z
+date: 2017-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman