parcels 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 658277fafd37674b7c9be8026a325a2cb907f89c
-  data.tar.gz: a1f74856053841ef8fb0573d246b64a712544c49
+  metadata.gz: f8c86251a8065212ca19858c38857005a7871457
+  data.tar.gz: 9be314a2db597e039c54dd4c4365e649e345fce5
 SHA512:
-  metadata.gz: a589bcc70801b68c35d4d3b16e2a07e7863a199e211a96ca50f171dac1ea623215ed034084f8efcd7845b827565275eb51d832bd6a8a00bcf8f17f90d6606fcb
-  data.tar.gz: 3005a5072e9d47179a0946ce166626cf6db502f9c99fe1f74b87014b57b2776369c1d04427a7f85be903a3d309d7002ca8a7abdad50d3327f8373343d90d8dd8
+  metadata.gz: 0507164165fabbce70addd13598209794d45bbfe414870db3cda85e8766de0edeca1597f973170b55444979443bd941823a00979396b3c5c7e8d02398d88be5b
+  data.tar.gz: 89b09a7873d8ebbf440aa06a8673bc7ddc66dd7e42975a0d72d3adf963cfddc649d252eef73691b595e25f1aa6d94da9392c5011ff222e724b506342eb43d54d

data/CHANGES.md

@@ -0,0 +1,8 @@
+# Parcels Releases
+
+## 0.0.2, 21 January 2015
+
+* Fixed an issue where having a `.rb` file in your `views` directory that didn't define a widget class, but rather a
+  module (presumably a "helper-like" module of view methods), would cause asset compilation to fail.
+* Improved compatibility with non-Rails build environments (_e.g._, that used by
+  [Middleman](http://middlemanapp.com/)).

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2014 Andrew Geweke
+Copyright (c) 2014-2015 Andrew Geweke
 
 MIT License
 

data/lib/parcels/environment.rb

@@ -9,6 +9,8 @@ module Parcels
     def initialize(sprockets_environment)
       @sprockets_environment = sprockets_environment
       @widget_trees = [ ]
+      @workaround_directories_for_widget_trees = { }
+      @workaround_directories_root = nil
 
       register_engines!
     end
@@ -30,8 +32,15 @@ module Parcels
       widget_tree.add_workaround_directory_to_sprockets!(sprockets_environment)
     end
 
-    def widget_class_from_file(pathname)
-      ::Fortitude::Widget.widget_class_from_file(pathname, :root_dirs => widget_trees.map(&:root))
+    def widget_class_from_file(full_path)
+      widget_trees.each do |widget_tree|
+        if (removed = widget_tree.remove_workaround_directory_from(full_path))
+          full_path = removed
+          break
+        end
+      end
+
+      ::Fortitude::Widget.widget_class_from_file(full_path, :root_dirs => widget_trees.map(&:widget_naming_root_dirs).flatten.uniq)
     end
 
     def create_and_add_all_workaround_directories!
@@ -41,12 +50,53 @@ module Parcels
     end
 
     def add_all_widgets_to!(sprockets_context, set_names)
+      if widget_trees.length == 0
+        raise %{Error: You have not defined any widget trees -- directories containing Fortitude widgets.
+You must call #add_widget_tree! on the Parcels environment, which usually is accessible
+as #parcels from your Sprockets environment.}
+      end
+
       widget_trees.each { |wt| wt.add_all_widgets_to_sprockets_context!(sprockets_context, set_names) }
     end
 
+    def workaround_directory_root_for_widget_tree(widget_tree)
+      @workaround_directories_for_widget_trees[widget_tree] ||= begin
+        if @workaround_directories_root
+          File.join(@workaround_directories_root, workaround_directory_name_for(widget_tree))
+        else
+          File.join(widget_tree.root, PARCELS_WORKAROUND_DIRECTORY_NAME)
+        end
+      end
+    end
+
+    def workaround_directories_root
+      @workaround_directories_root
+    end
+
+    def workaround_directories_root=(new_root)
+      new_root = File.expand_path(new_root)
+
+      if @workaround_directories_for_widget_trees.size > 0 && @workaround_directories_root != new_root
+        raise "You can't set the workaround directories root to:
+#{new_root}
+...it's already set to:
+#{@workaround_directories_root}"
+      end
+
+      @workaround_directories_root = new_root
+    end
+
     private
     attr_reader :widget_trees
 
+    PARCELS_WORKAROUND_DIRECTORY_NAME = ".parcels_sprockets_workaround".freeze
+
+    def workaround_directory_name_for(widget_tree)
+      require 'digest/md5'
+      digest = Digest::MD5.hexdigest(widget_tree.root).strip
+      "#{PARCELS_WORKAROUND_DIRECTORY_NAME}_#{digest}"
+    end
+
     def register_engines!
       @engines_registered ||= begin
         if ::Parcels.fortitude_available?

data/lib/parcels/fortitude/assets.rb

@@ -145,6 +145,9 @@ you may want to enable Parcels on any of its Fortitude superclasses, which are:
           end
 
           @_parcels_inline_css_fragments ||= [ ]
+          @_parcels_inline_css_fragments.delete_if do |fragment|
+            fragment.file == caller_file && fragment.line >= caller_line
+          end
           @_parcels_inline_css_fragments += css_strings.map do |css_string|
             ::Parcels::Fragments::CssFragment.new(css_string, self, caller_file, caller_line, _parcels_css_options.merge(options))
           end

data/lib/parcels/fortitude_alongside_parcel.rb

@@ -24,7 +24,7 @@ module Parcels
     end
 
     def usable?
-      widget_class_full_path
+      super && widget_class_full_path
     end
 
     private

data/lib/parcels/fortitude_parcel.rb

@@ -26,7 +26,7 @@ module Parcels
     end
 
     def usable?
-      true
+      !! widget_class
     end
 
     def included_in_any_set?(set_names)
@@ -96,7 +96,20 @@ module Parcels
     end
 
     def widget_class
-      @widget_class ||= ::Fortitude::Widget.widget_class_from_file(widget_class_full_path, :root_dirs => [ widget_tree.root ])
+      @widget_class ||= begin
+        if widget_class_full_path
+          begin
+            ::Fortitude::Widget.widget_class_from_file(widget_class_full_path,
+              :root_dirs => widget_tree.widget_naming_root_dirs)
+          rescue ::Fortitude::Widget::Files::CannotDetermineWidgetClassNameError => cdwcne
+            raise unless cdwcne.resulting_objects.detect { |o| o.kind_of?(Module) }
+            :none
+          end
+        else
+          :none
+        end
+      end
+      @widget_class unless @widget_class == :none
     end
   end
 end

data/lib/parcels/fragments/css_fragment.rb

@@ -8,6 +8,8 @@ module Parcels
         end
       end
 
+      attr_reader :file, :line
+
       def initialize(css_string, source, file, line, options)
         options.assert_valid_keys(:engines, :wrap, :prefix)
 
@@ -51,7 +53,7 @@ module Parcels
       end
 
       private
-      attr_reader :css_string, :source, :options, :file, :line
+      attr_reader :css_string, :source, :options
 
       def header_comment
         out = "/* From '#{file}'"
@@ -91,13 +93,8 @@ module Parcels
         result = data
 
         processors.each do |processor|
-          args = [ file ]
-          if processor == ::Tilt::ScssTemplate && (::Parcels::Sprockets.requires_explicit_load_paths_for_css_template?)
-            args = [ file, 1, { :load_paths => context.environment.paths } ]
-          end
-
-          template = processor.new(*args) { result }
-          result = template.render(context, {})
+          template = processor.new(file, 1, { :load_paths => context.environment.paths, :filename => file }) { result }
+          result = template.render(context, { :filename => file })
         end
 
         result

data/lib/parcels/sprockets.rb

@@ -9,10 +9,6 @@ module Parcels
       @sprockets_version_components ||= ::Sprockets::VERSION.split(".").map { |x| x.to_i }
     end
 
-    def self.requires_explicit_load_paths_for_css_template?
-      sprockets_version_components[0] <= 2 && sprockets_version_components[1] < 11
-    end
-
     def self.requires_fix_for_protected_methods_on_directive_processor?
       RUBY_VERSION =~ /^2/ && sprockets_version_components[0] <= 2 && sprockets_version_components[1] < 11
     end
@@ -95,7 +91,7 @@ if static_compiler_class
         end
 
         def each_logical_path(*args, &block)
-          @env.each_logical_path do |logical_path|
+          @env.each_logical_path(*args) do |logical_path|
             unless ::Parcels.is_fortitude_logical_path?(logical_path)
               block.call(logical_path)
             end
@@ -110,27 +106,42 @@ if static_compiler_class
 
     static_compiler_class.class_eval do
       def env
-        @_parcels_env_proxy ||= ::Parcels::Sprockets::StaticCompilerEnvProxy.new(@env)
+        ::Parcels::Sprockets::StaticCompilerEnvProxy.new(@env)
       end
     end
   end
 end
 
-begin
-  require 'sass/rails/importer'
-rescue LoadError => le
-  # oh, well
+[ 'sass/rails/importer', 'sprockets/sass/importer' ].each do |filename|
+  begin
+    require filename
+  rescue LoadError => le
+    # oh, well
+  end
 end
 
-[ '::Sprockets::SassImporter', '::Sass::Rails::Importer' ].each do |class_name|
+[ '::Sprockets::SassImporter', '::Sass::Rails::Importer', '::Sprockets::Sass::Importer' ].each do |class_name|
   klass = class_name.constantize rescue nil
 
   if klass
     klass.class_eval do
       if defined?(klass::GLOB)
         def find_relative_with_parcels(name, base, options)
-          parcels = context.environment.parcels
-          expanded_locations_to_search = context.environment.paths + [ File.dirname(base) ]
+          sprockets_context = (options[:custom] || { })[:sprockets_context]
+          sprockets_context ||= context if respond_to?(:context)
+          unless sprockets_context
+            raise "Unable to find the Sprockets context here; it was neither in options, nor do we have it in the class"
+          end
+
+          parcels = sprockets_context.environment.parcels
+          expanded_locations_to_search = sprockets_context.environment.paths + [ File.dirname(base) ]
+          expanded_locations_to_search = expanded_locations_to_search.map do |location|
+            if location.kind_of?(::Pathname)
+              location
+            else
+              ::Pathname.new(location.to_s)
+            end
+          end
 
           if name =~ self.class.const_get(:GLOB) && parcels.is_underneath_root?(base)
             paths_to_search = expanded_locations_to_search
@@ -143,7 +154,8 @@ end
             end
           end
 
-          return find_relative_without_parcels(name, base, options.merge(:load_paths => expanded_locations_to_search))
+          load_paths = (options[:load_paths] || [ ]) + expanded_locations_to_search
+          return find_relative_without_parcels(name, base, options.merge(:load_paths => load_paths))
         end
 
         alias_method_chain :find_relative, :parcels

data/lib/parcels/utils/path_utils.rb

@@ -3,6 +3,11 @@ module Parcels
     module PathUtils
       class << self
         def path_under(path, under_what)
+          maybe_path_under(path, under_what) ||
+            raise(Errno::ENOENT, %{Path #{path.inspect} is not underneath directory #{under_what.inspect}.})
+        end
+
+        def maybe_path_under(path, under_what)
           path = path.to_s.strip
           under_what = under_what.to_s.strip
 
@@ -10,8 +15,6 @@ module Parcels
             path[under_what.length..under_what.length] == File::SEPARATOR
 
             path[(under_what.length + 1)..-1]
-          else
-            raise Errno::ENOENT, %{Path #{path.inspect} is not underneath directory #{under_what.inspect}.}
           end
         end
 

data/lib/parcels/version.rb

@@ -1,3 +1,3 @@
 module Parcels
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/parcels/widget_tree.rb

@@ -4,6 +4,8 @@ require 'parcels/fortitude_inline_parcel'
 require 'parcels/fortitude_alongside_parcel'
 require 'parcels/dependency_parcel_list'
 
+require 'pathname'
+
 require 'find'
 
 module Parcels
@@ -17,11 +19,14 @@ module Parcels
       @sprockets_environments_added_to = { }
     end
 
+    def widget_naming_root_dirs
+      @widget_naming_root_dirs ||= [ root, File.dirname(root) ]
+    end
+
     def add_workaround_directory_to_sprockets!(sprockets_environment)
       return if (! root_exists?)
 
       @sprockets_environments_added_to[sprockets_environment] ||= begin
-        ensure_workaround_directory_is_set_up!
         sprockets_environment.prepend_path(workaround_directory)
         true
       end
@@ -31,6 +36,26 @@ module Parcels
       ::Parcels::Utils::PathUtils.path_under(full_path, root)
     end
 
+    def remove_workaround_directory_from(full_path)
+      if (path_under_workaround = ::Parcels::Utils::PathUtils.maybe_path_under(full_path, workaround_directory))
+        if (separator_index = path_under_workaround.index(File::SEPARATOR))
+          if (separator_index > 0 && separator_index < (path_under_workaround.length - 1))
+            link_name = path_under_workaround[0..(separator_index - 1)]
+            after_link = path_under_workaround[(separator_index + 1)..-1]
+
+            if File.symlink?(File.join(workaround_directory, link_name))
+              link_value = File.readlink(File.join(workaround_directory, link_name))
+              resolved_link = File.expand_path(File.join(workaround_directory, link_value))
+              final_path = File.join(resolved_link, after_link)
+              return final_path
+            end
+          end
+        end
+      end
+
+      nil
+    end
+
     def add_all_widgets_to_sprockets_context!(sprockets_context, set_names)
       return unless root_exists?
       sprockets_context.depend_on(root)
@@ -47,7 +72,13 @@ module Parcels
         extension = File.extname(full_path).strip.downcase
         if (klass = EXTENSION_TO_PARCEL_CLASS_MAP[extension])
           parcel = klass.new(self, full_path)
-          all_parcels << parcel if parcel.usable? && parcel.included_in_any_set?(set_names)
+          if parcel.usable? && parcel.included_in_any_set?(set_names)
+            if all_parcels.length == 0
+              ensure_workaround_directory_is_set_up!
+            end
+
+            all_parcels << parcel
+          end
         end
       end
 
@@ -66,8 +97,6 @@ module Parcels
 
     ALL_EXTENSIONS                    = EXTENSION_TO_PARCEL_CLASS_MAP.keys.dup.freeze
 
-    PARCELS_WORKAROUND_DIRECTORY_NAME = ".parcels_sprockets_workaround".freeze
-
     PARCELS_LOGICAL_PATH_PREFIXES     = EXTENSION_TO_PARCEL_CLASS_MAP.values.map { |k| k.logical_path_prefix }
 
     def root_exists?
@@ -75,7 +104,7 @@ module Parcels
     end
 
     def workaround_directory
-      @workaround_directory ||= File.join(root, PARCELS_WORKAROUND_DIRECTORY_NAME)
+      @workaround_directory ||= parcels_environment.workaround_directory_root_for_widget_tree(self)
     end
 
     def ensure_workaround_directory_is_set_up!
@@ -127,7 +156,7 @@ However, this directory currently contains other file(s) that we weren't expecti
           if File.symlink?(symlink)
             contents = File.readlink(symlink)
 
-            if contents == SYMLINK_TARGET
+            if contents == symlink_target
               # ok, great
             else
               File.delete(symlink)
@@ -151,8 +180,14 @@ and try again.}
 
     def create_symlink!(prefix)
       Dir.chdir(workaround_directory) do
-        FileUtils.ln_s(SYMLINK_TARGET, prefix)
+        FileUtils.ln_s(symlink_target, prefix)
       end
     end
+
+    def symlink_target
+      workaround_dir_path = Pathname.new(workaround_directory).realpath
+      root_path = Pathname.new(root).realpath
+      root_path.relative_path_from(workaround_dir_path).to_s
+    end
   end
 end

data/parcels.gemspec

@@ -44,5 +44,5 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "oop_rails_server", "~> 0", ">= 0.0.5"
   spec.add_development_dependency "css_parser"
-  spec.add_development_dependency "fortitude", ">= 0.9.1"
+  spec.add_development_dependency "fortitude", ">= 0.9.2"
 end

data/spec/helpers/parcels_rails_helpers.rb

@@ -5,7 +5,7 @@ module ParcelsRailsHelpers
 
   def rails_server_additional_gemfile_lines
     [
-      "gem 'fortitude', \">= 0.9.0\"",
+      "gem 'fortitude', \">= 0.9.2\"",
       "gem 'parcels', :path => '#{rails_server_project_root}'"
     ]
   end

data/spec/helpers/sprockets_helpers.rb

@@ -3,9 +3,13 @@ module SprocketsHelpers
     ::Sprockets::Environment.new(*args)
   end
 
+  def reset_sprockets_env!
+    per_example_data[:sprockets_env] = nil
+  end
+
   def sprockets_env(*args, &block)
     out = if args.length > 0 || block
-      raise "Duplicate creation of sprockets_env? #{args.inspect}" if @sprockets_env
+      raise "Duplicate creation of sprockets_env? #{args.inspect}" if per_example_data[:sprockets_env]
       args = [ this_example_root ] if args.length == 0
       per_example_data[:sprockets_env] = new_sprockets_env(*args)
       block.call(per_example_data[:sprockets_env]) if block
@@ -17,7 +21,13 @@ module SprocketsHelpers
       end
     end
 
-    out.parcels.add_widget_tree!(File.join(this_example_root, 'views'))
+    default_widget_trees_to_add_to_sprockets.each do |widget_tree_root|
+      out.parcels.add_widget_tree!(widget_tree_root)
+    end
     out
   end
+
+  def default_widget_trees_to_add_to_sprockets
+    [ File.join(this_example_root, 'views') ]
+  end
 end

data/spec/lib/spec/expected/expected_asset.rb

@@ -5,11 +5,14 @@ require 'spec/expected/base_expected_asset'
 module Spec
   module Expected
     class ExpectedAsset < BaseExpectedAsset
-      def initialize(root_directory, expected_subpath, &block)
+      def initialize(root_directory, expected_subpath, options = { }, &block)
         super(root_directory, expected_subpath)
 
         @expected_rules = { }
         @allow_extra_rules = false
+        @options = options
+
+        options.assert_valid_keys(:sequencing)
 
         instance_eval(&block) if block
       end
@@ -52,7 +55,7 @@ module Spec
             message << "\n\n    #{remaining_asset}:\n        #{remaining_asset.source}\n"
           end
           raise message
-        elsif matching_remaining_assets.length == 1
+        elsif matching_remaining_assets.length == 1 || options[:sequencing]
           matching_remaining_asset = matching_remaining_assets.first
 
           unless asset_matches?(matching_remaining_asset)
@@ -79,7 +82,7 @@ module Spec
       end
 
       private
-      attr_reader :expected_rules
+      attr_reader :expected_rules, :options
 
       def extra_rules_allowed?
         !! @allow_extra_rules

data/spec/lib/spec/expected/expected_asset_set.rb

@@ -12,8 +12,8 @@ module Spec
         instance_eval(&block) if block
       end
 
-      def asset(subpath, &block)
-        expected_asset = Spec::Expected::ExpectedAsset.new(root_directory, subpath, &block)
+      def asset(subpath, options = { }, &block)
+        expected_asset = Spec::Expected::ExpectedAsset.new(root_directory, subpath, options, &block)
         @expected_assets << expected_asset
       end
 

data/spec/rails/basic_rails_spec.rb

@@ -99,4 +99,15 @@ describe "Parcels Rails basic support", :type => :rails do
       allow_additional_assets!
     end)
   end
+
+  it "should allow putting a .rb file in views/ that is just a module (~ helpers)" do
+    compiled_rails_asset('application.css').should_match(rails_assets do
+      asset 'views/basic_rails_spec/helpers_module_test.rb' do
+        expect_wrapped_rule :p, 'color: green'
+      end
+      allow_additional_assets!
+    end)
+
+    expect_match("helpers_module_test", /and it is this is my_helper\! tada\!/)
+  end
 end

data/spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module.rb

@@ -0,0 +1,5 @@
+module Views::BasicRailsSpec::HelpersModule
+  def my_helper
+    text "this is my_helper!"
+  end
+end

data/spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module_test.rb

@@ -0,0 +1,13 @@
+class Views::BasicRailsSpec::HelpersModuleTest < Views::Widgets::Base
+  include Views::BasicRailsSpec::HelpersModule
+
+  css %{
+    p { color: green; }
+  }
+
+  def content
+    text "and it is "
+    my_helper
+    text " tada!"
+  end
+end

data/spec/system/ad_hoc_development_mode_spec.rb

@@ -0,0 +1,80 @@
+ describe "Parcels ad-hoc development mode support", :type => :system do
+  # Rails' development mode (which is really just ActiveSupport's class-unloading support) does the right thing:
+  # it actually completely unloads every class after each request cycle, removing the old class definition from memory
+  # (well, modulo GC, but you get the idea). This means that if you remove or change CSS definitions in a widget class,
+  # they will genuinely be gone/changed on the next iteration, and Parcels will behave correctly.
+  #
+  # Some programs, however -- Middleman is where I discovered this issue -- simply re-evaluate templates in development
+  # mode. For "normal" templates (e.g., ERb, HAML, whatever), this works just fine. But, with Parcels, this can be an
+  # issue: the widget class definition itself just gets re-processed, and, if you've removed CSS from the class or
+  # changed that CSS, the class will now have both the new *and* the old definitions around. This causes really
+  # frustrating chaos as your changes seem to sometimes, but not always, get picked up, depending on whether the new
+  # CSS actually overrides the old or depends on the old not being present (like removing a CSS selector, etc.).
+  #
+  # Unfortunately, due to the way Ruby classes work, there's no one "right way" around this issue: Ruby doesn't have
+  # any kind of callback saying "class X is getting defined", because re-opening classes means that there's really no
+  # objective way of deciding when that's true. Instead, we check for declarations of CSS on a widget class where the
+  # line number is equal to, or before, the line numbers of other CSS declarations, and remove those other declarations
+  # when this happens.
+  #
+  # This doesn't catch every single case -- if you simultaneously change CSS and add one or more lines above the .css
+  # call in a class, for example, it will be missed -- but it catches the vast majority of cases, and is uniformly an
+  # improvement to the system.
+  #
+  # This is a test to make sure that works.
+  context "with a widget class that gets changed" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget 'views/my_widget' do
+          css %{
+            p { color: red; }
+          }
+          css %{
+            span { color: green; }
+          }
+        end
+      }
+    end
+
+    it "should handle the case where the CSS changes" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.rb', :sequencing => true do
+          expect_wrapped_rule :p, 'color: red'
+        end
+
+        asset 'views/my_widget.rb', :sequencing => true do
+          expect_wrapped_rule :span, 'color: green'
+        end
+      end)
+
+      widget_path = File.join(this_example_root, 'views', 'my_widget.rb')
+      expect(File.exist?(widget_path)).to be
+
+      File.open(widget_path, 'w') do |f|
+        f << <<-EOS
+class Views::MyWidget < ::Spec::Fixtures::WidgetBase
+  css %{
+    p { color: yellow; }
+  }
+end
+EOS
+      end
+
+      old_sprockets_env = sprockets_env
+      reset_sprockets_env!
+
+      load(widget_path)
+      expect(sprockets_env.object_id).not_to eq(old_sprockets_env.object_id)
+
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.rb' do
+          expect_wrapped_rule :p, 'color: yellow'
+        end
+      end)
+    end
+  end
+end

data/spec/system/alongside_file_spec.rb

@@ -161,10 +161,12 @@ describe "Parcels alongside files", :type => :system do
       }
     }
 
+    compiled_sprockets_asset('basic').source # to make sure workaround directories get added
+
     ter = this_example_root
     asset = compiled_sprockets_asset('one')
     asset.should_match(file_assets do
-      asset 'views/.parcels_sprockets_workaround/_parcels_alongside/my_widget.pcss' do
+      asset 'views/my_widget.pcss' do
         expect_rule ".parcels_class__views__my_widget p", 'color: red'
       end
     end)

data/spec/system/class_from_file_spec.rb

@@ -0,0 +1,60 @@
+describe "Parcels determining classes from files", :type => :system do
+  context "with a widget with a non-guessable class name, but at the right point in its path" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        file 'views/foo/bar/baz.rb', %{
+          path = [ "Vie", "oo", "Ba", "az" ]
+          path[0] = path[0] + "ws"
+          path[1] = "F" + path[1]
+          path[2] = path[2] + "r"
+          path[3] = "B" + path[3]
+
+          (0..2).each do |index|
+            module_name = path[0..index].join("::")
+            eval("module \#{module_name}; end")
+          end
+
+          class_name = path.join("::")
+
+          klass = Class.new(::Spec::Fixtures::WidgetBase) do
+            css 'p { color: red; }'
+
+            def content
+              p 'hello!'
+              end
+          end
+
+          mod = eval(path[0..-2].join("::"))
+          mod.const_set(path[-1], klass)
+        }
+
+          # eval("class \#{class_name} < ::Spec::Fixtures::WidgetBase; css 'p { color: red; }'; def content; p 'hello!'; end; end")
+
+        file 'views/foo/bar/baz.pcss', %{
+          div { color: blue; }
+        }
+      }
+    end
+
+    it "should be able to aggregate the CSS correctly still" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/foo/bar/baz.rb' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+
+        asset 'views/foo/bar/baz.pcss' do
+          expect_wrapped_rule :div, 'color: blue'
+        end
+      end)
+    end
+
+    it "should apply the correct autogenerated class to the widget" do
+      doc = render_file_asset('views/foo/bar/baz')
+      expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/foo/bar/baz.rb').parcels_wrapping_class ])
+    end
+  end
+end

data/spec/system/workaround_directories_spec.rb

@@ -0,0 +1,81 @@
+describe "Parcels workaround directories", :type => :system do
+  context "with a simple widget" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget 'views/my_widget' do
+          css %{
+            p { color: red; }
+          }
+        end
+      }
+    end
+
+    def all_workaround_directories_in(base_dir)
+      out = [ ]
+      Find.find(base_dir) do |file|
+        out << file if File.basename(file) =~ /^#{Regexp.escape(::Parcels::Environment::PARCELS_WORKAROUND_DIRECTORY_NAME)}/
+      end
+      out
+    end
+
+    def ensure_asset_compiles!
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.rb' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+      end)
+    end
+
+    def default_widget_trees_to_add_to_sprockets
+      [ ]
+    end
+
+    it "should, by default, put workaround directories inside the widget-tree root" do
+      sprockets_env.parcels.add_widget_tree!(File.join(this_example_root, 'views'))
+      ensure_asset_compiles!
+      expect(all_workaround_directories_in(this_example_root)).to eq([
+        File.join(this_example_root, "views/#{::Parcels::Environment::PARCELS_WORKAROUND_DIRECTORY_NAME}")
+      ])
+    end
+
+    it "should put workaround directories elsewhere if you tell it to" do
+      require 'tmpdir'
+      test_dir = Dir.mktmpdir do |tempdir|
+        sprockets_env.parcels.workaround_directories_root = File.join(tempdir, 'foo', 'bar')
+        sprockets_env.parcels.add_widget_tree!(File.join(this_example_root, 'views'))
+        ensure_asset_compiles!
+
+        expect(all_workaround_directories_in(this_example_root)).to eq([ ])
+        expect(all_workaround_directories_in(tempdir).length).to eq(1)
+        expect(all_workaround_directories_in(tempdir)[0]).to match(
+          /^#{Regexp.escape(File.join(tempdir, 'foo', 'bar', ::Parcels::Environment::PARCELS_WORKAROUND_DIRECTORY_NAME))}_[0-9a-f]{32}$/
+        )
+      end
+    end
+
+    it "should let you re-set the workaround directory location to the same thing as much as you want" do
+      require 'tmpdir'
+      test_dir = Dir.mktmpdir do |tempdir|
+        sprockets_env.parcels.workaround_directories_root = File.join(tempdir, 'foo', 'bar')
+        sprockets_env.parcels.add_widget_tree!(File.join(this_example_root, 'views'))
+        ensure_asset_compiles!
+
+        sprockets_env.parcels.workaround_directories_root = File.join(tempdir, 'foo', 'bar')
+        ensure_asset_compiles!
+      end
+    end
+
+    it "should give you an exception if you try to change the workaround directory location after it's been used" do
+      sprockets_env.parcels.add_widget_tree!(File.join(this_example_root, 'views'))
+      ensure_asset_compiles!
+
+      expect do
+        sprockets_env.parcels.workaround_directories_root = this_example_root
+      end.to raise_exception(/can't set/)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: parcels
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Andrew Geweke
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-15 00:00:00.000000000 Z
+date: 2015-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -148,14 +148,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: 0.9.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: 0.9.2
 description: Allows you to package the CSS and Javascript for your view together with
   that view, creating self-contained parcels of view code that seamlessly work together.
   When used with Fortitude, automatically scopes your CSS, too. No more dueling CSS
@@ -169,6 +169,7 @@ files:
 - ".gitignore"
 - ".rspec-local"
 - ".travis.yml"
+- CHANGES.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -231,6 +232,8 @@ files:
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_and_inline_html.html.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_erb.pcss
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_erb.rb
+- spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module.rb
+- spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module_test.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/inline_erb.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/multiple_engines.pcss
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/multiple_engines.rb
@@ -314,14 +317,17 @@ files:
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_one.rb
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.pcss
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.rb
+- spec/system/ad_hoc_development_mode_spec.rb
 - spec/system/alongside_file_spec.rb
 - spec/system/autogenerated_class_application_spec.rb
 - spec/system/basic_system_spec.rb
+- spec/system/class_from_file_spec.rb
 - spec/system/dependency_sorting_spec.rb
 - spec/system/engines_spec.rb
 - spec/system/prefix_spec.rb
 - spec/system/sass_features_spec.rb
 - spec/system/sets_spec.rb
+- spec/system/workaround_directories_spec.rb
 homepage: https://github.com/ageweke/parcels
 licenses:
 - MIT
@@ -384,6 +390,8 @@ test_files:
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_and_inline_html.html.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_erb.pcss
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/alongside_erb.rb
+- spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module.rb
+- spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/helpers_module_test.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/inline_erb.rb
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/multiple_engines.pcss
 - spec/rails/templates/basic_rails_spec/app/views/basic_rails_spec/multiple_engines.rb
@@ -467,11 +475,15 @@ test_files:
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_one.rb
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.pcss
 - spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.rb
+- spec/system/ad_hoc_development_mode_spec.rb
 - spec/system/alongside_file_spec.rb
 - spec/system/autogenerated_class_application_spec.rb
 - spec/system/basic_system_spec.rb
+- spec/system/class_from_file_spec.rb
 - spec/system/dependency_sorting_spec.rb
 - spec/system/engines_spec.rb
 - spec/system/prefix_spec.rb
 - spec/system/sass_features_spec.rb
 - spec/system/sets_spec.rb
+- spec/system/workaround_directories_spec.rb
+has_rdoc: