tengine_core 0.5.28

data/spec/tengine/core/config/atd_spec.rb

@@ -0,0 +1,62 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+require 'tengine/event'
+
+describe Tengine::Core::Config::Atd do
+
+  describe :[] do
+    it "should convert a Hash to a Tengine::Core::Config::Atd" do
+      converted = Tengine::Core::Config::Atd[{:process => {:daemon => true}}]
+      converted.should be_a(Tengine::Core::Config::Atd)
+      converted[:process][:daemon].should be_true
+    end
+
+    it "should return same Tengine::Core::Config::Atd" do
+      converted = Tengine::Core::Config::Atd.new(:process => {:daemon => true})
+      Tengine::Core::Config::Atd[converted].should == converted
+    end
+  end
+
+  context "デフォルト" do
+    subject do
+      Tengine::Core::Config::Atd.new
+    end
+    its(:db){ should == {
+        'host' => 'localhost',
+        'port' => 27017,
+        'username' => nil,
+        'password' => nil,
+        'database' => 'tengine_production',
+      }}
+
+    it "db" do
+      subject.db.should == {
+        'host' => 'localhost',
+        'port' => 27017,
+        'username' => nil,
+        'password' => nil,
+        'database' => 'tengine_production',
+      }
+    end
+
+    it "process_daemon" do
+      subject[:process][:daemon].should be_false
+    end
+
+    it "heartbeat_atd" do
+      subject[:heartbeat][:atd][:interval].should == 0
+    end
+
+  end
+
+  context "指定した設定ファイルが存在しない場合" do
+    it "例外を生成します" do
+      config_path = File.expand_path("../config_spec/unexist_config.yml", File.dirname(__FILE__))
+      expect{
+        Tengine::Core::Config::Atd.new(:config => config_path)
+      }.to raise_error(Tengine::Core::ConfigError, /No such file or directory - #{config_path}/)
+    end
+  end
+
+end

data/spec/tengine/core/config/core_spec.rb

@@ -0,0 +1,479 @@
+# -*- coding: utf-8 -*-
+require 'spec_helper'
+
+require 'tengine/event'
+
+describe Tengine::Core::Config::Core do
+
+  # log_configのテストは spec/models/tengine/core/config_spec/log_config_spec.rb にあります。
+
+  describe :[] do
+    it "should convert a Hash to a Tengine::Core::Config::Core" do
+      converted = Tengine::Core::Config::Core[{ :process => { :daemon => true}}]
+      converted.should be_a(Tengine::Core::Config::Core)
+      converted[:process][:daemon].should be_true
+    end
+
+    it "should return same Tengine::Core::Config::Core" do
+      converted = Tengine::Core::Config::Core.new(:process => { :daemon => true})
+      Tengine::Core::Config::Core[converted].should == converted
+    end
+  end
+
+  context "デフォルト" do
+    subject do
+      Tengine::Core::Config::Core.new
+    end
+    its(:status_dir){ should == "./tmp/tengined_status" }
+    its(:activation_dir){ should == "./tmp/tengined_activations" }
+
+    its(:confirmation_threshold){ should == Tengine::Event::LEVELS_INV[:info] }
+    its(:heartbeat_enabled?){ should == false }
+
+    its(:db){ should == {
+        'host' => 'localhost',
+        'port' => 27017,
+        'username' => nil,
+        'password' => nil,
+        'database' => 'tengine_production',
+      }}
+
+    it do
+      subject.db.should == {
+        'host' => 'localhost',
+        'port' => 27017,
+        'username' => nil,
+        'password' => nil,
+        'database' => 'tengine_production',
+      }
+    end
+  end
+
+
+  shared_examples_for "ディレクトリもファイルも存在しない場合はエラー" do |path|
+    before do
+      @error_message = "file or directory doesn't exist. #{path}"
+      Dir.should_receive(:exist?).with(path).and_return(false)
+      File.should_receive(:exist?).with(path).and_return(false)
+    end
+    it :dsl_dir_path do
+      expect{ subject.dsl_dir_path }.should raise_error(Tengine::Core::ConfigError, @error_message)
+    end
+    it :dsl_file_paths do
+      expect{ subject.dsl_file_paths }.should raise_error(Tengine::Core::ConfigError, @error_message)
+    end
+    it :dsl_version_path do
+      expect{ subject.dsl_version_path }.should raise_error(Tengine::Core::ConfigError, @error_message)
+    end
+    it :dsl_version do
+      expect{ subject.dsl_version }.should raise_error(Tengine::Core::ConfigError, @error_message)
+    end
+  end
+
+
+
+  context "load_pathに相対パス" do
+    before do
+      # capistranoのデフォルトのデプロイ先を想定しています
+      # see "BACK TO CONFIGURATION" in https://github.com/capistrano/capistrano/wiki/2.x-From-The-Beginning
+      # http://www.slideshare.net/T2J/capistrano-tips-tips
+      Dir.stub!(:pwd).and_return("/u/apps/app1/current")
+    end
+
+    shared_examples_for "相対パス指定時のパスの振る舞い" do
+      it :dsl_dir_path do
+        subject.dsl_dir_path.should == "/u/apps/app1/current/tengine_dsls"
+      end
+
+      it :dsl_version_path do
+        subject.dsl_version_path.should == "/u/apps/app1/current/tengine_dsls/VERSION"
+      end
+
+      it "VERSIONファイルがある場合" do
+        File.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/VERSION").and_return(true)
+        File.should_receive(:read).and_return("TEST20110905164100")
+        subject.dsl_version.should == "TEST20110905164100"
+      end
+
+      it "VERSIONファイルがない場合" do
+        File.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/VERSION").and_return(false)
+        t = Time.local(2011,9,5,17,28,30)
+        Time.stub!(:now).and_return(t)
+        subject.dsl_version.should == "20110905172830"
+      end
+    end
+
+    context "のディレクトリを指定する設定ファイル" do
+      subject do
+        Tengine::Core::Config::Core.new(:config => File.expand_path("../config_spec/config_with_dir_relative_load_path.yml", File.dirname(__FILE__)))
+      end
+      it "should allow to read value by using []" do
+        expected = {
+          # :daemon => true,
+          :activation_timeout => 300,
+          :load_path => "tengine_dsls",
+          :cache_drivers => true,
+          # :pid_dir => "tmp/tengined_pids",
+          :status_dir => "tmp/tengined_status",
+          :activation_dir => "tmp/tengined_activations",
+          # :heartbeat_period => 600,
+          :confirmation_threshold => "warn",
+          :skip_load=>nil,
+          :skip_enablement=>nil,
+          :wait_activation=>nil,
+        }
+        subject[:tengined].to_hash.should == expected
+        subject['tengined'].to_hash.should == expected
+        subject[:process]['daemon'].should == true
+        subject[:process][:daemon].should == true
+        subject[:event_queue][:connection][:host].should == "localhost"
+        subject['event_queue']['connection']['host'].should == "localhost"
+        subject[:event_queue][:queue][:name].should == "tengine_event_queue2"
+        subject['event_queue']['queue']['name'].should == "tengine_event_queue2"
+      end
+      its(:heartbeat_enabled?){ should == true }
+
+      describe :relative_path_from_dsl_dir do
+        it "絶対パスが指定されるとdsl_dir_pathからの相対パスを返します" do
+          Dir.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls").and_return(true)
+          subject.relative_path_from_dsl_dir("/u/apps/app1/shared/config").should == "../../shared/config"
+        end
+
+        it "相対パスが指定されると（計算のしようがないので）そのまま返します" do
+          subject.relative_path_from_dsl_dir("lib/tengine/foo/bar").should == "lib/tengine/foo/bar"
+        end
+      end
+
+      describe :confirmation_threshold do
+        it "--tengined-confirmation-levelで設定した値を数値に変換する" do
+          subject.confirmation_threshold.should == Tengine::Event::LEVELS_INV[:warn]
+        end
+      end
+
+      context "ディレクトリが存在する場合" do
+        before do
+          Dir.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls").and_return(true)
+        end
+
+        it :dsl_file_paths do
+          Dir.should_receive(:glob).
+            with("/u/apps/app1/current/tengine_dsls/**/*.rb").
+            and_return(["/u/apps/app1/current/tengine_dsls/foo/bar.rb"])
+          subject.dsl_file_paths.should == ["/u/apps/app1/current/tengine_dsls/foo/bar.rb"]
+        end
+
+        it_should_behave_like "相対パス指定時のパスの振る舞い"
+      end
+
+      it_should_behave_like "ディレクトリもファイルも存在しない場合はエラー", "/u/apps/app1/current/tengine_dsls"
+    end
+
+    context "のファイルを指定する設定ファイル" do
+      subject do
+        Tengine::Core::Config::Core.new(:config => File.expand_path("../config_spec/config_with_file_relative_load_path.yml", File.dirname(__FILE__)))
+      end
+      it "should allow to read value by using []" do
+        expected = {
+          # :daemon => true,
+          :activation_timeout => 300,
+          :load_path => "tengine_dsls/init.rb",
+          :cache_drivers => true,
+          # :pid_dir => "tmp/tengined_pids",
+          :status_dir => "tmp/tengined_status",
+          :activation_dir => "tmp/tengined_activations",
+          # :heartbeat_period => 600,
+          :confirmation_threshold => "warn",
+          :skip_load=>nil,
+          :skip_enablement=>nil,
+          :wait_activation=>nil,
+        }
+        subject[:tengined].to_hash.should == expected
+        subject['tengined'].to_hash.should == expected
+        subject[:tengined]['load_path'].should == "tengine_dsls/init.rb"
+        subject[:tengined][:load_path].should == "tengine_dsls/init.rb"
+      end
+
+      describe :relative_path_from_dsl_dir do
+        it "絶対パスが指定されるとdsl_dir_pathからの相対パスを返します" do
+          Dir.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/init.rb").and_return(false)
+          File.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/init.rb").and_return(true)
+          subject.relative_path_from_dsl_dir("/u/apps/app1/shared/config").should == "../../shared/config"
+        end
+
+        it "相対パスが指定されると（計算のしようがないので）そのまま返します" do
+          subject.relative_path_from_dsl_dir("lib/tengine/foo/bar").should == "lib/tengine/foo/bar"
+        end
+      end
+
+      context "ファイルが存在する場合" do
+        before do
+          Dir.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/init.rb").and_return(false)
+          File.should_receive(:exist?).with("/u/apps/app1/current/tengine_dsls/init.rb").and_return(true)
+        end
+
+        it :dsl_file_paths do
+          subject.dsl_file_paths.should == ["/u/apps/app1/current/tengine_dsls/init.rb"]
+        end
+
+        it_should_behave_like "相対パス指定時のパスの振る舞い"
+      end
+
+      it_should_behave_like "ディレクトリもファイルも存在しない場合はエラー", "/u/apps/app1/current/tengine_dsls/init.rb"
+    end
+  end
+
+
+  context "load_pathに絶対パス" do
+
+    shared_examples_for "絶対パス指定時のパスの振る舞い" do
+      it :dsl_dir_path do
+        subject.dsl_dir_path.should == "/var/lib/tengine"
+      end
+
+      it :dsl_version_path do
+        subject.dsl_version_path.should == "/var/lib/tengine/VERSION"
+      end
+
+      it "VERSIONファイルがある場合" do
+        File.should_receive(:exist?).with("/var/lib/tengine/VERSION").and_return(true)
+        File.should_receive(:read).and_return("TEST20110905164100")
+        subject.dsl_version.should == "TEST20110905164100"
+      end
+
+      it "VERSIONファイルがない場合" do
+        File.should_receive(:exist?).with("/var/lib/tengine/VERSION").and_return(false)
+        t = Time.local(2011,9,5,17,28,30)
+        Time.stub!(:now).and_return(t)
+        subject.dsl_version.should == "20110905172830"
+      end
+    end
+
+    context "のディレクトリを指定する設定ファイル" do
+      subject do
+        Tengine::Core::Config::Core.new(:config => File.expand_path("../config_spec/config_with_dir_absolute_load_path.yml", File.dirname(__FILE__)))
+      end
+      it "should allow to read value by using []" do
+        expected = {
+          # :daemon => true,
+          :activation_timeout => 300,
+          :load_path => "/var/lib/tengine",
+          :cache_drivers => true,
+          # :pid_dir => "/var/run/tengined_pids",
+          :status_dir => "/var/run/tengined_status",
+          :activation_dir => "/var/run/tengined_activations",
+          # :heartbeat_period => 600,
+          :confirmation_threshold => "warn",
+          :skip_load=>nil,
+          :skip_enablement=>nil,
+          :wait_activation=>nil,
+        }
+        subject[:tengined].to_hash.should == expected
+        subject['tengined'].to_hash.should == expected
+        subject[:process]['daemon'].should == true
+        subject[:process][:daemon].should == true
+        subject[:event_queue][:connection][:host].should == "localhost"
+        subject['event_queue']['connection']['host'].should == "localhost"
+        subject[:event_queue][:queue][:name].should == "tengine_event_queue2"
+        subject['event_queue']['queue']['name'].should == "tengine_event_queue2"
+      end
+      its(:heartbeat_enabled?){ should == true }
+
+      describe :relative_path_from_dsl_dir do
+        it "絶対パスが指定されるとdsl_dir_pathからの相対パスを返します" do
+          Dir.should_receive(:exist?).with("/var/lib/tengine").and_return(true)
+          subject.relative_path_from_dsl_dir("/var/lib/tengine/foo/bar").should == "foo/bar"
+        end
+
+        it "相対パスが指定されると（計算のしようがないので）そのまま返します" do
+          subject.relative_path_from_dsl_dir("lib/tengine/foo/bar").should == "lib/tengine/foo/bar"
+        end
+      end
+
+      describe :confirmation_threshold do
+        it "--tengined-confirmation-levelで設定した値を数値に変換する" do
+          subject.confirmation_threshold.should == Tengine::Event::LEVELS_INV[:warn]
+        end
+      end
+
+      context "ディレクトリが存在する場合" do
+        before do
+          Dir.should_receive(:exist?).with("/var/lib/tengine").and_return(true)
+        end
+
+        it :dsl_file_paths do
+          Dir.should_receive(:glob).
+            with("/var/lib/tengine/**/*.rb").
+            and_return(["/var/lib/tengine/foo/bar.rb"])
+          subject.dsl_file_paths.should == ["/var/lib/tengine/foo/bar.rb"]
+        end
+
+        # C0カバレッジを100%にするために追加しています
+        it "dsl_dir_path and dsl_file_paths" do 
+          Dir.should_receive(:glob).
+            with("/var/lib/tengine/**/*.rb").
+            and_return(["/var/lib/tengine/foo/bar.rb"])
+          subject.dsl_dir_path.should == "/var/lib/tengine"
+          subject.dsl_file_paths.should == ["/var/lib/tengine/foo/bar.rb"]
+        end
+
+        it_should_behave_like "絶対パス指定時のパスの振る舞い"
+      end
+
+      it_should_behave_like "ディレクトリもファイルも存在しない場合はエラー", "/var/lib/tengine"
+    end
+
+    context "のファイルを指定する設定ファイル" do
+      subject do
+        Tengine::Core::Config::Core.new(:config => File.expand_path("../config_spec/config_with_file_absolute_load_path.yml", File.dirname(__FILE__)))
+      end
+      it "should allow to read value by using []" do
+        expected = {
+          # :daemon => true,
+          :activation_timeout => 300,
+          :load_path => "/var/lib/tengine/init.rb",
+          :cache_drivers => true,
+          # :pid_dir => "/var/run/tengined_pids",
+          :status_dir => "/var/run/tengined_status",
+          :activation_dir => "/var/run/tengined_activations",
+          # :heartbeat_period => 600,
+          :confirmation_threshold => "warn",
+          :skip_load=>nil,
+          :skip_enablement=>nil,
+          :wait_activation=>nil,
+        }
+        subject[:tengined].to_hash.should == expected
+        subject['tengined'].to_hash.should == expected
+        subject[:tengined]['load_path'].should == "/var/lib/tengine/init.rb"
+        subject[:tengined][:load_path].should == "/var/lib/tengine/init.rb"
+      end
+
+      describe :relative_path_from_dsl_dir do
+        it "絶対パスが指定されるとdsl_dir_pathからの相対パスを返します" do
+          Dir.should_receive(:exist?).with("/var/lib/tengine/init.rb").and_return(false)
+          File.should_receive(:exist?).with("/var/lib/tengine/init.rb").and_return(true)
+          subject.relative_path_from_dsl_dir("/var/lib/tengine/foo/bar").should == "foo/bar"
+        end
+
+        it "相対パスが指定されると（計算のしようがないので）そのまま返します" do
+          subject.relative_path_from_dsl_dir("lib/tengine/foo/bar").should == "lib/tengine/foo/bar"
+        end
+      end
+
+      context "ファイルが存在する場合" do
+        before do
+          Dir.should_receive(:exist?).with("/var/lib/tengine/init.rb").and_return(false)
+          File.should_receive(:exist?).with("/var/lib/tengine/init.rb").and_return(true)
+        end
+
+        it :dsl_file_paths do
+          subject.dsl_file_paths.should == ["/var/lib/tengine/init.rb"]
+        end
+
+        it_should_behave_like "絶対パス指定時のパスの振る舞い"
+      end
+
+      it_should_behave_like "ディレクトリもファイルも存在しない場合はエラー", "/var/lib/tengine/init.rb"
+    end
+  end
+
+  context "指定した設定ファイルが存在しない場合" do
+    it "例外を生成します" do
+      config_path = File.expand_path("../config_spec/unexist_config.yml", File.dirname(__FILE__))
+      expect{
+        Tengine::Core::Config::Core.new(:config => config_path)
+      }.to raise_error(Tengine::Core::ConfigError, /No such file or directory - #{config_path}/)
+    end
+  end
+
+
+  context "[BUG] tengined起動時に-fオプションで設定ファイルを指定した際に、設定ファイルに記載したdb-portの設定が有効でない" do
+    before do
+      @config_path = File.expand_path("../config_spec/another_port.yml", File.dirname(__FILE__))
+    end
+
+    shared_examples_for "正しく読み込む" do
+      it "DBについて" do
+        @config.should be_a(Tengine::Core::Config::Core) if @config
+        hash = @config || @hash
+        hash[:db]['port'].should == 21039
+        hash[:db]['host'].should == 'localhost'
+        hash[:db]['username'].should == nil
+        hash[:db]['password'].should == nil
+        hash[:db]['database'].should == "tengine_production"
+      end
+    end
+
+    context "バグストーリーに添付された設定ファイルをロード" do
+      # このテストは元々パスしてました
+      before do
+        @config = Tengine::Core::Config::Core.new(:config => @config_path)
+      end
+      it_should_behave_like "正しく読み込む"
+    end
+
+    context "起動コマンドの引数を解釈したConfig" do
+      before do
+        @config = Tengine::Core::Config::Core.parse(["-f", @config_path]) # bin/tenginedではARGVが渡されます
+      end
+      it_should_behave_like "正しく読み込む"
+    end
+
+    context "起動コマンドの引数を解釈したHash" do
+      before do
+        @hash = Tengine::Core::Config::Core.parse_to_hash(["-f", @config_path]) # bin/tenginedではARGVが渡されます
+      end
+      it ":configを除いてスケルトンとほとんど同じ" do
+        @hash[:config].should == @config_path
+        @hash[:config] = nil
+        @hash.should == Tengine::Core::Config::Core.skelton_hash
+      end
+    end
+  end
+
+  context "tenginedの設定ファイルが不正でエラーになった場合原因が分からないので、不正な箇所が解るようにする" do
+    it "erbの式の中に間違ったrubyのコードがある設定ファイル" do
+      path = File.expand_path("../config/syntax_error_in_erb.yml.erb", File.dirname(__FILE__))
+      begin
+        Tengine::Core::Config::Core.parse(["-f", path])
+        fail
+      rescue Tengine::Core::ConfigError => e
+        e.message.should =~ /syntax_error_in_erb.yml.erb:9: syntax error/
+        e.message.should include(path)
+      end
+    end
+
+    it "YAMLとして間違っている設定ファイル" do
+      path = File.expand_path("../config/wrong_yaml.yml.erb", File.dirname(__FILE__))
+      begin
+        Tengine::Core::Config::Core.parse(["-f", path])
+        fail
+      rescue Tengine::Core::ConfigError => e
+        e.message.should =~ /couldn't parse YAML at line \d+ column \d+/
+        e.message.should include(path)
+      end
+    end
+
+    it "項目名が間違っている設定ファイル" do
+      path = File.expand_path("../config/wrong_field_name.yml.erb", File.dirname(__FILE__))
+      begin
+        Tengine::Core::Config::Core.parse(["-f", path])
+        fail
+      rescue Tengine::Core::ConfigError => e
+        e.message.should =~ /undefined method `load_path='/
+        e.message.should include(path)
+      end
+    end
+
+    it "カテゴリ名が間違っている設定ファイル" do
+      path = File.expand_path("../config/wrong_category_name.yml.erb", File.dirname(__FILE__))
+      begin
+        Tengine::Core::Config::Core.parse(["-f", path])
+        fail
+      rescue Tengine::Core::ConfigError => e
+        e.message.should =~ /child not found for \"process_configs\"/
+        e.message.should include(path)
+      end
+    end
+  end
+
+end