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

trellis 0.0.1 → 0.0.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

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