hieracles 0.2.0 → 0.2.1

data/spec/lib/interpolate_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe Hieracles::Interpolate do
+  include Hieracles::Interpolate
+
+  describe ".parse" do
+    let(:input) { StringIO.new }
+    let(:output) { StringIO.new }
+    before { setio input, output }
+
+    context "when in non-interactive mode" do
+      let(:data) { 'something_%{a_var}' }
+      context "and has valid value" do
+        let(:values) { { a_var: 'value' } }
+        let(:expected) { 'something_value' }
+        it { expect(parse data, values).to eq expected }
+      end
+      context "and has invalid value" do
+        let(:values) { { b_var: 'novalue' } }
+        let(:expected) { 'something_' }
+        it { expect(parse data, values).to eq expected }
+      end
+      context "and has valid value with doublecolumn" do
+        let(:data) { 'something_%{::a_var}' }
+        let(:values) { { a_var: 'value' } }
+        let(:expected) { 'something_value' }
+        it { expect(parse data, values).to eq expected }
+      end
+    end
+    context "when in interactive mode" do
+      let(:data) { 'something_%{a_var}' }
+      context "and has valid value" do
+        let(:values) { { a_var: 'value' } }
+        let(:expected) { 'something_value' }
+        it { expect(parse data, values, true).to eq expected }
+      end
+      context "and has invalid value" do
+        let(:values) { { b_var: 'novalue' } }
+        let(:expected) { 'something_value' }
+        before { allow(input).to receive(:gets).and_return 'value' }
+        it { expect(parse data, values, true).to eq expected }
+      end
+    end
+  end
+
+  describe ".ask_about" do
+  end
+
+end

data/spec/lib/node_spec.rb

@@ -7,7 +7,8 @@ describe Hieracles::Node do
         config: 'spec/files/config.yml',
         hierafile: 'hiera.yaml',
         encpath: 'enc',
-        basepath: 'spec/files'
+        basepath: 'spec/files',
+        usedb: false
       }
     }
 
@@ -193,16 +194,56 @@ describe Hieracles::Node do
       end
 
       describe '.info' do
-        let(:expected) { {
-          classes: ['dev'],
-          fqdn: 'server.example.com',
-          datacenter: 'equinix',
-          country: 'fr',
-          farm: 'dev'
-        } }
-        it { expect(node.info).to eq expected }
+        context "without calling puppetd" do
+          let(:expected) { {
+            classes: ['dev'],
+            fqdn: 'server.example.com',
+            datacenter: 'equinix',
+            country: 'fr',
+            farm: 'dev'
+          } }
+          it { expect(node.info).to eq expected }
+        end
       end
+    end
+  end
 
+  context "with calling puppetd" do
+    let(:options) {
+      { 
+        config: 'spec/files/config_withdb.yml',
+        hierafile: 'hiera.yaml',
+        encpath: 'enc',
+        basepath: 'spec/files'
+      }
+    }
+    describe '.info' do
+      let(:expected) { {
+        classes: ['dev'],
+        fqdn: 'server.example.com',
+        datacenter: 'equinix',
+        country: 'fr',
+        farm: 'dev',
+        catalog_timestamp: '2015-12-01T23:09:41.540Z'
+      } }
+      let(:resp_info) {
+        Hieracles::Puppetdb::Response.new({ catalog_timestamp: '2015-12-01T23:09:41.540Z'}, 1)
+      }
+      let(:resp_facts) {
+        Hieracles::Puppetdb::Response.new([{ 'name' => 'datacenter', 'value' => 'tahiti' }], 1)
+      }
+      let(:node) { Hieracles::Node.new 'server.example.com', options }
+      before {
+        allow_any_instance_of(Hieracles::Puppetdb::Client).
+          to receive(:request).
+          with("nodes/server.example.com").
+          and_return(resp_info)
+        allow_any_instance_of(Hieracles::Puppetdb::Client).
+          to receive(:request).
+          with("nodes/server.example.com/facts").
+          and_return(resp_facts)
+      }
+      it { expect(node.info).to eq expected }
     end
   end
 

data/spec/lib/notification_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Hieracles::Notification do
+
+  describe '.new' do
+    let(:notif) { Hieracles::Notification.new 'a_source', 'a_message'}
+    it { expect(notif.level).to eq 'info' }
+    it { expect(notif.message).to eq 'a_message' }
+    it { expect(notif.source).to eq 'a_source' }
+  end
+
+  describe '.to_hash' do
+    let(:notif) { Hieracles::Notification.new 'a_source', 'a_message'}
+    before {
+      allow(Time).to receive(:new).and_return(Time.new(2015))
+    }
+    let(:expected) { 
+      { 
+        'source' => 'a_source',
+        'level' => 'info',
+        'message' => 'a_message',
+        'timestamp' => Time.new(2015)
+      }
+    }
+    it { expect(notif.to_hash).to eq expected }
+  end
+
+
+end

data/spec/lib/options/hc_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+require 'hieracles/options/hc'
+
+describe Hieracles::Options::Hc do
+  describe '.initialize' do
+    context 'with proper arguments' do
+      let(:array) { ['arg1', 'arg2', '-c', 'path/to/config-file', '-f', 'thatformat'] }
+      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_options) do
+        { config: 'path/to/config-file', format: 'thatformat' }
+      end
+      subject { Hieracles::Options::Hc.new array }
+      it "populates payload" do
+        expect(subject.payload).to eq expected_payload
+      end
+      it 'populates options' do
+        expect(subject.options).to eq expected_options
+      end
+    end
+
+    context 'with proper arguments in random order' do
+      let(:array) { ['-c', 'path/to/config-file', 'arg1', 'arg2', '-f', 'thatformat'] }
+      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_options) do
+        { config: 'path/to/config-file', format: 'thatformat' }
+      end
+      subject { Hieracles::Options::Hc.new array }
+      it "populates payload" do
+        expect(subject.payload).to eq expected_payload
+      end
+      it 'populates options' do
+        expect(subject.options).to eq expected_options
+      end
+    end
+
+    context 'with funnily ordered arguments' do
+      let(:array) { ['arg1', '-u', 'path/to/config-file', 'arg2', '-f', 'thatformat'] }
+      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_options) do
+        { format: 'thatformat' }
+      end
+      subject { Hieracles::Options::Hc.new array }
+      it "populates payload" do
+        expect(subject.payload).to eq expected_payload
+      end
+      it 'populates options' do
+        expect(subject.options).to eq expected_options
+      end
+    end
+
+    context 'with arguments in alternative syntax' do
+      let(:array) { ['arg1', 'arg2', '-config', 'path/to/config-file', '--format', 'thatformat'] }
+      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_options) do
+        { config: 'path/to/config-file', format: 'thatformat' }
+      end
+      subject { Hieracles::Options::Hc.new array }
+      it "populates payload" do
+        expect(subject.payload).to eq expected_payload
+      end
+      it 'populates options' do
+        expect(subject.options).to eq expected_options
+      end
+    end
+
+    context 'with arguments containing boolean element' do
+      let(:array) { ['arg1', 'arg2', '-i', '-format', 'thatformat'] }
+      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_options) do
+        { format: 'thatformat', interactive: true }
+      end
+      subject { Hieracles::Options::Hc.new array }
+      it "populates payload" do
+        expect(subject.payload).to eq expected_payload
+      end
+      it 'populates options' do
+        expect(subject.options).to eq expected_options
+      end
+    end
+
+  end
+end

data/spec/lib/optparse_spec.rb

@@ -6,13 +6,13 @@ describe Hieracles::Optparse do
       let(:array) { ['arg1', 'arg2', '-c', 'path/to/config-file', '-f', 'thatformat'] }
       let(:expected_payload) { ['arg1', 'arg2'] }
       let(:expected_options) do
-        { config: 'path/to/config-file', format: 'thatformat' }
+        { }
       end
       subject { Hieracles::Optparse.new array }
       it "populates payload" do
         expect(subject.payload).to eq expected_payload
       end
-      it 'populates options' do
+      it 'populates no options' do
         expect(subject.options).to eq expected_options
       end
     end
@@ -21,7 +21,7 @@ describe Hieracles::Optparse do
       let(:array) { ['-c', 'path/to/config-file', 'arg1', 'arg2', '-f', 'thatformat'] }
       let(:expected_payload) { ['arg1', 'arg2'] }
       let(:expected_options) do
-        { config: 'path/to/config-file', format: 'thatformat' }
+        { }
       end
       subject { Hieracles::Optparse.new array }
       it "populates payload" do
@@ -36,7 +36,7 @@ describe Hieracles::Optparse do
       let(:array) { ['arg1', '-u', 'path/to/config-file', 'arg2', '-f', 'thatformat'] }
       let(:expected_payload) { ['arg1', 'arg2'] }
       let(:expected_options) do
-        { format: 'thatformat' }
+        { }
       end
       subject { Hieracles::Optparse.new array }
       it "populates payload" do
@@ -51,7 +51,7 @@ describe Hieracles::Optparse do
       let(:array) { ['arg1', 'arg2', '-config', 'path/to/config-file', '--format', 'thatformat'] }
       let(:expected_payload) { ['arg1', 'arg2'] }
       let(:expected_options) do
-        { config: 'path/to/config-file', format: 'thatformat' }
+        { }
       end
       subject { Hieracles::Optparse.new array }
       it "populates payload" do
@@ -65,9 +65,9 @@ describe Hieracles::Optparse do
 
     context 'with arguments containing boolean element' do
       let(:array) { ['arg1', 'arg2', '-i', '-format', 'thatformat'] }
-      let(:expected_payload) { ['arg1', 'arg2'] }
+      let(:expected_payload) { ['arg1', 'arg2', 'thatformat'] }
       let(:expected_options) do
-        { format: 'thatformat', interactive: true }
+        { }
       end
       subject { Hieracles::Optparse.new array }
       it "populates payload" do

data/spec/lib/puppetdb/apierror_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe Hieracles::Puppetdb::APIError do
+
+  describe '.new' do
+    let(:response) { Object.new }
+    let(:error) { Hieracles::Puppetdb::APIError.new response }
+    it { expect(error.response).to eq response }
+  end
+  
+end

data/spec/lib/puppetdb/client_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Hieracles::Puppetdb::Client do
+
+  describe '.new' do
+    context 'when no option is passed' do
+      let(:client) { Hieracles::Puppetdb::Client.new Hash.new }
+      it { expect(client.class.instance_variable_get(:@default_options)[:base_uri]).to eq 'http://localhost:8080/v3' }
+    end
+  end
+  
+  describe '.setup_if_ssl' do
+    let(:basepath) { 'spec/files' }
+    let(:options) {
+      {
+        'usessl' => true,
+        'key' => File.join(basepath, 'ssl', 'key.pem'),
+        'cert' => File.join(basepath, 'ssl', 'cert.crt'),
+        'ca_file' => File.join(basepath, 'ssl', 'ca.crt')
+      }
+    }
+    let(:client) { Hieracles::Puppetdb::Client.new options }
+    it { expect(client.class.instance_variable_get(:@default_options)[:options]).to eq options }
+  end
+
+  describe '.request' do
+    context 'with a GET request' do
+      let(:client) { Hieracles::Puppetdb::Client.new Hash.new }
+      let(:request) { client.request('endpoint', 'get')}
+      let(:response) { Hieracles::Puppetdb::Response.new(Hash.new, 0, Array.new) }
+      before {
+        resp = double
+        allow(resp).
+          to receive(:code).
+          and_return(200)
+        allow(resp).
+          to receive(:parsed_response).
+          and_return('')
+        allow(client).
+          to receive(:get_request).
+          with('endpoint', nil, {}).
+          and_return(resp)
+      }
+      it { expect(request.data).to eq Hash.new }
+      it { expect(request.notifications.count).to eq 1 }
+    end
+  end
+
+  describe '.get_request' do
+    let(:client) { Hieracles::Puppetdb::Client.new Hash.new }
+    context 'without query' do
+      let(:request) { client.request('endpoint', 'get')}
+      before {
+        allow(client.class).
+          to receive(:get).
+          and_return('ha')
+      }
+      let(:get_request) { client.get_request('endpoint', nil, Hash.new) }
+      it { expect(get_request).to eq 'ha' }
+    end
+  end
+
+  
+end

data/spec/lib/puppetdb/fixsslconnectionadapter_spec.rb

@@ -0,0 +1,107 @@
+require 'spec_helper'
+
+describe Hieracles::Puppetdb::FixSSLConnectionAdapter do
+
+  let(:uri) { URI "https://localhost" }
+  let(:options) { {} }
+  let(:adapter) { Hieracles::Puppetdb::FixSSLConnectionAdapter.new uri, options }
+  describe '.attach_ssl_certificates' do
+
+    context "with cert file" do
+      context "not found" do
+        let(:options) {
+          {
+            'cert' => File.expand_path('spec/files/ssl/no-cert.crt'),
+            'key' => File.expand_path('spec/files/ssl/key.pem'),
+            'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+          }
+        }
+        it { expect { adapter.connection }.to raise_error(IOError, /Cert file .*no-cert.crt not found\./) }
+      end
+      context "badly formatted" do
+        let(:options) {
+          {
+            'cert' => File.expand_path('spec/files/ssl/bad-cert.crt'),
+            'key' => File.expand_path('spec/files/ssl/key.pem'),
+            'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+          }
+        }
+        it { expect { adapter.connection }.to raise_error(OpenSSL::X509::CertificateError) }
+      end
+    end
+
+    context "with key file" do
+      context "not found" do
+        let(:options) {
+          {
+            'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+            'key' => File.expand_path('spec/files/ssl/no-key.pem'),
+            'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+          }
+        }
+        it { expect { adapter.connection }.to raise_error(IOError, /Key file .*no-key.pem not found\./) }
+      end
+      context "badly formatted" do
+        let(:options) {
+          {
+            'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+            'key' => File.expand_path('spec/files/ssl/bad-key.pem'),
+            'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+          }
+        }
+        it { expect { adapter.connection }.to raise_error(OpenSSL::PKey::RSAError) }
+      end
+      context "with pasword" do
+        context "correct" do
+          let(:options) {
+            {
+              'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+              'key' => File.expand_path('spec/files/ssl/key-pass.pem'),
+              'key_password' => 'good-toto',
+              'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+            }
+          }
+          it { expect { adapter.connection }.not_to raise_error }
+        end
+        context "incorrect" do
+          let(:options) {
+            {
+              'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+              'key' => File.expand_path('spec/files/ssl/key-pass.pem'),
+              'key_password' => 'bad-toto',
+              'ca_file' => File.expand_path('spec/files/ssl/ca.crt')
+            }
+          }
+          it {expect { adapter.connection }.to raise_error(OpenSSL::PKey::RSAError) }
+        end
+      end
+    end
+
+    context "with CA file" do
+      context "not found" do
+        let(:options) {
+          {
+            'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+            'key' => File.expand_path('spec/files/ssl/key.pem'),
+            'ca_file' => File.expand_path('spec/files/ssl/no-ca.crt')
+          }
+        }
+        it { expect { adapter.connection }.to raise_error(IOError, /CA file .*no-ca.crt not found\./) }
+      end
+    end
+
+    context "all correct" do
+      let(:options) {
+        {
+          'cert' => File.expand_path('spec/files/ssl/cert.crt'),
+          'key' => File.expand_path('spec/files/ssl/key.pem'),
+          'ca_file' => File.expand_path('spec/files/ssl/ca.crt'),
+          'verify_peer' => '1'
+        }
+      }
+      it { expect { adapter.connection }.not_to raise_error }
+    end
+
+  end
+
+end