merb 0.4.2 → 0.5.0

data/spec/merb/core_ext/inflector_spec.rb

@@ -0,0 +1,34 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe Inflector do
+  it "should transform words from singular to plural" do
+    "post".pluralize.should == "posts"
+    "octopus".pluralize.should =="octopi"
+    "the blue mailman".pluralize.should == "the blue mailmen"
+    "CamelOctopus".pluralize.should == "CamelOctopi"
+  end
+  
+  it "should transform words from plural to singular" do
+    "posts".singularize.should == "post"
+    "octopi".singularize.should == "octopus"
+    "the blue mailmen".singularize.should == "the blue mailman"
+    "CamelOctopi".singularize.should == "CamelOctopus"
+  end
+  
+  it "should transform class names to table names" do
+    "RawScaledScorer".tableize.should == "raw_scaled_scorers"
+    "egg_and_ham".tableize.should == "egg_and_hams"
+    "fancyCategory".tableize.should == "fancy_categories"
+  end
+  
+  it "should tranform table names to class names" do
+    "egg_and_hams".classify.should == "EggAndHam"
+    "post".classify.should == "Post"
+  end
+  
+  it "should create a foreign key name from a class name" do
+    "Message".foreign_key.should == "message_id"
+    "Message".foreign_key(false).should == "messageid"
+    "Admin::Post".foreign_key.should == "post_id"
+  end
+end

data/spec/merb/core_ext/kernel_spec.rb

@@ -0,0 +1,25 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe "extracting options from arguments" do
+  def the_method(*args)
+    [extract_options_from_args!(args), args]
+  end
+  
+  it "should extract the hash if the last item is a hash" do
+    opts,args = the_method(:one, :two, :key => :value)
+    opts.should == { :key => :value }
+    args.should == [:one, :two]
+  end
+  
+  it "should return nil for the opts if no hash is provided" do
+    opts,args = the_method(:one, :two)
+    opts.should be_nil
+    args.should == [:one, :two]
+  end
+  
+  it "should return two hashes" do
+    opts,args = the_method({ :one => :two }, { :key => :value })
+    opts.should == {:key => :value}
+    args.should == [{:one => :two}]
+  end
+end

data/spec/merb/core_ext/numeric_spec.rb

@@ -0,0 +1,26 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe "A Numeric object" do
+  it "should be able to convert to US currency" do
+    1.5.to_currency.should == "$1.50"
+  end
+  
+  it "should be able to convert to Danish currency" do
+    15_000_000.5.to_currency(nil, ".", ",", "DM").should == "15.000.000,50DM"
+  end
+  
+  {
+    :microsecond => Float(10 ** -6), :millisecond => Float(10 ** -3), :second => 1,
+    :minute => 60, :hour => 3600, :day => 86400, :week => 604800,
+    :month => 2592000, :year => 31536000, :decade => 315360000
+  }.each do |method,seconds|
+    
+    it "should be able to convert to #{method}s (singular version)" do
+      1.send(method).should == seconds
+    end
+    
+    it "should be able to convert to #{method}s (plural version)" do
+      2.send("#{method}s").should == seconds * 2
+    end
+  end
+end

data/spec/merb/core_ext/object_spec.rb

@@ -0,0 +1,47 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+class String
+  def self.define_meta(meth, val)
+    meta_def(meth) { val }
+  end
+end
+class MyString < String; end
+
+describe "An object" do
+  it "should be able to return a passed in object that is modified by a block" do
+    (returning({}) {|x| x.merge!(:foo => :bar)}).should == {:foo => :bar}
+  end
+  
+  it "should be able to get a meta class" do
+    MyString.meta_class.to_s.should == "#<Class:MyString>"
+  end
+  
+  it "should be able to execute code in the meta-class' context" do
+    (MyString.meta_eval { self }).should == MyString.meta_class
+  end
+  
+  it "should be able to define a method on the meta-class" do
+    MyString.define_meta :foo, :bar
+    MyString.foo.should == :bar
+  end
+  
+  it "should be able to define methods on its instances" do
+    MyString.class_def(:foo) { :bar }
+    MyString.new.foo.should == :bar
+  end
+  
+  {[] => true,
+   [1] => false,
+   [nil] => false,
+   nil => true,
+   true => false,
+   false => true,
+   "" => true,
+   "   " => true,
+   "  hey " => false
+  }.each do |obj, expected|
+    it "should be able to determine whether the #{obj.class} #{obj.inspect} is blank" do
+      obj.blank?.should == expected
+    end
+  end
+end

data/spec/merb/core_ext/string_spec.rb

@@ -0,0 +1,22 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe "A String" do
+  it "should convert a path/like/this to a Constant::String::Like::This" do
+    "path/like/this".to_const_string.should == "Path::Like::This"
+    "path".to_const_string.should == "Path"
+    "snake_case/path/with_several_parts".to_const_string.should == "SnakeCase::Path::WithSeveralParts"
+  end
+  
+  it "should raise an error rather than freeze when trying to convert bad Paths/12Like/-this" do
+    Timeout::timeout(1) do
+      lambda do
+        "Paths/12Like/-this".to_const_string
+      end.should raise_error(String::InvalidPathConversion)
+    end.should_not raise_error(Timeout::Error)
+  end
+
+  it "should remove any indentation and add +indentation+ number of spaces" do
+    "foo\n  bar\n".indent(3).should == "   foo\n     bar\n"
+    "  foo\n    bar\n".indent(3).should == "   foo\n     bar\n"
+  end
+end

data/spec/merb/core_ext/symbol_spec.rb

@@ -0,0 +1,7 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe Symbol do
+  it "should be able to call Symbol#to_proc" do
+    ['foo', 'bar'].map(&:reverse).should == ['oof', 'rab']
+  end
+end

data/spec/merb/dependency_spec.rb

@@ -0,0 +1,22 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe "Kernel#dependency" do
+    
+  before do
+    File.stub!(:directory?).with(Merb.root / "gems").and_return(true)
+    Gem.stub!(:use_paths)
+    Gem.stub!(:clear_paths)
+  end
+    
+  it "loads in files from the local gem-cache first" do
+    Gem.should_receive(:activate).with("json_pure", true).and_return(true)
+    Kernel.dependency "json_pure"
+  end
+  
+  it "does a require if it can't find it in either gem cache" do
+    Gem.stub!(:activate).twice.with("RedCloth", true).and_raise(LoadError)
+    Kernel.should_receive(:require).with("RedCloth")
+    Kernel.dependency "RedCloth"
+  end
+  
+end

data/spec/merb/dispatch_spec.rb

@@ -6,22 +6,25 @@ $TESTING = true
 describe Merb::Dispatcher do
 
   before(:all) do
-    Merb::Server.config[:allow_reloading] = false
+    Merb::Config[:allow_reloading] = false
     Merb::Router.prepare do |r|
       r.resource :icon
-      r.resources :posts, :member => {:stats => [:get, :put]}, 
-                          :collection => {:filter => [:get]} do |post|
-        post.resources :comments,  :member => {:stats => [:get, :put]}
+      r.resources :posts,
+        :member => { :stats => [:get, :put] },
+        :collection => { :filter => [:get] } do |post|
+        post.resources :comments,  :member => { :stats => [:get, :put] }
         post.resource :profile
-      end  
+      end
       r.resources :as do |a|
         a.resources :bs do |b|
           b.resources :cs
         end  
       end
-      r.match("/admin") do |admin|
+      
+      r.match('/admin') do |admin|
         admin.resources :tags
       end
+      
       r.default_routes
     end
   end
@@ -31,12 +34,12 @@ describe Merb::Dispatcher do
     controller.class.should == Foo
     action.should == "bar"
     controller.params[:id].should == '42'
-  end  
+  end
   
   it "should not allow private and protected methods to be called" do
     controller, action = request(:get, '/foo/call_filters')
     controller.status.should == Merb::ControllerExceptions::ActionNotFound::STATUS
-  end  
+  end
   
   it "should handle request: GET /foo/bar and return Foo#bar" do
     controller, action = request(:get, '/foo/bar')
@@ -148,7 +151,7 @@ describe Merb::Dispatcher do
     controller.params[:format].should == 'xml'
     controller.body.should == :index
   end
-
+  
   it "should handle request: GET /posts and return Posts#index" do
     controller, action = request(:get, '/posts')
     controller.class.should == Posts
@@ -493,7 +496,7 @@ describe Merb::Dispatcher do
     controller, action = request(:get, '/pos-ts/1')
     controller.status.should == Merb::ControllerExceptions::NotFound::STATUS
   end
-
+  
   if defined?(ParseTreeArray)
     it "should support parameterized actions with required arguments" do
       controller, action = request(:get, '/bar/foo/1')
@@ -514,7 +517,15 @@ describe Merb::Dispatcher do
       controller, action = request(:get, '/bar/baz?a=1&c=5')
       controller.body.should == "1 2 5"
     end
+    
+    it "should support actionargs with false as defaults" do
+      controller, action = request(:get, '/bar/bat?a=1')
+      controller.body.should == "1 false"
+    end
+    
+    it "should support actionargs with nil as defaults" do
+      controller, action = request(:get, '/bar/bam?a=1')
+      controller.body.should == "1 nil"
+    end    
   end
-
-
 end

data/spec/merb/fake_request_spec.rb

@@ -69,4 +69,12 @@ describe Merb::Test::FakeRequest, "[]=(key, value)" do
     @mock['HTTP_ACCEPT'] = 'also/credit_card'
     @mock['HTTP_ACCEPT'].should == 'also/credit_card'     
   end
+end
+
+describe Merb::Test::FakeRequest, "session" do
+  it "should have a memory-based session" do
+    @mock = Merb::Test::FakeRequest.new
+    @mock.session[:random] = "random"
+    @mock.session[:random].should == "random"
+  end
 end

data/spec/merb/generator_spec.rb

@@ -1,13 +1,23 @@
 require File.dirname(__FILE__) + '/../spec_helper'
 require File.dirname(__FILE__) + '/../spec_generator_helper'
 
-module Kernel
-  undef dependency
-end
 
 describe "an app generator" do
   include RubiGen::GeneratorTestHelper
-  
+
+  before :all do
+    module Kernel
+      alias_method :old_dependency, :dependency
+      undef dependency
+    end  
+  end
+
+  after :all do
+    module Kernel
+      alias_method :dependency, :old_dependency
+    end  
+  end
+
   before do
     @generator =  build_generator('merb', [APP_ROOT], sources, {})
   end
@@ -97,33 +107,142 @@ describe "an app generator" do
       script/destroy
     }.each{|file| file_should_be_executable(file)}
   end
-    
-  
   
+  def sources
+    [RubiGen::PathSource.new(:test, File.join(File.dirname(__FILE__),"../../", generator_path))
+    ]
+  end
   
+  def generator_path
+    "app_generators"
+  end
   
-  def directory_should_be_created(directory)
-    File.should be_exist(File.join(APP_ROOT, directory))
-    File.should be_directory(File.join(APP_ROOT, directory))
+end
+
+describe "a controller generator" do
+  include RubiGen::GeneratorTestHelper
+
+  before :all do
+    module Kernel
+      alias_method :old_dependency, :dependency
+      undef dependency
+    end  
   end
 
-  def file_should_be_created(file)
-    File.should be_exist(File.join(APP_ROOT, file))
-    File.should be_file(File.join(APP_ROOT, file))
+  after :all do
+    module Kernel
+      alias_method :dependency, :old_dependency
+    end  
   end
 
-  def file_should_be_executable(file)
-    File.should be_executable(File.join(APP_ROOT, file))
+  before do
+    @generator =  build_generator('controller', ["my_posts"], sources, {})
   end
-    
-  
+
+  after do
+    bare_teardown  # Cleans up the temporary application directory that gets created as part of the test.
+  end
+
+  it "should be get created" do
+    @generator.should_not be_nil
+  end
+
+  it "should be a ControllerGenerator" do
+    @generator.should be_an_instance_of(ControllerGenerator)
+  end
+
+  it "should create directory structure" do
+    silence_generator do
+      @generator.command(:create).invoke!
+    end
+    %w{
+      app
+      app/controllers
+      app/helpers
+      app/views
+      app/views/my_posts
+    }.each{|dir| directory_should_be_created(dir)}    
+  end
+
+  it "should create files from templates" do
+    silence_generator do
+      @generator.command(:create).invoke!
+    end
+    %w{
+      app/controllers/my_posts.rb
+      app/helpers/my_posts_helper.rb
+      app/views/my_posts/index.html.erb
+    }.each{|file| file_should_be_created(file)}
+  end
+
   def sources
-    [RubiGen::PathSource.new(:test, File.join(File.dirname(__FILE__),"../../", generator_path))
+    [
+      RubiGen::PathSource.new(:app, File.join(File.dirname(__FILE__),"../../", "merb_generators")),
+      RubiGen::PathSource.new(:app, File.join(File.dirname(__FILE__),"../../", "rspec_generators"))
     ]
   end
-  
-  def generator_path
-    "app_generators"
+end
+
+describe "a part_controller generator" do
+  include RubiGen::GeneratorTestHelper
+
+  before :all do
+    module Kernel
+      alias_method :old_dependency, :dependency
+      undef dependency
+    end  
+  end
+
+  after :all do
+    module Kernel
+      alias_method :dependency, :old_dependency
+    end  
+  end
+
+  before do
+    @generator =  build_generator('part_controller', ["my_posts"], sources, {})
+  end
+
+  after do
+    bare_teardown  # Cleans up the temporary application directory that gets created as part of the test.
+  end
+
+  it "should be get created" do
+    @generator.should_not be_nil
+  end
+
+  it "should be a PartControllerGenerator" do
+    @generator.should be_an_instance_of(PartControllerGenerator)
+  end
+
+  it "should create directory structure" do
+    silence_generator do
+      @generator.command(:create).invoke!
+    end
+    %w{
+      app
+      app/parts/controllers
+      app/parts/helpers
+      app/parts/views
+      app/parts/views/my_posts_part
+    }.each{|dir| directory_should_be_created(dir)}    
+  end
+
+  it "should create files from templates" do
+    silence_generator do
+      @generator.command(:create).invoke!
+    end
+    %w{
+      app/parts/controllers/my_posts_part.rb
+      app/parts/helpers/my_posts_part_helper.rb
+      app/parts/views/my_posts_part/index.html.erb
+    }.each{|file| file_should_be_created(file)}
+  end
+
+  def sources
+    [
+      RubiGen::PathSource.new(:app, File.join(File.dirname(__FILE__),"../../", "merb_generators")),
+      RubiGen::PathSource.new(:app, File.join(File.dirname(__FILE__),"../../", "rspec_generators"))
+    ]
   end
-  
 end