engine-assets 0.2.1 → 0.3.0

data/TODO.md

@@ -1,4 +1,6 @@
 
   * support@coolerator.net
   * Visit http://github.com/coreyti/engine-assets/edit and click 'Enable RubyGems'
+  * from AssetTagHelper:
+    `collect_asset_files(JAVASCRIPTS_DIR, ('**' if recursive), '*.js')`
   

data/VERSION

@@ -1 +1 @@
-0.2.1
+0.3.0

data/engine-assets.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{engine-assets}
-  s.version = "0.2.1"
+  s.version = "0.3.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Corey Innis"]
-  s.date = %q{2009-11-07}
+  s.date = %q{2009-11-08}
   s.description = %q{A Rails Engine, which enables Rails Engines to provide assets (javascript, css and images)}
   s.email = %q{support@coolerator.net}
   s.extra_rdoc_files = [
@@ -30,12 +30,16 @@ Gem::Specification.new do |s|
      "config/routes.rb",
      "engine-assets.gemspec",
      "lib/engine-assets.rb",
-     "lib/engine_assets/extensions/rails/plugin.rb",
+     "lib/engine_assets/extensions/rails/assets.rb",
+     "lib/engine_assets/extensions/rails/plugins.rb",
      "lib/engine_assets/extensions/rails/routes.rb",
      "lib/engine_assets/public_locator.rb",
      "rails/init.rb",
      "spec/controllers/javascripts_controller_spec.rb",
      "spec/controllers/stylesheets_controller_spec.rb",
+     "spec/lib/engine_assets/extensions/rails/plugins_spec.rb",
+     "spec/lib/engine_assets/extensions/rails/routes_spec.rb",
+     "spec/lib/engine_assets/public_locator_spec.rb",
      "spec/routing/javascripts_routing_spec.rb",
      "spec/routing/stylesheets_routing_spec.rb",
      "spec/spec.opts",
@@ -60,6 +64,9 @@ Gem::Specification.new do |s|
   s.test_files = [
     "spec/controllers/javascripts_controller_spec.rb",
      "spec/controllers/stylesheets_controller_spec.rb",
+     "spec/lib/engine_assets/extensions/rails/plugins_spec.rb",
+     "spec/lib/engine_assets/extensions/rails/routes_spec.rb",
+     "spec/lib/engine_assets/public_locator_spec.rb",
      "spec/routing/javascripts_routing_spec.rb",
      "spec/routing/stylesheets_routing_spec.rb",
      "spec/spec_helper.rb",

data/lib/engine-assets.rb

@@ -1,5 +1,6 @@
 module EngineAssets ; end
 
 require 'engine_assets/public_locator'
-require 'engine_assets/extensions/rails/plugin'
+require 'engine_assets/extensions/rails/assets'
+require 'engine_assets/extensions/rails/plugins'
 require 'engine_assets/extensions/rails/routes'

data/lib/engine_assets/extensions/rails/assets.rb

@@ -0,0 +1,15 @@
+module ActionView::Helpers::AssetTagHelper
+  def expand_javascript_sources(sources, recursive = false)
+    if sources.include?(:all)
+      all_javascript_files = collect_asset_files(JAVASCRIPTS_DIR, ('**' if recursive), '*.js')
+      result = ((determine_source(:defaults, @@javascript_expansions).dup) + all_javascript_files).uniq
+      result
+    else
+      expanded_sources = sources.collect do |source|
+        determine_source(source, @@javascript_expansions)
+      end.flatten
+      expanded_sources << "application" if sources.include?(:defaults) && File.exist?(File.join(JAVASCRIPTS_DIR, "application.js"))
+      expanded_sources
+    end
+  end
+end

data/lib/engine_assets/extensions/rails/routes.rb

@@ -1,5 +1,4 @@
 if defined?(ActionController::Routing::RouteSet)
-
   class ActionController::Routing::RouteSet
     def load_routes_with_assets!
       lib_path      = File.dirname(__FILE__)

data/spec/lib/engine_assets/extensions/rails/plugins_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe Rails::Plugin::Loader do
+  attr_reader :loader, :plugin, :directory
+
+  before do
+    @loader = Rails::Plugin::Loader.new(OpenStruct.new({ :loaded_plugins => [] }))
+  end
+
+  describe "#register_plugin_as_loaded" do
+    before do
+      paths = []
+      stub(EngineAssets::PublicLocator).paths { paths }
+    end
+
+    context "given a plugin which is an engine and contains a 'public' directory" do
+      before do
+        @directory = File.join(basedir, 'spec', 'support', 'fixtures')
+        @plugin    = OpenStruct.new({ :engine? => true, :directory => directory })
+      end
+
+      it "registers the plugin with the EngineAssets::PublicLocator" do
+        loader.send(:register_plugin_as_loaded, plugin)
+        EngineAssets::PublicLocator.paths.should == [ "#{directory}/public" ]
+      end
+    end
+
+    context "given a plugin which is an engine and does NOT contain a 'public' directory" do
+      before do
+        @directory = basedir
+        @plugin    = OpenStruct.new({ :engine? => true, :directory => directory })
+      end
+
+      it "does not register the plugin with the EngineAssets::PublicLocator" do
+        loader.send(:register_plugin_as_loaded, plugin)
+        EngineAssets::PublicLocator.paths.should == []
+      end
+    end
+
+    context "given a plugin which is NOT an engine" do
+      before do
+        @directory = File.join(basedir, 'spec', 'support', 'fixtures')
+        @plugin    = OpenStruct.new({ :engine? => false, :directory => directory })
+      end
+
+      it "does not register the plugin with the EngineAssets::PublicLocator" do
+        loader.send(:register_plugin_as_loaded, plugin)
+        EngineAssets::PublicLocator.paths.should == []
+      end
+    end
+  end
+end

data/spec/lib/engine_assets/extensions/rails/routes_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe ActionController::Routing::RouteSet do
+  describe "#load_routes!" do
+    class TestRouteSet < ActionController::Routing::RouteSet
+      attr_reader :configuration_files
+
+      def initialize
+        @configuration_files = []
+      end
+
+      def add_configuration_file(routes)
+        configuration_files << routes
+      end
+    end
+
+    attr_reader :routeset
+
+    before do
+      @routeset = TestRouteSet.new
+    end
+
+    it "adds the engine-assets routes configuration" do
+      routeset.load_routes!
+      routeset.configuration_files[0].should =~ /engine-assets.*\/config\/routes\.rb/
+    end
+  end
+end

data/spec/lib/engine_assets/public_locator_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe EngineAssets::PublicLocator do
+  attr_reader :base_path, :find_path, :miss_path
+
+  before do
+    @base_path = File.join(basedir, 'spec', 'support', 'fixtures', 'public')
+    @find_path = '/javascripts/dual.js'
+    @miss_path = '/javascripts/miss.js'
+
+    EngineAssets::PublicLocator.paths << base_path
+  end
+
+  describe "#locate" do
+    context "when the requested sub-path is located" do
+      it "returns the full path to the file" do
+        EngineAssets::PublicLocator.locate(find_path).should == File.join(base_path, find_path)
+      end
+    end
+
+    context "when the requested sub-path is not located" do
+      it "returns nil" do
+        EngineAssets::PublicLocator.locate(miss_path).should be_nil
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -3,6 +3,10 @@ RAILS_PATH = File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'van
 dir = File.dirname(__FILE__)
 $LOAD_PATH.unshift(dir)
 
+def require_files(*segments)
+  Dir[File.expand_path(File.join(*segments))].each { |file| require file }
+end
+
 ENV["RAILS_ENV"] = "test"
 
 require "#{File.join(RAILS_PATH, 'config', 'environment')}"
@@ -10,10 +14,8 @@ require 'engine-assets'
 require 'spec'
 require 'spec/autorun'
 require 'spec/rails'
-require 'support/helpers/fixture_helper'
-require 'support/helpers/textmate_helper'
-require 'support/shared/assets_controller_spec'
-require 'support/shared/assets_routing_spec'
+
+require_files(dir, 'support', '**', '*.rb')
 
 Spec::Runner.configure do |config|
   include Support::Helpers
@@ -23,12 +25,7 @@ Spec::Runner.configure do |config|
   config.use_transactional_fixtures = true
   config.use_instantiated_fixtures  = false
 
-  # def add_fixture_views(controller_class)
-  #   controller_class.prepend_view_path(File.join(basepath, '..', 'fixtures', 'app', 'views'))
-  #   controller_class.prepend_view_path(File.join(basepath, '..', 'fixtures', 'public'))
-  # end
-  # 
-  # def basepath
-  #   @basepath ||= File.join(File.dirname(__FILE__), '..')
-  # end
+  def basedir
+    @basedir ||= "#{File.expand_path(File.join(File.dirname(__FILE__), '..'))}"
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: engine-assets
 version: !ruby/object:Gem::Version 
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors: 
 - Corey Innis
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-11-07 00:00:00 -05:00
+date: 2009-11-08 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -45,12 +45,16 @@ files:
 - config/routes.rb
 - engine-assets.gemspec
 - lib/engine-assets.rb
-- lib/engine_assets/extensions/rails/plugin.rb
+- lib/engine_assets/extensions/rails/assets.rb
+- lib/engine_assets/extensions/rails/plugins.rb
 - lib/engine_assets/extensions/rails/routes.rb
 - lib/engine_assets/public_locator.rb
 - rails/init.rb
 - spec/controllers/javascripts_controller_spec.rb
 - spec/controllers/stylesheets_controller_spec.rb
+- spec/lib/engine_assets/extensions/rails/plugins_spec.rb
+- spec/lib/engine_assets/extensions/rails/routes_spec.rb
+- spec/lib/engine_assets/public_locator_spec.rb
 - spec/routing/javascripts_routing_spec.rb
 - spec/routing/stylesheets_routing_spec.rb
 - spec/spec.opts
@@ -97,6 +101,9 @@ summary: Rails Engines with assets.
 test_files: 
 - spec/controllers/javascripts_controller_spec.rb
 - spec/controllers/stylesheets_controller_spec.rb
+- spec/lib/engine_assets/extensions/rails/plugins_spec.rb
+- spec/lib/engine_assets/extensions/rails/routes_spec.rb
+- spec/lib/engine_assets/public_locator_spec.rb
 - spec/routing/javascripts_routing_spec.rb
 - spec/routing/stylesheets_routing_spec.rb
 - spec/spec_helper.rb