navigatrix 0.0.1 → 0.1.0

data/spec/item_builder_spec.rb

@@ -0,0 +1,120 @@
+require "spec_helper"
+require "navigatrix/rendering/strategies/html_attributes"
+
+module Navigatrix
+  describe ItemBuilder do
+    class SimpleItem
+      def name_or_link
+        "name_or_link"
+      end
+
+      def linked_content
+        "linked_content"
+      end
+
+      def unlinked_content
+        "unlinked_content"
+      end
+
+      def render_children
+        "children"
+      end
+
+      def name
+        "name"
+      end
+
+      def path
+        "/path"
+      end
+
+      def html_attributes
+        Rendering::Strategies::HTMLAttributes.new({class: "class"})
+      end
+
+      def children_options
+        {class: "class"}
+      end
+
+      def render?
+        true
+      end
+    end
+
+    describe ItemBuilder do
+      before { described_class.stub(klass: SimpleItem) }
+
+      describe "#wrapper" do
+        it "wraps rendered items" do
+          renderer = described_class.build do |builder|
+            builder.wrapper do |content, children, html_attributes|
+              "<li class='#{html_attributes[:class]}'>#{content}<div>#{children}</div></li>"
+            end
+          end
+          rendered = renderer.new.render
+
+          rendered.should == "<li class='class'>name_or_link<div>children</div></li>"
+        end
+      end
+
+      describe "#linked" do
+        it "overrides the linked content" do
+          renderer = described_class.build do |builder|
+            builder.linked do |name, path|
+              "<span>#{name}</span>"
+            end
+          end
+
+          renderer.new.linked_content.should == "<span>name</span>"
+        end
+      end
+
+      describe "#unlinked" do
+        it "overrides the unlinked content" do
+          renderer = described_class.build do |builder|
+            builder.linked do |name, path|
+              "<span>#{name}</span>"
+            end
+          end
+
+          renderer.new.linked_content.should == "<span>name</span>"
+        end
+      end
+
+      describe "#children_options" do
+        it "merges options" do
+          renderer = described_class.build do |builder|
+            builder.children_options do |defaults|
+              defaults.merge({
+                item: {
+                  renderer: "child"
+                }
+              })
+            end
+          end
+
+          renderer.new.children_options.should == {
+            class: "class",
+            item: {
+              renderer: "child"
+            }
+          }
+        end
+      end
+
+      describe "#html_attributes" do
+        it "merges the options" do
+          renderer = described_class.build do |builder|
+            builder.html_attributes do |defaults|
+              {class: "hide"}
+            end
+          end
+
+          renderer.new.html_attributes.should == {
+            "class" => "class hide"
+          }
+        end
+      end
+    end
+  end
+end

data/spec/item_spec.rb

@@ -1,6 +1,4 @@
-require File.expand_path("../../lib/navigatrix/item", __FILE__)
-require File.expand_path("../../lib/navigatrix/item_collection", __FILE__)
-require File.expand_path("../../lib/navigatrix/configuration", __FILE__)
+require "spec_helper"
 
 module Navigatrix
   describe Item do

data/spec/list_builder_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+
+module Navigatrix
+  describe ListBuilder do
+    class SimpleList
+      def render
+      end
+
+      def render_items
+        "children"
+      end
+
+      def html_attributes
+        "class='class'"
+      end
+    end
+
+    before { ListBuilder.stub(klass: SimpleList) }
+
+    describe "#wrapper" do
+      it "wraps rendered items" do
+        renderer = ListBuilder.build do |builder|
+          builder.wrapper do |content, html_attributes|
+            "<li #{html_attributes}>#{content}</li>"
+          end
+        end
+
+        renderer.new.render.should == "<li class='class'>children</li>"
+      end
+    end
+  end
+end

data/spec/renderer_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("../../lib/navigatrix/renderer", __FILE__)
+require "spec_helper"
 
 module Navigatrix
   describe Renderer do
@@ -22,8 +22,6 @@ module Navigatrix
         end
 
         def rendered_strategy?(strategy_klass)
-          puts klass
-          puts strategy_klass
           rendered? and klass == strategy_klass
         end
       end
@@ -33,13 +31,22 @@ module Navigatrix
       let(:item_collection)   { double(:items => double) }
 
       before do
+        Rendering::StrategyFactory.stub(:find_list_strategy)
+          .with("list")
+          .and_return(StrategyDouble)
+
         wrapped_context = double
-        Rendering::Context.stub(:new).with(rendering_context).and_return(wrapped_context)
-        ItemCollection.stub(:new).with(config, wrapped_context).and_return(item_collection)
+        Rendering::Context.stub(:new)
+          .with(rendering_context)
+          .and_return(wrapped_context)
+
+        ItemCollection.stub(:new)
+          .with(config, wrapped_context)
+          .and_return(item_collection)
       end
 
-      context "a specified strategy" do
-        subject { render(:strategy => StrategyDouble) }
+      describe "#render" do
+        subject { render }
 
         it "renders via the strategy" do
           subject.should be_rendered
@@ -50,24 +57,20 @@ module Navigatrix
         end
 
         it "passes the rendering options" do
-          subject.options.should == {:render_option  => "an option"}
+          subject.options.should == {
+            :renderer      =>"list",
+            :render_option => "an option"
+          }
         end
       end
 
-      it "finds strategies by name" do
-        stub_const("Navigatrix::Renderer::REGISTERED_STRATEGIES", {:bootstrap => StrategyDouble})
-        render({:strategy => :bootstrap}).should be_rendered
-      end
-
-      it "defaults to the unordered_list strategy" do
-        stub_const("Navigatrix::Renderer::REGISTERED_STRATEGIES", {:unordered_list => StrategyDouble})
-        render({:strategy => nil}).should be_rendered
-      end
-
-      def render(options)
+      def render(options = {})
         defaults = {
           :render_context => rendering_context,
-          :render_option  => "an option"
+          :list => {
+            :renderer      => "list",
+            :render_option => "an option"
+          }
         }
 
         Renderer.new(config, defaults.merge(options)).render

data/spec/{strategies → rendering/strategies}/bootstrap/navbar_spec.rb

@@ -1,6 +1,4 @@
-require "capybara"
-require File.expand_path("../../../support/list_rendering_strategy", __FILE__)
-require File.expand_path("../../../../lib/navigatrix/rendering/strategies/bootstrap/navbar", __FILE__)
+require "spec_helper"
 
 module Navigatrix::Rendering::Strategies
   module Bootstrap
@@ -17,7 +15,7 @@ module Navigatrix::Rendering::Strategies
 
       describe "a list item" do
         it "does not have a class of 'dropdown'" do
-          rendered_item["class"].should_not include("dropdown")
+          rendered_item["class"].should be_nil
         end
 
         context "with children" do

data/spec/{strategies → rendering/strategies}/bootstrap/tabs_spec.rb

@@ -1,6 +1,4 @@
-require "capybara"
-require File.expand_path("../../../support/list_rendering_strategy", __FILE__)
-require File.expand_path("../../../../lib/navigatrix/rendering/strategies/bootstrap/tabs", __FILE__)
+require "spec_helper"
 
 module Navigatrix::Rendering::Strategies
   module Bootstrap

data/spec/rendering/strategies/list_spec.rb

@@ -0,0 +1,13 @@
+require "spec_helper"
+
+module Navigatrix
+  module Rendering
+    module Strategies
+      describe List do
+        include ListRenderingStrategy
+
+        it_should_behave_like "a list rendering strategy"
+      end
+    end
+  end
+end

data/spec/rendering/strategy_factory_spec.rb

@@ -0,0 +1,58 @@
+module Navigatrix
+  module Rendering
+    describe StrategyFactory do
+      describe ".find_list_strategy" do
+        let(:bootstrap_strategy) { Class.new }
+        let(:default_strategy)   { double }
+        let(:strategies) do
+          {
+            :unordered_list => default_strategy,
+            :bootstrap      => bootstrap_strategy
+          }
+        end
+
+        it "returns a strategy class" do
+          strategy = described_class.find_list_strategy(bootstrap_strategy, strategies)
+          strategy.should == bootstrap_strategy
+        end
+
+        it "finds strategies by name" do
+          strategy = described_class.find_list_strategy(:bootstrap, strategies)
+          strategy.should == bootstrap_strategy
+        end
+
+        it "defaults to the unordered_list strategy" do
+          strategy = described_class.find_list_strategy(nil, strategies)
+          strategy.should == default_strategy
+        end
+      end
+
+      describe ".find_item_strategy" do
+        let(:bootstrap_strategy) { Class.new }
+        let(:default_strategy)   { double }
+        let(:strategies) do
+          {
+            :item      => default_strategy,
+            :bootstrap => bootstrap_strategy
+          }
+        end
+
+        it "returns a strategy class" do
+          strategy = described_class.find_item_strategy(bootstrap_strategy, strategies)
+          strategy.should == bootstrap_strategy
+        end
+
+        it "finds strategies by name" do
+          strategy = described_class.find_item_strategy(:bootstrap, strategies)
+          strategy.should == bootstrap_strategy
+        end
+
+        it "defaults to the unordered_list strategy" do
+          strategy = described_class.find_item_strategy(nil, strategies)
+          strategy.should == default_strategy
+        end
+      end
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -0,0 +1,5 @@
+require "navigatrix"
+require "capybara"
+
+Dir["./spec/support/**/*.rb"].each { |f| require f }
+

data/spec/support/list_rendering_strategy.rb

@@ -3,12 +3,12 @@ module ListRenderingStrategy
     base.let(:item) { build_item }
   end
 
-  def rendered_item(options = {})
-    rendered(options).find("wrapper > ul > li")
+  def rendered_item(*args)
+    rendered(*args).find("wrapper > ul > li")
   end
 
-  def rendered(options = {})
-    Capybara.string("<wrapper>" + described_class.new([item], options).render + "</wrapper>")
+  def rendered(list_options = {}, item_options = {})
+    Capybara.string("<wrapper>" + described_class.new([item], list_options, item_options).render + "</wrapper>")
   end
 
   def build_item(options = {})
@@ -17,6 +17,8 @@ module ListRenderingStrategy
       :render?         => true,
       :linked?         => false,
       :active?         => false,
+      :active_class    => nil,
+      :inactive_class  => nil,
       :html_attributes => {},
       :has_children?   => false
     }
@@ -55,6 +57,10 @@ module ListRenderingStrategy
         rendered.find("li")["id"].should include("item-1")
       end
 
+      it "it does include blank HTML attributes" do
+        rendered.find("li")["class"].should be_nil
+      end
+
       context "the item is linked" do
         before do
           item.stub(:linked? => true, :path => "/path")
@@ -79,7 +85,7 @@ module ListRenderingStrategy
         end
 
         it "allows the active class to be specified" do
-          rendered_item(:active_class => "alt-active")["class"].should include("alt-active")
+          rendered_item({}, {:active_class => "alt-active"})["class"].should include("alt-active")
         end
 
         it "does not overwrite item attributes" do

metadata

@@ -1,55 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: navigatrix
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - Ben Eddy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-22 00:00:00.000000000 Z
+date: 2014-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
@@ -70,14 +70,14 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
@@ -98,14 +98,14 @@ dependencies:
   name: capybara
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: Navigation Generation
@@ -122,6 +122,7 @@ files:
 - README.md
 - Rakefile
 - lib/navigatrix.rb
+- lib/navigatrix/builder.rb
 - lib/navigatrix/configuration.rb
 - lib/navigatrix/integration/rails.rb
 - lib/navigatrix/integration/sinatra.rb
@@ -131,15 +132,24 @@ files:
 - lib/navigatrix/rendering/context.rb
 - lib/navigatrix/rendering/strategies/bootstrap/navbar.rb
 - lib/navigatrix/rendering/strategies/bootstrap/tabs.rb
+- lib/navigatrix/rendering/strategies/content_helpers.rb
+- lib/navigatrix/rendering/strategies/html_attributes.rb
+- lib/navigatrix/rendering/strategies/item.rb
 - lib/navigatrix/rendering/strategies/list.rb
+- lib/navigatrix/rendering/strategy_factory.rb
 - lib/navigatrix/version.rb
 - lib/navigatrix/view_helpers.rb
 - navigatrix.gemspec
+- spec/integration/documentation_spec.rb
+- spec/item_builder_spec.rb
 - spec/item_spec.rb
+- spec/list_builder_spec.rb
 - spec/renderer_spec.rb
-- spec/strategies/bootstrap/navbar_spec.rb
-- spec/strategies/bootstrap/tabs_spec.rb
-- spec/strategies/list_spec.rb
+- spec/rendering/strategies/bootstrap/navbar_spec.rb
+- spec/rendering/strategies/bootstrap/tabs_spec.rb
+- spec/rendering/strategies/list_spec.rb
+- spec/rendering/strategy_factory_spec.rb
+- spec/spec_helper.rb
 - spec/support/list_rendering_strategy.rb
 homepage: https://github.com/foraker/navigatrix
 licenses:
@@ -151,24 +161,30 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.1
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: Navigation generation for Rails and Sinatra
 test_files:
+- spec/integration/documentation_spec.rb
+- spec/item_builder_spec.rb
 - spec/item_spec.rb
+- spec/list_builder_spec.rb
 - spec/renderer_spec.rb
-- spec/strategies/bootstrap/navbar_spec.rb
-- spec/strategies/bootstrap/tabs_spec.rb
-- spec/strategies/list_spec.rb
+- spec/rendering/strategies/bootstrap/navbar_spec.rb
+- spec/rendering/strategies/bootstrap/tabs_spec.rb
+- spec/rendering/strategies/list_spec.rb
+- spec/rendering/strategy_factory_spec.rb
+- spec/spec_helper.rb
 - spec/support/list_rendering_strategy.rb
+has_rdoc: