active-component 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 250d060c002ceec3a69685cf0af4fa2088cdce7c
-  data.tar.gz: 102f562d032f9c6dba3ce9a6f35768bc359eed61
+  metadata.gz: a7de4ea36b77ba8216a9ac0c9788cf5fdc15a64f
+  data.tar.gz: 5eba3cd8f644f1fdd92d2875c6176b895b162a8b
 SHA512:
-  metadata.gz: f730cfc6de3d9dadad424771f751ecc90fd613cdb02c85806772459296881c41025998f04ba4fb7527cc721df1594281316ec7c8618b24f82076742e83ca527d
-  data.tar.gz: bd5b759e2ed8b786386e9d1d8e2eaf6820a71b5b96c2c702eff08c0ea058979d455491888adaa0f2367936cbc882e6271892d0db89a036a225bf6024e37a1953
+  metadata.gz: 4619068c02a72ad5aeee95e4a6c9495fa551b41b5cd19515b798f104f34d64f6c96bbd1926e2fc0683048fde8c4910e29df26557ff4606096f4e01a4fe0b9cab
+  data.tar.gz: 24f96ae408f07f70d7464dd26105a90c300c1852914e47da7226930694f2ae96db0fb005b5c78f0093fb5cd41453dc528b7bb70f629286fc01026700ab3ab71e

data/lib/active_component/base.rb

@@ -51,5 +51,9 @@ module ActiveComponent
     def controller
       @controller
     end
+
+    def render
+      super.html_safe
+    end
   end
 end

data/lib/active_component/context.rb

@@ -1,4 +1,5 @@
 #
+#
 # This is an autoincluded module for ActionController::Base
 # 
 # It will store the controller context for the given request in a
@@ -15,6 +16,15 @@ module ActiveComponent
   @@controllers = {}
 
   class << self
+
+    def controllers
+      if Rails.env.test?
+        @@controllers
+      else
+        puts "WARNING: ActiveComponent#controllers cannot be accessible from outside of the class"
+      end
+    end
+
     def get_controller
       @@controllers[Thread.current.object_id]
     end

data/lib/active_component/railtie.rb

@@ -6,7 +6,6 @@ module ActiveComponent
     initializer "Appends a method to the action controller class" do
       ActiveSupport.on_load(:action_controller) do
         ActionController::Base.send(:include, ActiveComponent::Context)
-        ActionController::Base.send(:include, ActiveComponent::Renderable)
       end
     end
   end

data/lib/active_component/renderable.rb

@@ -1,20 +1,28 @@
 require 'active_support/concern'
 
 # 
-# Work In Progress
+# This provides a clean implementation for rendering your template
+# inside your view.
 #
-# The idea is to provide a clean rails-like methods to
-# render view components nicely from the view or even respond
-# with them from a controller instance
-#
-# render_component(:view_component, var: var)
+# <%= render_component(:view, var: var) %>
 #
 module ActiveComponent
   module Renderable
-    extend ActiveSupport::Concern
 
-    def render_component
-      ""
+    def render_component(type, args={})
+      resource = constantize("#{type.to_s}_component").new(args)
+      resource.render
+    end
+
+    def constantize(type)
+      type = type.camelize.constantize
+      unless type.superclass == ActiveComponent::Base
+        raise TypeError, ":type must be compliant to any existing component class"
+      end
+      type
     end
+
   end
 end
+
+ActionView::Base.send(:include, ActiveComponent::Renderable)

data/lib/active_component/version.rb

@@ -1,3 +1,3 @@
 module ActiveComponent
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/test/active_component/base_test.rb

@@ -4,7 +4,7 @@ class ActiveComponent::BaseTest < ActiveSupport::TestCase
 
   setup do
     FooController.new.get_context
-    @foo = Foo.new(bar: "bar")
+    @foo = FooComponent.new(bar: "bar")
   end
 
   test "has included date helper from action view" do
@@ -12,11 +12,11 @@ class ActiveComponent::BaseTest < ActiveSupport::TestCase
   end
 
   test "a new instance of active_component/base accepts only a hash of arguments" do
-    assert_nothing_raised(ArgumentError, nil) { Foo.new(bar: Faker::Name.name) }
+    assert_nothing_raised(ArgumentError, nil) { FooComponent.new(bar: Faker::Name.name) }
   end
 
   test "a new instance of active component should raise an error if the given value is not a hash" do
-    assert_raise(ArgumentError, nil) { Foo.new(Faker::Lorem.sentence) }
+    assert_raise(ArgumentError, nil) { FooComponent.new(Faker::Lorem.sentence) }
   end
   
   test "assigns the values to instance variables" do

data/test/active_component/context_test.rb

@@ -2,9 +2,24 @@ require 'test_helper'
 
 class ActiveComponent::ContextTest < ActiveSupport::TestCase
 
-  test "returns a cotroller instance" do
+  def setup
     FooController.new.get_context
+  end
+
+  test "returns a cotroller instance" do
     assert_kind_of FooController, ActiveComponent.get_controller
   end
 
+  test "saves the controller in a @@controllers hash" do
+    assert_nothing_raised Exception do
+      ActiveComponent.set_controller(FooController.new)
+    end
+  end
+
+  test "it returns exactly the same controller sent" do
+    controller = FooController.new
+    controller.get_context
+    assert_equal controller, ActiveComponent.controllers[Thread.current.object_id]
+  end
+
 end

data/test/active_component/railtie_test.rb

@@ -2,8 +2,14 @@ require 'test_helper'
 
 class ActiveComponent::RailtieTest < ActiveSupport::TestCase
 
-  test "get the controller context" do
-    assert FooController.new.render_component
+  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,7 +3,15 @@ require 'test_helper'
 class ActiveComponent::RenderableTest < ActiveSupport::TestCase
 
   test "renders a component" do
-    assert_kind_of(String, FooController.new.render_component)
+    FooComponent.any_instance.stubs(:render).returns("")
+    assert_nothing_raised TypeError do
+      ActionView::Base.new.render_component(:foo)
+    end
+  end
+
+  test "gets a component instance from symbol" do
+    FooComponent.any_instance.stubs(:render).returns("")
+    assert_kind_of String, ActionView::Base.new.render_component(:foo)
   end
 
 end

data/test/dummy/log/test.log

@@ -2777,3 +2777,68 @@ ActiveComponent::RenderableTest: test_renders_a_component
 ActiveComponentTest: test_has_a_get_controller_method
 -----------------------------------------------------
    (0.1ms)  rollback transaction
+   (0.3ms)  begin transaction
+--------------------------------------------------------------------------------------------------------
+ActiveComponent::BaseTest: test_a_new_instance_of_active_component/base_accepts_only_a_hash_of_arguments
+--------------------------------------------------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.1ms)  begin transaction
+-------------------------------------------------------------------------------------------------------------------------
+ActiveComponent::BaseTest: test_a_new_instance_of_active_component_should_raise_an_error_if_the_given_value_is_not_a_hash
+-------------------------------------------------------------------------------------------------------------------------
+   (0.0ms)  rollback transaction
+   (0.1ms)  begin transaction
+------------------------------------------------------------------------
+ActiveComponent::BaseTest: test_assigns_the_values_to_instance_variables
+------------------------------------------------------------------------
+   (0.0ms)  rollback transaction
+   (0.0ms)  begin transaction
+-------------------------------------------------------------------------
+ActiveComponent::BaseTest: test_has_included_date_helper_from_action_view
+-------------------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.0ms)  begin transaction
+-------------------------------------------------------------------------
+ActiveComponent::BaseTest: test_sets_controller_for_the_component_context
+-------------------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.1ms)  begin transaction
+------------------------------------------------------------------------------
+ActiveComponent::ContextTest: test_it_returns_exactly_the_same_controller_sent
+------------------------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.1ms)  begin transaction
+---------------------------------------------------------------
+ActiveComponent::ContextTest: test_returns_a_cotroller_instance
+---------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.1ms)  begin transaction
+-------------------------------------------------------------------------------
+ActiveComponent::ContextTest: test_saves_the_controller_in_a_@@controllers_hash
+-------------------------------------------------------------------------------
+   (0.0ms)  rollback transaction
+   (0.0ms)  begin transaction
+----------------------------------------------------
+ActiveComponent::RailtieTest: test_has_before_filter
+----------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.0ms)  begin transaction
+---------------------------------------------------------
+ActiveComponent::RailtieTest: test_has_the_context_method
+---------------------------------------------------------
+   (0.0ms)  rollback transaction
+   (0.0ms)  begin transaction
+---------------------------------------------------------------------------
+ActiveComponent::RenderableTest: test_gets_a_component_instance_from_symbol
+---------------------------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.0ms)  begin transaction
+---------------------------------------------------------
+ActiveComponent::RenderableTest: test_renders_a_component
+---------------------------------------------------------
+   (0.1ms)  rollback transaction
+   (0.0ms)  begin transaction
+-----------------------------------------------------
+ActiveComponentTest: test_has_a_get_controller_method
+-----------------------------------------------------
+   (0.0ms)  rollback transaction

data/test/support/foo_component.mustache

@@ -0,0 +1 @@
+

data/test/support/foo_component.rb

@@ -0,0 +1,3 @@
+class FooComponent < ActiveComponent::Base
+  attr_accessor :bar
+end

data/test/test_helper.rb

@@ -5,6 +5,7 @@ require File.expand_path("../dummy/config/environment.rb",  __FILE__)
 require "rails/test_help"
 require 'faker'
 require 'pry'
+require 'mocha/setup'
 
 Rails.backtrace_cleaner.remove_silencers!
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active-component
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Antonio Chavez
@@ -81,6 +81,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  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
@@ -159,7 +173,8 @@ files:
 - test/dummy/tmp/cache/assets/development/sprockets/edbef6e0d0a4742346cf479f2c522eb0
 - test/dummy/tmp/cache/assets/development/sprockets/f7cbd26ba1d28d48de824f0e94586655
 - test/generators/component_generator_test.rb
-- test/support/foo.rb
+- test/support/foo_component.mustache
+- test/support/foo_component.rb
 - test/support/foo_controller.rb
 - test/test_helper.rb
 homepage: https://github.com/TheNaoX/active_component
@@ -245,7 +260,8 @@ test_files:
 - test/dummy/tmp/cache/assets/development/sprockets/edbef6e0d0a4742346cf479f2c522eb0
 - test/dummy/tmp/cache/assets/development/sprockets/f7cbd26ba1d28d48de824f0e94586655
 - test/generators/component_generator_test.rb
-- test/support/foo.rb
+- test/support/foo_component.mustache
+- test/support/foo_component.rb
 - test/support/foo_controller.rb
 - test/test_helper.rb
 has_rdoc: 

data/test/support/foo.rb

@@ -1,3 +0,0 @@
-class Foo < ActiveComponent::Base
-  attr_accessor :bar
-end