cassandra_migrations 0.0.1.pre0 → 0.0.1.pre1

data/lib/cassandra_migrations/cassandra.rb

@@ -77,9 +77,11 @@ private
   def self.connect_to_server
     load_config
     
+    Rails.logger.try(:info, "Connecting to Cassandra on #{config['host']}:#{config['port']}")
+    
     begin
       self.client = Cql::Client.new(:host => config['host'], :port => config['port'])
-      client.start!
+      client.connect
     rescue Cql::Io::ConnectionError => e
       raise Errors::ConnectionError, e.message      
     end

data/lib/cassandra_migrations/railtie/initializer.rb

@@ -0,0 +1,17 @@
+# encoding : utf-8
+
+# Em produção (como usamos o Passenger), vários processos ruby são criados através do fork
+# do original. No UNIX, quando um fork é realizado, is file descriptors (arquivos, conexões a base de dados,
+# sockets, e etc) são copiados em estado aberto para o processo filho. Se não reabrirmos a conexão com o cassandra
+# no processo filho, quando ela for ser usada ela estará 'locked' pelo processo pai, o que resultará em deadlock. 
+
+# Mais explicações em: http://www.modrails.com/documentation/Users%20guide%20Apache.html#spawning_methods_explained
+if defined?(PhusionPassenger)
+  PhusionPassenger.on_event(:starting_worker_process) do |forked|
+    if forked
+      CassandraMigrations::Cassandra.shutdown!
+    end
+  end
+end
+
+CassandraMigrations::Cassandra.start!

data/lib/cassandra_migrations/railtie.rb

@@ -1,15 +1,14 @@
 # encoding : utf-8
 
-require 'cassandra_migrations/cassandra'
-
 class CassandraMigrations::Railtie < ::Rails::Railtie
   
-  initializer "cassandra_migrations.start" do
-    CassandraMigrations::Cassandra.start!
+  initializer "cassandra_migrations.initializer" do
+    require File.expand_path('railtie/initializer', File.dirname(__FILE__))
   end
 
   rake_tasks do
-    Dir[File.expand_path("tasks/**/*.rake", File.dirname(__FILE__))].each do |file| 
+    Dir[File.expand_path("railtie/**/*.rake", File.dirname(__FILE__))].each do |file| 
+      # 'load' is used because 'require' is only for .rb files
       load file
     end
   end

data/lib/cassandra_migrations.rb

@@ -2,4 +2,5 @@
 module CassandraMigrations
 end
 
+require 'cassandra_migrations/cassandra'
 require 'cassandra_migrations/railtie' if defined?(Rails)

data/spec/cassandra_migrations/cassandra_spec.rb

@@ -0,0 +1,88 @@
+# encoding : utf-8
+require 'spec_helper'
+
+describe CassandraMigrations::Cassandra do
+  
+  it "should initialize without client or config" do
+    CassandraMigrations::Cassandra.client.should be_nil
+    CassandraMigrations::Cassandra.config.should be_nil
+  end
+  
+  before do
+    Rails.stub(:root).and_return Pathname.new("spec/fixtures")
+    Rails.stub(:env).and_return ActiveSupport::StringInquirer.new("development")
+  end
+  
+  after do
+    CassandraMigrations::Cassandra.shutdown!
+  end
+  
+  describe ".start!" do
+    it "should raise exception if config/cassandra.yaml is not found" do
+      Rails.stub(:root).and_return Pathname.new("wrong_path")
+      
+      expect do
+        CassandraMigrations::Cassandra.start!
+      end.to raise_exception CassandraMigrations::Cassandra::Errors::MissingConfigurationError
+    end
+    
+    it "should load config/cassandra.yaml environment specific part in self.config" do
+      begin
+        CassandraMigrations::Cassandra.start!
+      rescue
+      end
+      
+      CassandraMigrations::Cassandra.config.should == {
+        'host' => "127.0.0.1",
+        'port' => 9042,
+        'keyspace' => "cassandra_migrations_development",
+        'replication' => {
+          'class' => "SimpleStrategy",
+          'replication_factor' => 1
+        }
+      }
+    end
+    
+    it "should use host and port configurations to create cassandra client" do
+      cql_client_mock = Cql::Client.new
+      cql_client_mock.should_receive(:start!)
+      Cql::Client.should_receive(:new).with(:host => '127.0.0.1', :port => 9042).and_return(cql_client_mock)
+      
+      begin
+        CassandraMigrations::Cassandra.start!
+      rescue
+      end
+    end
+    
+    it "should raise exception if not able to connect to cassandra host" do
+      cql_client_mock = Cql::Client.new
+      cql_client_mock.stub(:start!).and_raise Cql::Io::ConnectionError
+      Cql::Client.stub(:new).and_return cql_client_mock
+      
+      expect do
+        CassandraMigrations::Cassandra.start!
+      end.to raise_exception CassandraMigrations::Cassandra::Errors::ConnectionError
+    end
+    
+    it "should automatically use configured keyspace" do
+      CassandraMigrations::Cassandra.should_receive(:use).with('cassandra_migrations_development')
+      CassandraMigrations::Cassandra.start!
+    end
+    
+    it "should raise exception if configured keyspace does not exist" do
+      expect do
+        CassandraMigrations::Cassandra.start!
+      end.to raise_exception CassandraMigrations::Cassandra::Errors::UnexistingKeyspaceError
+    end
+  end
+  
+  describe "query interface" do
+    it "should respond to query methods" do
+      CassandraMigrations::Cassandra.should respond_to :select
+      CassandraMigrations::Cassandra.should respond_to :write
+      CassandraMigrations::Cassandra.should respond_to :delete
+      CassandraMigrations::Cassandra.should respond_to :truncate 
+    end
+  end
+  
+end

data/spec/cassandra_migrations_spec.rb

@@ -0,0 +1,13 @@
+# encoding : utf-8
+require 'spec_helper'
+
+describe CassandraMigrations do
+  it "should define modules" do
+    defined?(CassandraMigrations).should be_true
+    defined?(CassandraMigrations::Railtie).should be_true
+    defined?(CassandraMigrations::Cassandra).should be_true
+    defined?(CassandraMigrations::Cassandra::Query).should be_true
+    defined?(CassandraMigrations::Cassandra::Errors).should be_true
+    defined?(CassandraMigrations::Cassandra::Migrator).should be_true
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cassandra_migrations
 version: !ruby/object:Gem::Version
-  version: 0.0.1.pre0
+  version: 0.0.1.pre1
   prerelease: 6
 platform: ruby
 authors:
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0.pre3
+        version: 1.0.0.pre5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.0.0.pre3
+        version: 1.0.0.pre5
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -98,13 +98,16 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/cassandra_migrations/cassandra.rb
-- lib/cassandra_migrations/cassandra/errors.rb
+- lib/cassandra_migrations.rb
 - lib/cassandra_migrations/cassandra/migrator.rb
 - lib/cassandra_migrations/cassandra/query.rb
-- lib/cassandra_migrations/tasks/cassandra.rake
+- lib/cassandra_migrations/cassandra/errors.rb
+- lib/cassandra_migrations/railtie/tasks.rake
+- lib/cassandra_migrations/railtie/initializer.rb
+- lib/cassandra_migrations/cassandra.rb
 - lib/cassandra_migrations/railtie.rb
-- lib/cassandra_migrations.rb
+- spec/cassandra_migrations/cassandra_spec.rb
+- spec/cassandra_migrations_spec.rb
 homepage: https://github.com/hsgubert/cassandra_migrations
 licenses: []
 post_install_message: 
@@ -125,8 +128,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.8.0
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Cassandra schema management for a multi-environment developer.
-test_files: []
+test_files:
+- spec/cassandra_migrations/cassandra_spec.rb
+- spec/cassandra_migrations_spec.rb