puppetdb_foreman 2.0.0 → 3.0.0

data/test/static_fixtures/query_nodes.json

@@ -0,0 +1,61 @@
+[ {
+  "name" : "server1.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:07:27.114Z",
+  "facts_timestamp" : "2017-04-20T15:06:52.738Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server2.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:06:28.282Z",
+  "facts_timestamp" : "2017-04-20T15:06:12.253Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server3.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:12:56.090Z",
+  "facts_timestamp" : "2017-04-20T15:12:32.117Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server4.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:04:01.154Z",
+  "facts_timestamp" : "2017-04-20T15:03:38.150Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server5.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:20:06.853Z",
+  "facts_timestamp" : "2017-04-20T15:19:49.088Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server6.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:21:11.520Z",
+  "facts_timestamp" : "2017-04-20T15:20:57.685Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server7.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:15:37.033Z",
+  "facts_timestamp" : "2017-04-20T15:15:13.918Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server8.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:19:18.236Z",
+  "facts_timestamp" : "2017-04-20T15:19:02.659Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server9.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T14:39:35.070Z",
+  "facts_timestamp" : "2017-04-20T14:39:19.003Z",
+  "report_timestamp" : null
+}, {
+  "name" : "server10.example.com",
+  "deactivated" : null,
+  "catalog_timestamp" : "2017-04-20T15:17:06.209Z",
+  "facts_timestamp" : "2017-04-20T15:16:50.007Z",
+  "report_timestamp" : null
+} ]

data/test/test_plugin_helper.rb

@@ -0,0 +1,27 @@
+# This calls the main test_helper in Foreman-core
+require 'test_helper'
+require 'database_cleaner'
+require 'webmock/minitest'
+
+# Foreman's setup doesn't handle cleaning up for Minitest::Spec
+DatabaseCleaner.strategy = :transaction
+
+def setup_settings
+  Setting::Puppetdb.load_defaults
+end
+
+def fixture(name)
+  File.read(File.expand_path("../static_fixtures/#{name}", __FILE__))
+end
+
+module Minitest
+  class Spec
+    before :each do
+      DatabaseCleaner.start
+    end
+
+    after :each do
+      DatabaseCleaner.clean
+    end
+  end
+end

data/test/unit/puppetdb_host_test.rb

@@ -0,0 +1,43 @@
+require 'test_plugin_helper'
+
+class PuppetdbHostTest < ActiveSupport::TestCase
+  setup do
+    User.current = FactoryGirl.create(:user, :admin)
+    disable_orchestration
+    setup_settings
+  end
+
+  let(:sample_facts) do
+    JSON.parse(fixture('facts.json'))
+  end
+
+  let(:pdbhost) do
+    PuppetdbHost.new(:facts => sample_facts)
+  end
+
+  test 'parses facts' do
+    assert_kind_of HashWithIndifferentAccess, pdbhost.facts
+    assert_equal 'host.example.com', pdbhost.certname
+    assert_equal 'CEST', pdbhost.facts[:timezone]
+    assert_nil pdbhost.environment
+  end
+
+  test 'creates a new host by facts' do
+    host = pdbhost.to_host
+    assert_equal 'host.example.com', host.name
+    assert_equal '1.1.1.174', host.ip
+    assert_equal Operatingsystem.find_by(title: 'RedHat 7.3'), host.operatingsystem
+    assert_equal Domain.find_by(name: 'example.com'), host.domain
+  end
+
+  test 'updates an existing host by facts' do
+    Setting[:update_subnets_from_facts] = true
+    FactoryGirl.create(:host, :managed, :hostname => 'host', :domain => FactoryGirl.create(:domain, :name => 'example.com'))
+    pdbhost.to_host
+    host = Host.find_by(name: 'host.example.com')
+    assert_equal 'host.example.com', host.name
+    # foreman core does not delete old interfaces when importing interfaces from facts
+    assert host.interfaces.where(:ip => '1.1.1.174').first
+    assert_equal Operatingsystem.find_by(title: 'RedHat 7.3'), host.operatingsystem
+  end
+end

data/test/unit/puppetdb_test.rb

@@ -0,0 +1,86 @@
+require 'test_plugin_helper'
+
+class PuppetdbTest < ActiveSupport::TestCase
+  setup do
+    User.current = FactoryGirl.build(:user, :admin)
+    setup_settings
+    disable_orchestration
+  end
+
+  let(:client) { Puppetdb.client }
+  let(:uuid) { SecureRandom.uuid }
+
+  context 'with V1 API' do
+    setup do
+      Setting[:puppetdb_address] = 'https://localhost:8080/v3/commands'
+    end
+
+    test 'deactivate_node' do
+      stub_request(:post, 'https://localhost:8080/v3/commands')
+        .with(:body => 'payload={"command":"deactivate node","version":1,"payload":"\\"www.example.com\\""}',
+              :headers => { 'Accept' => 'application/json' })
+        .to_return(:status => 200, :body => "{\"uuid\" : \"#{uuid}\"}", :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+
+      assert_equal uuid, client.deactivate_node('www.example.com')
+    end
+
+    test 'query_nodes' do
+      stub_request(:get, 'https://localhost:8080/v3/nodes')
+        .with(:headers => { 'Accept' => 'application/json' })
+        .to_return(:status => 200, :body => fixture('query_nodes.json'), :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+      expected = (1..10).map { |i| "server#{i}.example.com" }
+      assert_equal expected, client.query_nodes
+    end
+
+    test 'facts' do
+      stub_request(:get, 'https://localhost:8080/v3/facts?query=%5B%22=%22,%20%22certname%22,%20%22host.example.com%22%5D')
+        .with(:headers => { 'Accept' => 'application/json' })
+        .to_return(:status => 200, :body => fixture('facts.json'), :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+      facts = client.facts('host.example.com')
+      assert_kind_of Array, facts
+      sample = {
+        'value' => 'CEST',
+        'name' => 'timezone',
+        'certname' => 'host.example.com'
+      }
+      assert_includes facts, sample
+    end
+  end
+
+  context 'with V3 API' do
+    let(:producer_timestamp) { Time.now.iso8601.to_s }
+
+    test 'deactivate_node' do
+      client.stubs(:producer_timestamp).returns(producer_timestamp)
+
+      stub_request(:post, 'https://puppetdb:8081/pdb/cmd/v1')
+        .with(:body => "{\"command\":\"deactivate node\",\"version\":3,\"payload\":{\"certname\":\"www.example.com\",\"producer_timestamp\":\"#{producer_timestamp}\"}}",
+              :headers => { 'Accept' => 'application/json', 'Content-Type' => 'application/json' })
+        .to_return(:status => 200, :body => "{\"uuid\" : \"#{uuid}\"}", :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+
+      assert_equal uuid, client.deactivate_node('www.example.com')
+    end
+
+    test 'query_nodes' do
+      stub_request(:get, 'https://puppetdb:8081/pdb/query/v4/nodes')
+        .with(:headers => { 'Accept' => 'application/json' })
+        .to_return(:status => 200, :body => fixture('query_nodes.json'), :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+      expected = (1..10).map { |i| "server#{i}.example.com" }
+      assert_equal expected, client.query_nodes
+    end
+
+    test 'facts' do
+      stub_request(:get, 'https://puppetdb:8081/pdb/query/v4/facts?query=%5B%22=%22,%20%22certname%22,%20%22host.example.com%22%5D')
+        .with(:headers => { 'Accept' => 'application/json' })
+        .to_return(:status => 200, :body => fixture('facts.json'), :headers => { 'Content-Type' => 'application/json; charset=utf-8' })
+      facts = client.facts('host.example.com')
+      assert_kind_of Array, facts
+      sample = {
+        'value' => 'CEST',
+        'name' => 'timezone',
+        'certname' => 'host.example.com'
+      }
+      assert_includes facts, sample
+    end
+  end
+end

metadata

@@ -1,15 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: puppetdb_foreman
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Daniel Lobato Garcia
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-25 00:00:00.000000000 Z
-dependencies: []
+date: 2017-04-25 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rdoc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 'Disable hosts on PuppetDB after they are deleted or built in Foreman,
   and proxy the PuppetDB dashboard to Foreman. Follow https://github.com/theforeman/puppetdb_foreman
   and raise an issue/submit a pull request if you need extra functionality. You can
@@ -19,17 +75,41 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- LICENSE
+- README.md
+- Rakefile
+- app/controllers/api/v2/puppetdb_nodes_controller.rb
+- app/controllers/puppetdb_foreman/nodes_controller.rb
 - app/controllers/puppetdb_foreman/puppetdb_controller.rb
+- app/models/concerns/orchestration/puppetdb.rb
 - app/models/puppetdb_foreman/host_extensions.rb
 - app/models/setting/puppetdb.rb
+- app/services/puppetdb.rb
+- app/services/puppetdb_client/base.rb
+- app/services/puppetdb_client/v1.rb
+- app/services/puppetdb_client/v3.rb
+- app/services/puppetdb_host.rb
+- app/views/api/v2/puppetdb_nodes/import.json.rabl
+- app/views/api/v2/puppetdb_nodes/index.json.rabl
+- app/views/api/v2/puppetdb_nodes/unknown.json.rabl
+- app/views/puppetdb_foreman/nodes/index.html.erb
 - app/views/puppetdb_foreman/puppetdb/error.html.erb
 - config/routes.rb
 - lib/puppetdb_foreman.rb
 - lib/puppetdb_foreman/engine.rb
 - lib/puppetdb_foreman/version.rb
+- lib/tasks/puppetdb_foreman_tasks.rake
+- test/controllers/api/v2/puppetdb_nodes_controller_test.rb
+- test/controllers/nodes_controller_test.rb
+- test/models/host_test.rb
+- test/static_fixtures/facts.json
+- test/static_fixtures/query_nodes.json
+- test/test_plugin_helper.rb
+- test/unit/puppetdb_host_test.rb
+- test/unit/puppetdb_test.rb
 homepage: http://www.github.com/theforeman/puppetdb_foreman
 licenses:
-- GPL-3
+- GPL-3.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -47,8 +127,16 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: This is a Foreman plugin to interact with PuppetDB.
-test_files: []
+test_files:
+- test/controllers/api/v2/puppetdb_nodes_controller_test.rb
+- test/controllers/nodes_controller_test.rb
+- test/models/host_test.rb
+- test/static_fixtures/facts.json
+- test/static_fixtures/query_nodes.json
+- test/test_plugin_helper.rb
+- test/unit/puppetdb_host_test.rb
+- test/unit/puppetdb_test.rb