RubyGems - trellis - Versions diffs - 0.0.1 → 0.0.2 - Mend

trellis 0.0.1 → 0.0.2

Files changed (205) hide show

data/test/component_spec.rb CHANGED Viewed

@@ -1,88 +1,102 @@
 require File.dirname(__FILE__) + '/spec_helper.rb'
-describe Trellis::Component do
-  before(:each) do
-    #@component = Component.new
+require "rack"
+require 'rack/test'
+require_fixtures 'component_spec_components'
+describe Trellis::Component, " in an application" do
+  include Rack::Test::Methods
+  def app
+    TestComponents::ApplicationWithComponents.new
   end
-  it "should desc" do
-    # TODO
+  it "should return its intended content" do
+    get '/'
+    last_response.body.should eql('<html><body>hello from simple component</body></html>')
+  end
+  it "should render each instance of a component in the template" do
+    get '/counters'
+    counter_one_markup = %[<div id="one">0<a href="/page_with_stateful_component/events/add.counter_one">++</a><a href="/page_with_stateful_component/events/subtract.counter_one">--</a></div>]
+    counter_two_markup = %[<div id="two">0<a href="/page_with_stateful_component/events/add.counter_two">++</a><a href="/page_with_stateful_component/events/subtract.counter_two">--</a></div>]
+    counter_three_markup = %[<div id="three">0<a href="/page_with_stateful_component/events/add.counter_three">++</a><a href="/page_with_stateful_component/events/subtract.counter_three">--</a></div]
+    last_response.body.should include(counter_one_markup)
+    last_response.body.should include(counter_two_markup)
+    last_response.body.should include(counter_three_markup)
+  end
+  it "should provide an instance of each component in the template to the page" do
+    counters_page = TestComponents::PageWithStatefulComponent.new
+    counter_one = counters_page.instance_eval { @counter_one }
+    counter_two = counters_page.instance_eval { @counter_two }
+    counter_three = counters_page.instance_eval { @counter_three }
+    counter_one.should be_an_instance_of(TestComponents::Counter)
+    counter_two.should be_an_instance_of(TestComponents::Counter)
+    counter_three.should be_an_instance_of(TestComponents::Counter)
+  end
+  it "should provide a logger" do
+    counters_page = TestComponents::PageWithStatefulComponent.new
+    counter_one = counters_page.instance_eval { @counter_one }
+    counter_one.should respond_to(:logger)
+  end
+  it "should respond to an event for which it provides an event handler" do
+    env = Hash.new
+    env["rack.session"] = Hash.new
+    get '/page_with_stateful_component/events/add.counter_one', {}, env
+    redirect = last_response.headers['Location']
+    redirect.should eql('/page_with_stateful_component')
+    get redirect, {}, env
+    counter_one_markup = %[<div id="one">1]
+    counter_two_markup = %[<div id="two">0]
+    counter_three_markup = %[<div id="three">0]
+    last_response.body.should include(counter_one_markup)
+    last_response.body.should include(counter_two_markup)
+    last_response.body.should include(counter_three_markup)
+  end
+  it "should be able to provide a style link contribution to the page" do
+    get '/page_with_contributions'
+    last_response.body.should include('<link href="/someplace/my_styles.css" rel="stylesheet" type="text/css" />')
+  end
+  it "should be able to provide a script link contribution to the page" do
+    get '/page_with_contributions'
+    last_response.body.should include('<script src="/someplace/my_script.js" type="text/javascript"></script>')
+  end
+  it "should be able to provide a style contribution to the page" do
+    get '/page_with_contributions'
+    last_response.body.should include('<style type="text/css">html { color:#555555; background-color:#303030; }</style>')
   end
-end
+  it "should be able to provide a style contribution to the page per instance" do
+    get '/page_with_contributions'
+    last_response.body.scan(%[<style type=\"text/css\">/* just a comment */</style>]).should have_exactly(2).matches
+  end
-# module TestTrellis
-#   class TestComponent < Test::Unit::TestCase
-#     def test_class_body
-#       raise NotImplementedError, 'Need to write test_class_body'
-#     end
-#
-#     def test_class_contained_in
-#       raise NotImplementedError, 'Need to write test_class_contained_in'
-#     end
-#
-#     def test_class_depends_on
-#       raise NotImplementedError, 'Need to write test_class_depends_on'
-#     end
-#
-#     def test_class_dom_contribution
-#       raise NotImplementedError, 'Need to write test_class_dom_contribution'
-#     end
-#
-#     def test_class_enhance_page
-#       raise NotImplementedError, 'Need to write test_class_enhance_page'
-#     end
-#
-#     def test_class_field
-#       raise NotImplementedError, 'Need to write test_class_field'
-#     end
-#
-#     def test_class_get_component
-#       raise NotImplementedError, 'Need to write test_class_get_component'
-#     end
-#
-#     def test_class_inherited
-#       raise NotImplementedError, 'Need to write test_class_inherited'
-#     end
-#
-#     def test_class_is_stateful
-#       raise NotImplementedError, 'Need to write test_class_is_stateful'
-#     end
-#
-#     def test_class_page_contribution
-#       raise NotImplementedError, 'Need to write test_class_page_contribution'
-#     end
-#
-#     def test_class_register_with_tag_context
-#       raise NotImplementedError, 'Need to write test_class_register_with_tag_context'
-#     end
-#
-#     def test_class_tag_name
-#       raise NotImplementedError, 'Need to write test_class_tag_name'
-#     end
-#
-#     def test_load_component_session_information
-#       raise NotImplementedError, 'Need to write test_load_component_session_information'
-#     end
-#
-#     def test_logger
-#       raise NotImplementedError, 'Need to write test_logger'
-#     end
-#
-#     def test_logger_equals
-#       raise NotImplementedError, 'Need to write test_logger_equals'
-#     end
-#
-#     def test_page
-#       raise NotImplementedError, 'Need to write test_page'
-#     end
-#
-#     def test_page_equals
-#       raise NotImplementedError, 'Need to write test_page_equals'
-#     end
-#
-#     def test_save_component_session_information
-#       raise NotImplementedError, 'Need to write test_save_component_session_information'
-#     end
-#   end
-# end
+  it "should be able to provide a script contribution per instance and access instance information" do
+    get '/page_with_contributions'
+    last_response.body.should include(%[<script type="text/javascript">alert('hello from one');</script>])
+    last_response.body.should include(%[<script type="text/javascript">alert('hello from two');</script>])
+  end
+  it "should be able to provide a script contribution per class" do
+    get '/page_with_contributions'
+    last_response.body.should include(%[alert('hello just once');])
+  end
+  it "should be able to provide a dom modification block" do
+    get '/page_with_contributions'
+    last_response.body.scan(%[<body class=\"new_class\">]).should have_exactly(1).match
+  end
+end

data/test/core_extensions_spec.rb CHANGED Viewed

@@ -80,7 +80,7 @@ end
 describe Class, " when calling attr_array with default options" do
-  before(:each) do
+  before :each do
     @target = Class.new
     @target.attr_array(:elements)
   end
@@ -95,7 +95,7 @@ end
 describe Class, " when calling attr_array with options {:create_accessor => false}" do
-  before(:each) do
+  before :each do
     @target = Class.new
     @target.attr_array(:elements, :create_accessor => false)
   end
@@ -105,4 +105,36 @@ describe Class, " when calling attr_array with options {:create_accessor => fals
   it "should not create an accessor method for the array" do
     @target.should_not respond_to(:elements)
   end
+end
+describe Class do
+  it "when calling class_attr_accessor should add class attribute accessors for each symbol passed" do
+    class Foo; end
+    Foo.class_attr_accessor(:bar)
+    Foo.should respond_to(:bar)
+    Foo.should respond_to(:bar=)
+  end
+  it "when calling class_attr_reader should add class attribute readers for each symbol passed" do
+    class Bar; end
+    Bar.class_attr_reader(:foo)
+    Bar.should respond_to(:foo)
+    Bar.should_not respond_to(:foo=)
+  end
+  it "when calling class_attr_writer should add class attribute readers for each symbol passed" do
+    class FooBar; end
+    FooBar.class_attr_writer(:bar)
+    FooBar.should_not respond_to(:bar)
+    FooBar.should respond_to(:bar=)
+  end
+  it "when calling instance_attr_accessor should add instance attribute accessors for each symbol passed" do
+    class Snafu; end
+    Snafu.instance_attr_accessor(:bar)
+    instance = Snafu.new
+    instance.should respond_to(:bar)
+    instance.should respond_to(:bar=)
+  end
 end

data/test/default_router_spec.rb ADDED Viewed

@@ -0,0 +1,91 @@
+require File.dirname(__FILE__) + '/spec_helper.rb'
+describe Trellis::DefaultRouter, " when processing a request" do
+  before do
+    @regexp = Trellis::DefaultRouter::ROUTE_REGEX
+    @router = Trellis::DefaultRouter.new
+  end
+  it "should extract the page" do
+    value, source, event, destination = "/some_page".match(@regexp).to_a.reverse
+    value.should be_nil
+    source.should be_nil
+    event.should be_nil
+    destination.should_not be_nil
+    destination.should == "some_page"
+  end
+  it "should extract the page and event" do
+    value, source, event, destination = "/some_page/events/event".match(@regexp).to_a.reverse
+    value.should be_nil
+    source.should be_nil
+    event.should == "event"
+    destination.should_not be_nil
+    destination.should == "some_page"
+  end
+  it "should extract the page, event and source" do
+    value, source, event, destination = "/some_page/events/event.source".match(@regexp).to_a.reverse
+    value.should be_nil
+    source.should == "source"
+    event.should == "event"
+    destination.should_not be_nil
+    destination.should == "some_page"
+  end
+  it "should extract the page, event and source and value" do
+    value, source, event, destination = "/some_page/events/event.source/123".match(@regexp).to_a.reverse
+    value.should == "123"
+    source.should == "source"
+    event.should == "event"
+    destination.should_not be_nil
+    destination.should == "some_page"
+  end
+  it "should extract the page, event and value" do
+    value, source, event, destination = "/some_page/events/event/123".match(@regexp).to_a.reverse
+    value.should == "123"
+    source.should be_nil
+    event.should == "event"
+    destination.should_not be_nil
+    destination.should == "some_page"
+  end
+  it "should build a path given page" do
+    page = TestApp::Home.new
+    path = Trellis::DefaultRouter.to_uri(:page => page)
+    path.should == "/home"
+  end
+  it "should build a path given page and an event" do
+    page = TestApp::Home.new
+    path = Trellis::DefaultRouter.to_uri(:page => page, :event => 'event')
+    path.should == "/home/events/event"
+  end
+  it "should build a path given page, an event and a source" do
+    page = TestApp::Home.new
+    path = Trellis::DefaultRouter.to_uri(:page => page, :event => 'event', :source => 'source')
+    path.should == "/home/events/event.source"
+  end
+  it "should build a path given page, an event, a source and a value" do
+    page = TestApp::Home.new
+    path = Trellis::DefaultRouter.to_uri(:page => page, :event => 'event', :source => 'source', :value => 'value')
+    path.should == "/home/events/event.source/value"
+  end
+  it "should match valid patterns" do
+    page_request = mock('request', :path_info => '/page')
+    page_event_request = mock('request', :path_info => '/page/events/event')
+    page_event_source_request = mock('request', :path_info => '/page/events/event.source')
+    page_event_value_request = mock('request', :path_info => '/page/events/event/value')
+    page_event_source_value_request = mock('request', :path_info => '/page/events/event.source/value')
+    @router.matches?(page_request).should be_true
+    @router.matches?(page_event_request).should be_true
+    @router.matches?(page_event_source_request).should be_true
+    @router.matches?(page_event_value_request).should be_true
+    @router.matches?(page_event_source_value_request).should be_true
+  end
+end

data/test/fixtures/application_spec_applications.rb CHANGED Viewed

@@ -1,10 +1,9 @@
 module TestApp
   class MyApp < Trellis::Application
     home :home
-    pages :other
     map_static ['/images', '/style', '/favicon.ico']
-    map_static ['/yui'], "./js"
+    map_static ['/jquery'], "./js"
   end
   class Home < Trellis::Page
@@ -47,5 +46,94 @@ module TestApp
     end
     template do html { body { text %[<trellis:value name="some_value"/>] }} end
-  end
+  end
+  class RoutedDifferently < Trellis::Page
+    route '/whoa'
+    template do html { body { text %[whoa!] }} end
+  end
+  class RoutedDifferentlyWithAParam < Trellis::Page
+    route '/hello/:name'
+    template do
+      html {
+        body {
+          h2 "Hello"
+          text %[<trellis:value name="name"/>]
+        }
+      }
+    end
+  end
+  class RoutedDifferentlyWithParams < Trellis::Page
+    route '/report/:year/:month/:day'
+    template do
+      html {
+        body {
+          h2 "Report for"
+          text %[<trellis:value name="month"/>]
+          text '/'
+          text %[<trellis:value name="day"/>]
+          text '/'
+          text %[<trellis:value name="year"/>]
+        }
+      }
+    end
+  end
+  class RoutedWithOptionalParams < Trellis::Page
+    route '/foobar/?:foo?/?:bar?'
+    template do
+      html {
+        body {
+          text %[<trellis:value name="foo"/>]
+          text '-'
+          text %[<trellis:value name="bar"/>]
+        }
+      }
+    end
+  end
+  class RoutedWithSingleWildcard < Trellis::Page
+    route '/splat/*'
+    template do
+      html {
+        body {
+          text %[<trellis:value name="splat"/>]
+        }
+      }
+    end
+  end
+  class RoutedWithMultipleWildcards < Trellis::Page
+    route '/splats/*/foo/*/*'
+    template do
+      html {
+        body {
+          text %[<trellis:value name="splat"/>]
+        }
+      }
+    end
+  end
+  class RoutedWithMixedParams < Trellis::Page
+    route '/mixed/:foo/*'
+    template do
+      html {
+        body {
+          text %[<trellis:value name="splat"/>]
+          text '-'
+          text %[<trellis:value name="foo"/>]
+        }
+      }
+    end
+  end
 end

data/test/page_spec.rb CHANGED Viewed

@@ -9,19 +9,21 @@ describe Trellis::Page, " when sending an event to a page" do
     @request = Rack::MockRequest.new(@application)
   end
-  it "if the event handler returns self it should render the receiving page" do
-    response = @request.get("/home.event1")
-    response.body.should == "<html><body><h1>Hello World!</h1></body></html>"
+  it "should redirect to the receiving page if the event handler returns self" do
+    response = @request.get("/home/events/event1")
+    response.status.should be(302)
+    response.headers['Location'].should == '/home'
   end
   it "should return a response as a string if the event handler returns a String" do
-    response = @request.get("/home.event2")
+    response = @request.get("/home/events/event2")
     response.body.should == "just some text"
   end
-  it "should render the injected page as a response if the event handler returns an injected page " do
-    response = @request.get("/home.event3")
-    response.body.should == "<html><body><p>Goodbye Cruel World </p></body></html>"
+  it "should redirect to the injected page as a response if the event handler returns an injected page " do
+    response = @request.get("/home/events/event3")
+    response.status.should be(302)
+    response.headers['Location'].should == '/other'
   end
   it "should invoke the before_load method if provided by the page" do
@@ -35,7 +37,7 @@ describe Trellis::Page, " when sending an event to a page" do
   end
 end
-describe Trellis::Page, " when calling inject_dependent_pages on an instance of child class of Page" do
+describe Trellis::Page, " when calling inject_dependent_pages on an instance of Page" do
   it "should contain instances of the injected pages" do
     homepage = TestApp::Home.new
     homepage.inject_dependent_pages
@@ -44,3 +46,11 @@ describe Trellis::Page, " when calling inject_dependent_pages on an instance of
   end
 end
+describe Trellis::Page, " when created with a custom route" do
+  it "should contain an instance of Router" do
+    router = TestApp::RoutedDifferently.router
+    router.should_not be_nil
+    router.should be_an_instance_of(Trellis::Router)
+  end
+end

data/test/router_spec.rb CHANGED Viewed

@@ -1,52 +1,52 @@
 require File.dirname(__FILE__) + '/spec_helper.rb'
-describe Trellis::DefaultRouter, " when processing a request" do
-  before do
-    @regexp = Trellis::DefaultRouter::ROUTE_REGEX
-  end
-  it "should extract the page " do
-    value, source, event, destination = "/some_page".match(@regexp).to_a.reverse
-    value.should be_nil
-    source.should be_nil
-    event.should be_nil
-    destination.should_not be_nil
-    destination.should == "some_page"
-  end
-  it "should extract the page and event " do
-    value, source, event, destination = "/some_page.event".match(@regexp).to_a.reverse
-    value.should be_nil
-    source.should be_nil
-    event.should == "event"
-    destination.should_not be_nil
-    destination.should == "some_page"
-  end
-  it "should extract the page, event and source " do
-    value, source, event, destination = "/some_page.event_source".match(@regexp).to_a.reverse
-    value.should be_nil
-    source.should == "source"
-    event.should == "event"
-    destination.should_not be_nil
-    destination.should == "some_page"
-  end
-  it "should extract the page, event and source and value" do
-    value, source, event, destination = "/some_page.event_source/123".match(@regexp).to_a.reverse
-    value.should == "123"
-    source.should == "source"
-    event.should == "event"
-    destination.should_not be_nil
-    destination.should == "some_page"
-  end
-  it "should extract the page, event and value" do
-    value, source, event, destination = "/some_page.event/123".match(@regexp).to_a.reverse
-    value.should == "123"
-    source.should be_nil
-    event.should == "event"
-    destination.should_not be_nil
-    destination.should == "some_page"
-  end
+describe Trellis::Router, " when constructed" do
+  it "with a page parameter should route to that page " do
+    @router = Trellis::Router.new(:page => TestApp::Home)
+    @router.route().destination.should == TestApp::Home
+  end
+  it "with a simple path it should match that path" do
+    request = mock('request', :path_info => '/hello')
+    @router = Trellis::Router.new(:path => '/hello')
+    @router.matches?(request).should be_true
+  end
+  it "with a path containing a single named parameter should match valid variations of that path" do
+    request_brian = mock('request', :path_info => '/hello/brian')
+    request_michael = mock('request', :path_info => '/hello/michael')
+    @router = Trellis::Router.new(:path => '/hello/:name')
+    @router.matches?(request_brian).should be_true
+    @router.matches?(request_michael).should be_true
+  end
+  it "with a path containing multiple named parameters should match valid variations of that path" do
+    request_xmas = mock('request', :path_info => '/hello/2009/12/25')
+    request_labor_day = mock('request', :path_info => '/hello/2009/09/07')
+    @router = Trellis::Router.new(:path => '/hello/:year/:month/:day')
+    @router.matches?(request_xmas).should be_true
+    @router.matches?(request_labor_day).should be_true
+  end
+  it "with a path containing a single named parameters should collect the parameter" do
+    @router = Trellis::Router.new(:path => '/hello/:name')
+    @router.keys.should include('name')
+  end
+  it "with a path containing multiple parameters should collect all parameters" do
+    @router = Trellis::Router.new(:path => '/hello/:year/:month/:day')
+    @router.keys.should include('year', 'month', 'day')
+  end
+  it "with a path containing multiple optional parameters should collect all parameters" do
+    @router = Trellis::Router.new(:path => '/?:foo?/?:bar?')
+    @router.keys.should include('foo', 'bar')
+  end
+  it "with a path containing a single wildcard param it should capture the values in splat" do
+    @router = Trellis::Router.new(:path => '/*')
+    @router.keys.should include('splat')
+  end
 end