temping 1.2.0 → 1.3.0

data/README.md

@@ -17,17 +17,19 @@ As we're using temporary tables all data will be dropped when the database conne
 
 The basic setup of a model involves calling _create_model_ with a symbol that represents the plural table name of the model you wish to create. By default, this will create a temporary table with an _id_ column.
 
+    require 'temping'
     include Temping
     
-    create_model :dogs
+    create_model :dog
     
     Dog.create => #<Dog id: 1>
     
 Additional database columns can be specified via the _with_columns_ method which uses Rails migration syntax:
 
+    require 'temping'
     include Temping
     
-    create_model :dogs do
+    create_model :dog do
       with_columns do |t|
         t.string :name
         t.integer :age, :weight
@@ -38,9 +40,10 @@ Additional database columns can be specified via the _with_columns_ method which
     
 When a block is passed to _create_model_, it is evaluated in the context of the ActiveRecord class. This means anything you do in an ActiveRecord model can be accomplished in the block including method definitions, validations, module includes, etc.
 
+    require 'temping'
     include Temping
     
-    create_model :dogs do
+    create_model :dog do
       include Duckish
 
       with_columns do |t|

data/VERSION

@@ -1 +1 @@
-1.2.0
+1.3.0

data/lib/temping.rb

@@ -1,16 +1,19 @@
 require 'active_record'
 
 module Temping
-  ModelAlreadyDefined = Class.new(StandardError)
-
   def self.included(base)
-    ActiveRecord::Base.configurations['temping'] = { :adapter  => 'sqlite3', :database => ':memory:' }
-    ActiveRecord::Base.establish_connection 'temping' unless ActiveRecord::Base.connected?
+    unless ActiveRecord::Base.connected?
+      ActiveRecord::Base.configurations['temping'] = { 
+        :adapter  => 'sqlite3', :database => ':memory:'
+      }    
+      ActiveRecord::Base.establish_connection 'temping' 
+    end
   end
   
   def create_model(model_name, &block)    
-    unless eval("defined?(#{model_name.to_s.classify})")
-      factory = ModelFactory.new(model_name, &block)
+    model_class = model_name.to_s.classify
+    unless eval("defined?(#{model_class})")
+      factory = ModelFactory.new(model_class, &block)
       factory.klass
     end
   end
@@ -18,9 +21,9 @@ module Temping
   class ModelFactory
     attr_accessor :klass
     
-    def initialize(model_name, &block)
+    def initialize(model_class, &block)
       @klass = Class.new(ActiveRecord::Base)
-      Object.const_set(model_name.to_s.classify, @klass)
+      Object.const_set(model_class, @klass)
       create_table
       add_methods
       @klass.class_eval(&block) if block_given?
@@ -39,6 +42,10 @@ module Temping
           def with_columns
             connection.change_table(table_name) { |table| yield(table) }
           end
+
+          def table_exists?
+            true
+          end
         end
       end
     

data/spec/temping_spec.rb

@@ -5,12 +5,14 @@ describe Temping do
 
   describe ".create_model" do    
     it "creates and returns an AR::Base-derived model" do
-      posts = create_model :posts
-      posts.ancestors.should include(ActiveRecord::Base)
+      post_class = create_model :post
+      post_class.ancestors.should include(ActiveRecord::Base)
+      post_class.should == Post
+      post_class.table_name.should == "posts"
     end
 
     it "evals all statements passed in through a block" do
-      create_model :votes do
+      create_model :vote do
         with_columns do |table|
           table.string :voter
         end
@@ -21,17 +23,22 @@ describe Temping do
       vote = Vote.new
       vote.should_not be_valid
       
-      vote.voter = "John Pignata"
+      vote.voter = "Foo Bar"
       vote.should be_valid
     end
     
-    it "silently skip initialization if a const is already defined" do
-      lambda { 2.times { create_model :dogs }}.should_not raise_error(Temping::ModelAlreadyDefined)
+    it "silently skips initialization if a const is already defined" do
+      lambda { 2.times { create_model :dog } }.should_not raise_error
+    end
+    
+    it "returns nil if a const is already defined" do
+      create_model :cat
+      create_model(:cat).should be_nil
     end
     
     describe ".with_columns" do
       it "creates columns passed in through a block" do
-        create_model :comments do 
+        create_model :comment do 
           with_columns do |table|
             table.integer :count
             table.string :headline
@@ -47,17 +54,20 @@ describe Temping do
   describe "database agnostism" do
     it "supports Sqlite3" do
       ActiveRecord::Base.establish_connection 'temping'
-      create_model(:oranges).should == Orange
+      create_model(:orange).should == Orange
+      Orange.table_name.should == "oranges"
     end
 
     it "supports MySQL" do
       ActiveRecord::Base.establish_connection 'mysql'
-      create_model(:apples).should == Apple
+      create_model(:apple).should == Apple
+      Apple.table_name.should == "apples"
     end
 
     it "supports PostgreSQL" do
       ActiveRecord::Base.establish_connection 'postgres'
-      create_model(:cherries).should == Cherry
+      create_model(:cherry).should == Cherry
+      Cherry.table_name.should == "cherries"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: temping
 version: !ruby/object:Gem::Version 
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors: 
 - John Pignata
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-12-30 00:00:00 -05:00
+date: 2010-01-03 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency