parcels 0.0.1

data/spec/rails/templates/sass_rails_spec/lib/more_assets/twenty.scss

@@ -0,0 +1 @@
+$mycolor20: #a0b1c2;

data/spec/rails/templates/sass_rails_spec/vendor/assets/stylesheets/thirteen.scss

@@ -0,0 +1 @@
+$mycolor13: #13abcd;

data/spec/rails/templates/sass_rails_spec/vendor/assets/stylesheets/three.scss

@@ -0,0 +1 @@
+$mycolor3: #9a8b7c;

data/spec/rails/templates/sets_rails_spec/app/assets/stylesheets/admin.css.scss

@@ -0,0 +1 @@
+//= require_parcels admin

data/spec/rails/templates/sets_rails_spec/app/assets/stylesheets/all.css.scss

@@ -0,0 +1 @@
+//= require_parcels

data/spec/rails/templates/sets_rails_spec/app/assets/stylesheets/normal.css.scss

@@ -0,0 +1 @@
+//= require_parcels normal

data/spec/rails/templates/sets_rails_spec/app/controllers/sets_rails_spec_controller.rb

@@ -0,0 +1,3 @@
+class SetsRailsSpecController < ApplicationController
+  # nothing here yet
+end

data/spec/rails/templates/sets_rails_spec/app/views/admin/admin_one.pcss

@@ -0,0 +1 @@
+p.a { color: red; }

data/spec/rails/templates/sets_rails_spec/app/views/admin/admin_one.rb

@@ -0,0 +1,3 @@
+class Views::Admin::AdminOne < Views::Admin::Base
+  css %{p { color: red; }}
+end

data/spec/rails/templates/sets_rails_spec/app/views/admin/admin_two.pcss

@@ -0,0 +1 @@
+div.a { color: green; }

data/spec/rails/templates/sets_rails_spec/app/views/admin/admin_two.rb

@@ -0,0 +1,3 @@
+class Views::Admin::AdminTwo < Views::Root
+  css %{div { color: green; }}
+end

data/spec/rails/templates/sets_rails_spec/app/views/admin/base.rb

@@ -0,0 +1,3 @@
+class Views::Admin::Base < Views::SetsBase
+  parcels_sets :admin
+end

data/spec/rails/templates/sets_rails_spec/app/views/root.rb

@@ -0,0 +1,9 @@
+class Views::Root < Views::Widgets::Base
+  parcels_sets do |klass, filename|
+    if filename =~ /admin/
+      :admin
+    else
+      :normal
+    end
+  end
+end

data/spec/rails/templates/sets_rails_spec/app/views/sets_base.rb

@@ -0,0 +1,3 @@
+class Views::SetsBase < Views::Widgets::Base
+  parcels_sets :normal
+end

data/spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_one.pcss

@@ -0,0 +1 @@
+span.a { color: blue; }

data/spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_one.rb

@@ -0,0 +1,3 @@
+class Views::SetsRailsSpec::NormalOne < Views::SetsBase
+  css %{span { color: blue; }}
+end

data/spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.pcss

@@ -0,0 +1 @@
+em.a { color: yellow; }

data/spec/rails/templates/sets_rails_spec/app/views/sets_rails_spec/normal_two.rb

@@ -0,0 +1,3 @@
+class Views::SetsRailsSpec::NormalTwo < Views::Root
+  css %{em { color: yellow; }}
+end

data/spec/system/alongside_file_spec.rb

@@ -0,0 +1,193 @@
+describe "Parcels alongside files", :type => :system do
+  context "with a very simple alongside file" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget('views/my_widget') { }
+        file   'views/my_widget.pcss', %{
+          p { color: red; }
+        }
+      }
+    end
+
+    it "should apply the correct autogenerated class to the widget" do
+      doc = render_file_asset('views/my_widget')
+      expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+    end
+
+    it "should aggregate the contents of that file" do
+      asset = compiled_sprockets_asset('basic')
+      asset.should_match(file_assets do
+        asset 'views/my_widget.pcss' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+      end)
+    end
+  end
+
+  context "with a widget ending in .rb and an alongside file ending in .html.css" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget('views/my_widget') { }
+        file   'views/my_widget.html.css', %{
+          p { color: red; }
+        }
+      }
+    end
+
+    it "should not aggregate that alongside file" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset_must_not_be_present 'views/my_widget.html.css'
+      end)
+    end
+  end
+
+  context "with a widget ending in .html.rb and an alongside file ending in .css" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget('views/my_widget.html') { }
+        file   'views/my_widget.css', %{
+          p { color: red; }
+        }
+      }
+    end
+
+    it "should not aggregate that alongside file" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset_must_not_be_present 'views/my_widget.css'
+      end)
+    end
+  end
+
+  context "with a widget ending in .html.rb, and an alongside file ending in .html.pcss" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget('views/my_widget.html') { }
+        file   'views/my_widget.html.pcss', %{
+          p { color: red; }
+        }
+      }
+    end
+
+    it "should apply the correct autogenerated class to the widget" do
+      doc = render_file_asset('views/my_widget.html')
+      expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+    end
+
+    it "should aggregate the contents of that file" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.html.pcss' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+      end)
+    end
+  end
+
+  it "should combine an alongside file and inline CSS just fine" do
+    files {
+      file 'assets/basic.css', %{
+        //= require_parcels
+      }
+
+      widget 'views/my_widget' do
+        css %{
+          p { color: green; }
+        }
+      end
+
+      file 'views/my_widget.pcss', %{
+        div { color: blue; }
+      }
+    }
+
+    compiled_sprockets_asset('basic').should_match(file_assets do
+      asset 'views/my_widget.pcss' do
+        expect_wrapped_rule :div, 'color: blue'
+      end
+
+      asset 'views/my_widget.rb' do
+        expect_wrapped_rule :p, 'color: green'
+      end
+    end)
+  end
+
+  it "should not allow you to pick up the alongside file with a direct 'require'" do
+    files {
+      file 'assets/basic.css', %{
+        //= require_parcels
+      }
+
+      file 'assets/one.css', %{
+        //= require 'views/my_widget.pcss'
+      }
+
+      widget('views/my_widget') { }
+      file   'views/my_widget.pcss', %{
+        p { color: red; }
+      }
+    }
+
+    expect { compiled_sprockets_asset('one').source }.to raise_error(::Sprockets::FileNotFound, %r{views/my_widget\.pcss}i)
+  end
+
+  it "should allow you to pick up the alongside file with a 'require' using '_parcels_alongside/', and it should be wrapped" do
+    files {
+      file 'assets/basic.css', %{
+        //= require_parcels
+      }
+
+      file 'assets/one.css', %{
+        //= require _parcels_alongside/my_widget.pcss
+      }
+
+      widget('views/my_widget') { }
+      file   'views/my_widget.pcss', %{
+        p { color: red; }
+      }
+    }
+
+    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
+        expect_rule ".parcels_class__views__my_widget p", 'color: red'
+      end
+    end)
+  end
+
+  it "should not pick up alongside files if there is no corresponding widget class" do
+    files {
+      file 'assets/basic.css', %{
+        //= require_parcels
+      }
+
+      file   'views/my_widget.pcss', %{
+        p { color: red; }
+      }
+
+      file   'views/another_widget.html.pcss', %{
+        div { color: green; }
+      }
+    }
+
+    compiled_sprockets_asset('one').should_match(file_assets do
+      asset_must_not_be_present 'views/my_widget.pcss'
+      asset_must_not_be_present 'views/another_widget.pcss'
+    end)
+  end
+end

data/spec/system/autogenerated_class_application_spec.rb

@@ -0,0 +1,240 @@
+describe "Parcels autogenerated class application", :type => :system do
+  it "should apply the autogenerated class to the outermost element of a widget" do
+    files {
+      widget 'views/my_widget' do
+        css "p { color: red; }"
+      end
+    }
+
+    doc = render_file_asset('views/my_widget')
+
+    expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+  end
+
+  it "should name that class correctly, using double underscores for path separators and single in place of CamelCase" do
+    files {
+      file 'lib/foo.rb', %{
+        module Views::Foo
+        end
+      }
+
+      widget 'views/foo/my_widget' do
+        requires 'lib/foo'
+        css "p { color: red; }"
+      end
+    }
+
+    doc = render_file_asset('views/foo/my_widget')
+    classes = classes_from(doc, 'p')
+
+    expect(classes.length).to eq(1)
+    expect(classes[0]).to match(/views__foo__my_widget/)
+  end
+
+  it "should apply the autogenerated class to multiple outermost elements of a widget" do
+    files {
+      widget 'views/my_widget' do
+        css "p { color: red; }"
+
+        content %{
+          p "hello"
+          span "goodbye"
+        }
+      end
+    }
+
+    doc = render_file_asset('views/my_widget')
+    expected_asset = expected_file_asset('views/my_widget.rb')
+
+    expect(classes_from(doc, 'p')).to eq([ expected_asset.parcels_wrapping_class ])
+    expect(classes_from(doc, 'span')).to eq([ expected_asset.parcels_wrapping_class ])
+  end
+
+  it "should not apply the autogenerated class to elements that aren't outermost" do
+    files {
+      widget 'views/my_widget' do
+        css "p { color: red; }"
+
+        content %{
+          div {
+            p "hello"
+            p "there"
+            p {
+              span "goodbye"
+            }
+          }
+        }
+      end
+    }
+
+    doc = render_file_asset('views/my_widget')
+
+    expect(classes_from(doc, 'div')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+    expect(classes_from(doc, 'div/p[1]')).to be_empty
+    expect(classes_from(doc, 'div/p[2]')).to be_empty
+    expect(classes_from(doc, 'div/p[3]')).to be_empty
+    expect(classes_from(doc, 'div/p/span')).to be_empty
+  end
+
+  it "should not apply any classes if there is no CSS for a widget" do
+    files {
+      widget 'views/my_widget' do
+      end
+    }
+
+    doc = render_file_asset('views/my_widget')
+    expect(classes_from(doc, 'p')).to eq([ ])
+  end
+
+  it "should apply classes if there's CSS in an alongside file" do
+    files {
+      widget 'views/my_widget' do
+      end
+
+      file 'views/my_widget.pcss', %{
+        p { color: red; }
+      }
+    }
+
+    doc = render_file_asset('views/my_widget')
+    expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+  end
+
+  it "should apply autogenerated classes from a parent widget, too" do
+    files {
+      widget 'views/parent_widget' do
+        css "p { color: red; }"
+      end
+
+      widget 'views/child_widget', :superclass => 'Views::ParentWidget' do
+        requires %{views/parent_widget}
+        css "p { color: blue; }"
+      end
+    }
+
+    doc = render_file_asset("views/child_widget")
+    expect(classes_from(doc, 'p').sort).to eq([
+      expected_file_asset('views/parent_widget.rb').parcels_wrapping_class,
+      expected_file_asset('views/child_widget.rb').parcels_wrapping_class
+    ].sort)
+  end
+
+  it "should apply autogenerated classes from a whole parent hierarchy" do
+    files {
+      widget 'views/grandparent_widget' do
+        css "p { color: red; }"
+      end
+
+      widget 'views/parent_widget', :superclass => 'Views::GrandparentWidget' do
+        requires %{views/grandparent_widget}
+        css "p { color: green; }"
+      end
+
+      widget 'views/child_widget', :superclass => 'Views::ParentWidget' do
+        requires %w{views/parent_widget}
+        css "p { color: blue; }"
+      end
+    }
+
+    doc = render_file_asset("views/child_widget")
+    expect(classes_from(doc, 'p').sort).to eq([
+      expected_file_asset('views/grandparent_widget.rb').parcels_wrapping_class,
+      expected_file_asset('views/parent_widget.rb').parcels_wrapping_class,
+      expected_file_asset('views/child_widget.rb').parcels_wrapping_class
+    ].sort)
+  end
+
+  it "should not apply autogenerated classes from a widget in the hierarchy for which there's no CSS" do
+    files {
+      widget 'views/grandparent_widget' do
+        css "p { color: red; }"
+      end
+
+      widget 'views/parent_widget', :superclass => 'Views::GrandparentWidget' do
+        requires %{views/grandparent_widget}
+      end
+
+      widget 'views/child_widget', :superclass => 'Views::ParentWidget' do
+        requires %w{views/parent_widget}
+        css "p { color: blue; }"
+      end
+    }
+
+    doc = render_file_asset("views/child_widget")
+    expect(classes_from(doc, 'p').sort).to eq([
+      expected_file_asset('views/grandparent_widget.rb').parcels_wrapping_class,
+      expected_file_asset('views/child_widget.rb').parcels_wrapping_class
+    ].sort)
+  end
+
+  it "should still apply autogenerated classes if the class has no CSS, but a parent does" do
+    files {
+      widget 'views/parent_widget' do
+        css "p { color: red; }"
+      end
+
+      widget 'views/child_widget', :superclass => 'Views::ParentWidget' do
+        requires %{views/parent_widget}
+      end
+    }
+
+    doc = render_file_asset("views/child_widget")
+    expect(classes_from(doc, 'p')).to eq([
+      expected_file_asset('views/parent_widget.rb').parcels_wrapping_class
+    ])
+  end
+
+  it "should still apply autogenerated classes to a class that enables parcels itself" do
+    files {
+      widget 'views/my_widget', :superclass => 'Fortitude::Widgets::Html5' do
+        class_text 'enable_parcels!'
+
+        content %{
+          p "hello"
+        }
+        css "p { color: red; }"
+      end
+    }
+
+    doc = render_file_asset("views/my_widget")
+    expect(classes_from(doc, 'p')).to eq([
+      expected_file_asset('views/my_widget.rb').parcels_wrapping_class
+    ])
+  end
+
+  it "should stop applying autogenerated classes at a parent widget that isn't enabled for Parcels" do
+    files {
+      widget 'views/grandparent_widget', :superclass => 'Fortitude::Widget' do
+        class_text "doctype :html5"
+
+        content %{
+          p "hello"
+        }
+      end
+
+      widget 'views/parent_widget', :superclass => 'Views::GrandparentWidget' do
+        requires %{views/grandparent_widget}
+        class_text "enable_parcels!"
+        content %{
+          p "hello"
+        }
+        css "p { color: red; }"
+      end
+
+      widget 'views/child_widget', :superclass => 'Views::ParentWidget' do
+        requires %w{views/parent_widget}
+        css "p { color: blue; }"
+
+        content %{
+          div "hello"
+        }
+      end
+    }
+
+    doc = render_file_asset("views/child_widget")
+    expect(classes_from(doc, 'div').sort).to eq([
+      expected_file_asset('views/parent_widget.rb').parcels_wrapping_class,
+      expected_file_asset('views/child_widget.rb').parcels_wrapping_class
+    ].sort)
+  end
+end

data/spec/system/basic_system_spec.rb

@@ -0,0 +1,75 @@
+describe "Parcels basic operations", :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
+
+        file 'views/my_widget.pcss', %{
+          div { color: blue; }
+        }
+      }
+    end
+
+    it "should aggregate the CSS from a simple widget properly" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.rb' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+
+        asset 'views/my_widget.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/my_widget')
+      expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.rb').parcels_wrapping_class ])
+    end
+  end
+
+  context "with a widget ending in .html.rb" do
+    before :each do
+      files {
+        file 'assets/basic.css', %{
+          //= require_parcels
+        }
+
+        widget 'views/my_widget.html' do
+          css %{
+            p { color: red; }
+          }
+        end
+
+        file 'views/my_widget.html.pcss', %{
+          div { color: blue; }
+        }
+      }
+    end
+
+    it "should aggregate the CSS from a simple widget properly" do
+      compiled_sprockets_asset('basic').should_match(file_assets do
+        asset 'views/my_widget.html.rb' do
+          expect_wrapped_rule :p, 'color: red'
+        end
+
+        asset 'views/my_widget.html.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/my_widget.html')
+      expect(classes_from(doc, 'p')).to eq([ expected_file_asset('views/my_widget.html.rb').parcels_wrapping_class ])
+    end
+  end
+end

data/spec/system/dependency_sorting_spec.rb

@@ -0,0 +1,48 @@
+# Superclasses BEFORE subclasses -- it's the last-encountered CSS rule that applies
+describe "Parcels dependency sorting", :type => :system do
+  it "should sort superclasses before subclasses in the generated file" do
+    sequence = (0..20).map { |i| rand(500) }.uniq
+
+    files {
+      file 'assets/basic.css', %{
+        //= require_parcels
+      }
+
+      creation_sequence = (0..(sequence.length - 1)).to_a.shuffle
+
+      creation_sequence.each do |cs|
+        seq = sequence[cs]
+        superclass_num = sequence[cs - 1] if cs > 0
+        superclass = superclass_num ? "Views::Widget#{superclass_num}" : ::Spec::Fixtures::WidgetBase
+
+        widget "views/widget#{seq}", :superclass => superclass do
+          css %{
+            p { color: #0000#{'%02d' % seq}; }
+          }
+        end
+
+        file   "views/widget#{seq}.pcss", %{
+          div { color: #FFFF#{'%02d' % seq}; }
+        }
+      end
+    }
+
+    sequence.each { |s| require File.join(this_example_root, "views/widget#{s}") }
+
+    asset = compiled_sprockets_asset('basic')
+
+    asset.should_match((file_assets do
+      sequence.each do |seq|
+        # It's important that we keep .pcss before .rb here -- because we want CSS in the .rb file (_i.e._, directly
+        # in the widget class itself) to take priority over CSS in the .pcss alongside file.
+        asset "views/widget#{seq}.pcss" do
+          expect_wrapped_rule :div, "color: #FFFF#{'%02d' % seq}"
+        end
+
+        asset "views/widget#{seq}.rb" do
+          expect_wrapped_rule :p, "color: #0000#{'%02d' % seq}"
+        end
+      end
+    end), :ordered => true)
+  end
+end