active-component 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b475c4c94a10c124c9e7f1abde4e94457bd676ff
-  data.tar.gz: 50687bf5b33b5d32d9d940014ff8410cf004ebcf
+  metadata.gz: a25b946f3c2dcac2bc08f225ee0750fc92b8dd3b
+  data.tar.gz: 4bd337f140175fa144da9646015be48c5bb5981a
 SHA512:
-  metadata.gz: 6d0c880b34ddcc2c3c400b77da985d125fc1347f7167408cd2a420eee984df6ebbdebad3c16bbeeb265d8f9139874bb36d2583734bd960e65f96c54cf272d530
-  data.tar.gz: fc09071da743121546495a67b5e638e452b78191cb60e41f4330f7950d1652a760f66bf9299707e1f6e56da34d435b32b13d774755bd22c16909cbc7c607a136
+  metadata.gz: 05d28e9c891720503d97aa596afb2bd37716e4e79f7d8b0368cec63e6d42b218dd9ad8f1cda7e5c1feab50cdd0d36ea390f4cfd4d727f3ab719fa49108d780e8
+  data.tar.gz: 0c60c0052b1ce195d4165272b899f7722d4fb42bae956905d8eb37f76a61d5da1df1d9bd6747a2588a01e7b977f758087fd31cfc3bb197fc98bc73f4fcc42120

data/lib/active_component/base.rb

@@ -35,17 +35,17 @@ module ActiveComponent
     # self.controller from anywhere in the inherited classes.
     #
     def initialize(args = {})
-      if args.kind_of? Hash
-        assign_variables args
-        @controller = ActiveComponent.get_controller rescue nil
-      else
-        raise ArgumentError, "Expected: Hash. Received: #{args.class.name}"
-      end
+      assign_variables args
+      @controller = ActiveComponent.get_controller
     end
 
     def assign_variables(args)
-      args.each do |key, value|
-        self.send("#{key}=", value)
+      if args.kind_of? Hash
+        args.each do |key, value|
+          self.send("#{key}=", value)
+        end
+      else
+        raise ArgumentError, "Expected: Hash. Received: #{args.class.name}"
       end
     end
 

data/lib/active_component/renderable.rb

@@ -23,7 +23,7 @@ module ActiveComponent
       unless args[:collection].present?
         template(type, args)
       else
-        collection(type, args[:collection], args[:as])
+        collection(type, args[:collection], args[:as], args[:locals])
       end
     end
 
@@ -34,13 +34,21 @@ module ActiveComponent
       resource.render
     end
 
-    def collection(type, collection, as)
+    def collection(type, collection, as, locals)
+
       error_string = "Attempting to render a collection without specifying the ':as' value"
       raise ArgumentError, error_string if as.nil?
 
-      collection.map do |item|
-        template(type, as => item)
-      end.join('')
+      locals = locals || {}
+
+      html_string = collection.map do |item|
+        template(
+          type,
+          locals.merge(as => item)
+        )
+      end
+
+      html_string.join('')
     end
 
     def constantize(type)

data/lib/active_component/version.rb

@@ -1,3 +1,3 @@
 module ActiveComponent
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/test/active_component/base_test.rb

@@ -33,4 +33,29 @@ class ActiveComponent::BaseTest < ActiveSupport::TestCase
       @foo.render_component(:foo, :bar => "bar")
     end
   end
+
+  #
+  # Unit tests
+  # 
+
+  test "only accepts hashes" do
+    assert_raise ArgumentError do
+      @foo.assign_variables('a')
+    end
+  end
+
+  test "assigns from the given hash to the instance variables" do
+    @foo.assign_variables(:bar => "foo")
+    assert_equal @foo.bar, "foo"
+  end
+
+  test "has a controller method" do
+    FooComponent.any_instance.stubs(:controller).returns(true)
+    assert @foo.controller
+  end
+
+  test "has a render method" do
+    assert @foo.render
+  end
+
 end

data/test/active_component/context_test.rb

@@ -21,4 +21,14 @@ class ActiveComponent::ContextTest < ActiveSupport::TestCase
     controller.get_context
     assert_equal controller, ActiveComponent.controllers[Thread.current.object_id]
   end
+
+  test "includes the before filter into action controller" do
+    assert_equal :get_context, ActionController::Base.new._process_action_callbacks.first.filter
+  end
+
+  test "gets the context for a given contrlller" do
+    assert_nothing_raised Exception do
+      ActionController::Base.new.get_context
+    end
+  end
 end

data/test/active_component/railtie_test.rb

@@ -1,15 +1,4 @@
 require 'test_helper'
 
 class ActiveComponent::RailtieTest < ActiveSupport::TestCase
-
-  test "has before filter" do
-    assert_equal :get_context, ActionController::Base.new._process_action_callbacks.first.filter
-  end
-
-  test "has the context method" do
-    assert_nothing_raised Exception do
-      ActionController::Base.new.get_context
-    end
-  end
-
 end

data/test/active_component/renderable_test.rb

@@ -3,8 +3,8 @@ require 'test_helper'
 class ActiveComponent::RenderableTest < ActiveSupport::TestCase
 
   setup do
-    FooComponent.any_instance.stubs(:render).returns("<div>hello!</div>")
     @view = ActionView::Base.new
+    @collection = ["hello!", "world!"]
   end
 
   test "renders a component" do
@@ -24,8 +24,8 @@ class ActiveComponent::RenderableTest < ActiveSupport::TestCase
   end
 
   test "it renders multiple components if receives a collection array" do
-    string = @view.render_component(:foo, :collection => [1,2], :as => :bar)
-    assert_equal "<div>hello!</div><div>hello!</div>", string
+    string = @view.render_component(:foo, :collection => @collection, :as => :bar)
+    assert_equal "<div>hello!</div><div>world!</div>", string
   end
 
   test "it raises an error if the given arguments didn't include the :as argument" do
@@ -34,4 +34,38 @@ class ActiveComponent::RenderableTest < ActiveSupport::TestCase
     end
   end
 
+  test "it accepts locals for a given collection" do
+    string = @view.render_component(
+      :foo,
+      :collection => @collection,
+      :as => :bar,
+      :locals => { :baz => "baz" }
+    )
+    assert_match /baz/, string
+  end
+
+  # 
+  # Private methods
+  #
+
+  test "it returns a string of multiple elements" do
+    string = @view.send(:collection, :foo, @collection, :bar, nil)
+    assert_equal "<div>hello!</div><div>world!</div>", string
+  end
+
+  test "it returns a string from the template" do
+    string = @view.send(:template, :foo, :bar => "Hello world!")
+    assert_equal "<div>Hello world!</div>", string
+  end
+
+  test "it turns a string into a constant" do
+    assert_equal FooComponent, @view.send(:constantize, "foo_component")
+  end
+
+  test "it raises an error" do
+    assert_raise ArgumentError do
+      @view.send(:raise_errors, :foo, {})
+    end
+  end
+
 end

data/test/support/foo_component.rb

@@ -1,3 +1,7 @@
 class FooComponent < ActiveComponent::Base
-  attr_accessor :bar
+  attr_accessor :bar, :baz
+
+  def render
+    "<div>#{bar}</div>#{baz}"
+  end
 end

data/test/test_helper.rb

@@ -6,6 +6,9 @@ require "rails/test_help"
 require 'faker'
 require 'pry'
 require 'mocha/setup'
+require 'coveralls'
+
+Coveralls.wear!
 
 Rails.backtrace_cleaner.remove_silencers!
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active-component
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Antonio Chavez
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-08 00:00:00.000000000 Z
+date: 2013-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -95,6 +95,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  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: A View-Component framework for Rails applications.
 email:
 - cavjzz@gmail.com