hammer_cli_foreman_webhooks 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 39d7e941934be4b7423126f92dce297e653866a903ea936e6b9551293c0881c0
-  data.tar.gz: 5e39bdea6ab727a502fb9e2a71d5e4382ad420e9c51f2fcabff193a7582f4996
+  metadata.gz: f3349e5ec869114bfd9b702fa6f1e43204ec39b63d47dd0a2557fdc071fab3e1
+  data.tar.gz: 00424fc9386abcd61e9aee533b73b2dbce63dfafaf46cc5f59303e826b75455a
 SHA512:
-  metadata.gz: fa5c5cf321e38a1e22046b194c65a47c1ec522de0f7fb86632aff62fd16704eccd0ab256fd0dc650fe4a5294d6d6ae70d18c7591bd300326118cbb9ea7b35eb9
-  data.tar.gz: 810fdf342cceb67558f09ff437e6f1a5819e6c798d485a9344c33c4169b7d881d619961eb4be8377452edb488267e6435d4c803a2cbe8465ec781d364601576b
+  metadata.gz: 81d6d4db570f2e88a9706cf79a579488b86d690848dacc52101ec3c45fd487b2643879bc2bb9e558f63764ecf7b31f9810111912668ec14f37b1bf6267c5bdda
+  data.tar.gz: cdc9dcf879643d4d4aefbd7673393f8e9bda5d0e2035be834604bb6e408715f34772b90f2fea5b914b4d5a94a0f48e2e9b5edb6da2621e14bc67f5861a12c672

data/lib/hammer_cli_foreman_webhooks/version.rb

@@ -2,6 +2,6 @@
 
 module HammerCLIForemanWebhooks
   def self.version
-    @version ||= Gem::Version.new '0.0.3'
+    @version ||= Gem::Version.new '0.0.4'
   end
 end

data/lib/hammer_cli_foreman_webhooks/webhook.rb

@@ -21,11 +21,11 @@ module HammerCLIForemanWebhooks
         field :http_method, _('HTTP Method')
         field :http_content_type, _('HTTP Content Type')
         custom_field Fields::Reference, label: _('Webhook Template'), path: [:webhook_template]
-        field :user, _('User'), Fields::Field, sets: %w[ADDITIONAL ALL]
-        field :verify_sll, _('Verify SSL'), Fields::Boolean, sets: %w[ADDITIONAL ALL]
-        field :proxy_authorization, _('Proxy Authorization'), Fields::Boolean, sets: %w[ADDITIONAL ALL]
+        field :user, _('User'), Fields::Field
+        field :verify_ssl, _('Verify SSL'), Fields::Boolean
+        field :proxy_authorization, _('Proxy Authorization'), Fields::Boolean
         field :ssl_ca_certs, _('X509 Certification Authorities'), Fields::Text, sets: %w[ADDITIONAL ALL]
-        collection :http_headers, _('HTTP Headers'), sets: %w[ADDITIONAL ALL] do
+        collection :http_headers, _('HTTP Headers') do
           custom_field Fields::KeyValue
         end
         HammerCLIForeman::References.timestamps(self)
@@ -42,7 +42,7 @@ module HammerCLIForemanWebhooks
 
       build_options without: %i[ssl_ca_certs http_headers]
 
-      extend_with(HammerCLIForemanWebhooks::CommandExtensions::Webhook.new(only: :options))
+      extend_with(HammerCLIForemanWebhooks::CommandExtensions::Webhook.new(only: %i[option request_params]))
     end
 
     class UpdateCommand < HammerCLIForeman::UpdateCommand
@@ -51,7 +51,7 @@ module HammerCLIForemanWebhooks
 
       build_options without: %i[ssl_ca_certs http_headers]
 
-      extend_with(HammerCLIForemanWebhooks::CommandExtensions::Webhook.new(only: :options))
+      extend_with(HammerCLIForemanWebhooks::CommandExtensions::Webhook.new(only: %i[option request_params]))
     end
 
     class DeleteCommand < HammerCLIForeman::DeleteCommand

data/test/functional/webhook_test.rb

@@ -0,0 +1,126 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe 'webhook' do
+  let(:base_cmd) { %w[webhook] }
+  let(:webhook) do
+    {
+      id: 1,
+      name: 'test',
+      target_url: 'https://app.example.com',
+      enabled: true,
+      event: 'user_created.event.foreman',
+      http_method: 'POST',
+      http_content_type: 'application/json',
+      webhook_template: { id: 1, name: 'test' },
+      user: 'admin',
+      verify_ssl: true,
+      proxy_authorization: true,
+      ssl_ca_certs: 'certs',
+      'http_headers' => '{"X-Shellhook-Arg-0":"test","X-Shellhook-Arg-1":"2"}',
+      created_at: '01/10/2022',
+      updated_at: '01/10/2022'
+    }
+  end
+
+  describe 'list' do
+    let(:cmd) { base_cmd << 'list' }
+
+    it 'should list all webhooks' do
+      api_expects(:webhooks, :index, 'List').with_params(
+        'page' => 1, 'per_page' => 1000
+      ).returns(index_response([webhook]))
+
+      output = IndexMatcher.new(
+        [
+          ['ID', 'NAME',  'TARGET URL',              'ENABLED'],
+          ['1',  'test',  'https://app.example.com', 'yes']
+        ]
+      )
+      expected_result = success_result(output)
+
+      result = run_cmd(cmd)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'create' do
+    let(:cmd) { base_cmd << 'create' }
+    let(:params) do
+      %w[
+        --name=test --target-url=https://app.example.com --enabled=true
+        --event=user_created.event.foreman
+        --http-headers=X-Shellhook-Arg-0=test,X-Shellhook-Arg-1=2
+      ]
+    end
+
+    it 'should create a webhook' do
+      api_expects(:webhooks, :create).with_params(
+        'webhook' => {
+          'target_url' => 'https://app.example.com', 'name' => 'test',
+          'enabled' => true, 'event' => 'user_created.event.foreman',
+          'http_headers' => '{"X-Shellhook-Arg-0":"test","X-Shellhook-Arg-1":"2"}'
+        }
+      ).returns(webhook)
+
+      expected_result = success_result("Webhook [test] created.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'update' do
+    let(:cmd) { base_cmd << 'update' }
+    let(:params) do
+      %w[
+        --id=1 --target-url=https://app.example.com --enabled=true
+        --event=user_created.event.foreman
+        --http-headers=X-Shellhook-Arg-0=test,X-Shellhook-Arg-1=2
+      ]
+    end
+
+    it 'should update a webhook' do
+      api_expects(:webhooks, :update).with_params(
+        'id' => '1',
+        'webhook' => {
+          'target_url' => 'https://app.example.com',
+          'enabled' => true, 'event' => 'user_created.event.foreman',
+          'http_headers' => '{"X-Shellhook-Arg-0":"test","X-Shellhook-Arg-1":"2"}'
+        }
+      ).returns(webhook)
+
+      expected_result = success_result("Webhook [test] updated.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'info' do
+    let(:cmd) { base_cmd << 'info' }
+    let(:params) { %w[--id=1] }
+
+    it 'should show a webhook' do
+      api_expects(:webhooks, :show, 'Show').with_params('id' => '1').returns(webhook)
+
+      expected_result = success_result(/X-Shellhook-Arg-0/)
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+
+  describe 'delete' do
+    let(:cmd) { base_cmd << 'delete' }
+    let(:params) { %w[--id=1] }
+
+    it 'should delete a webhook' do
+      api_expects(:webhooks, :destroy, 'Delete').with_params('id' => '1').returns(webhook)
+
+      expected_result = success_result("Webhook [test] deleted.\n")
+
+      result = run_cmd(cmd + params)
+      assert_cmd(expected_result, result)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman_webhooks
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Oleh Fedorenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-06-01 00:00:00.000000000 Z
+date: 2022-09-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli_foreman
@@ -49,6 +49,7 @@ files:
 - test/data/3.3/foreman_api.json
 - test/functional/test_helper.rb
 - test/functional/webhook_template_test.rb
+- test/functional/webhook_test.rb
 - test/test_helper.rb
 homepage: https://github.com/theforeman/hammer-cli-foreman-webhooks
 licenses:
@@ -77,4 +78,5 @@ test_files:
 - test/data/3.3/foreman_api.json
 - test/functional/test_helper.rb
 - test/functional/webhook_template_test.rb
+- test/functional/webhook_test.rb
 - test/test_helper.rb