stylo 0.5

data/spec/combiner_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+describe Stylo::Combiner do
+  let(:request_path_directory) { 'assets' }
+  let(:combiner) { Stylo::Combiner.new(request_path_directory, /require "(.*)";/) }
+
+  describe "when the content does not contain the require pattern" do
+    let(:content) { "This is some text." }
+
+    it "should return the content" do
+      combiner.process(content).should == content
+    end
+  end
+
+  describe "when the content contains the require pattern" do
+    let(:content) { 'require "some_other_file"; and then some other content' }
+
+    before(:each) do
+      Stylo::AssetLoader.stub(:load_content).with('assets/some_other_file').and_return('the required content.')
+    end
+
+    it "should load required content from the asset loader" do
+      Stylo::AssetLoader.should_receive(:load_content).with('assets/some_other_file').and_return('the required content.')
+
+      combiner.process(content)
+    end
+
+    it "should return the combined content" do
+      combiner.process(content).should == 'the required content. and then some other content'
+    end
+
+    describe "and the required content has the require pattern" do
+      it "should combine the content recursively" do
+        Stylo::AssetLoader.stub(:load_content).with('assets/some_other_file').and_return('require "yet_another_file"; the required content.')
+        Stylo::AssetLoader.stub(:load_content).with('assets/yet_another_file').and_return('the other required content.')
+
+        combiner.process(content).should == "the other required content. the required content. and then some other content"
+      end
+    end
+  end
+end

data/spec/pipeline_steps/caching_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Stylo::PipelineSteps::Caching do
+  describe "call" do
+    describe "when no response has been set" do
+      it "should not add a cache header" do
+        response = Stylo::Response.new('stylesheets/test.css')
+
+        Stylo::PipelineSteps::Caching.new().call(response)
+
+        response.headers.should be_empty
+      end
+    end
+
+    describe "when a response has been set" do
+      it "should add a cache header" do
+        response = Stylo::Response.new('stylesheets/test.css')
+        response.set_body('alert ("hello");', :javascript)
+
+        Stylo::PipelineSteps::Caching.new().call(response)
+
+        response.headers['Cache-Control'].should == 'public, max-age=86400'
+      end
+    end
+  end
+end

data/spec/pipeline_steps/javascript_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe Stylo::PipelineSteps::Javascript do
+  let(:step) { Stylo::PipelineSteps::Javascript.new }
+
+  describe "when the response has not already been set" do
+    describe "and the request is not for a javascript asset" do
+      it "should not set the response" do
+        response = Stylo::Response.new('stylesheets/test.css')
+        Stylo::AssetLoader.stub(:load_content).with(response.path).and_return('some-content')
+
+        step.call(response)
+
+        response.has_content?.should be_false
+      end
+    end
+
+    describe "and the request is for a javascript asset" do
+      let(:response) { Stylo::Response.new('javascripts/test.js') }
+
+      before(:each) do
+
+      end
+
+      it "should ask the asset loader to load the javascript content" do
+        Stylo::AssetLoader.should_receive(:load_content).with(response.path).and_return(nil)
+
+        step.call(response)
+      end
+
+      describe "and the asset does not exist" do
+        it "should not set the response" do
+          Stylo::AssetLoader.stub(:load_content).and_return(nil)
+
+          step.call(response)
+
+          response.has_content?.should be_false
+        end
+      end
+
+      describe "and the asset exists" do
+        let(:javascript_content) { '///require "child.js";' }
+        let(:combined_javascript_content) { "alert('hello world');" }
+        let(:combiner) { mock(:combiner) }
+
+        before(:each) do
+          Stylo::AssetLoader.stub(:load_content).and_return(javascript_content)
+          Stylo::Combiner.stub(:new).with('javascripts', /\/\/\/require "(.*)";/).and_return(combiner)
+          combiner.stub(:process).with(javascript_content).and_return(combined_javascript_content)
+        end
+
+        it "should tell the combiner to process the javascript content" do
+          Stylo::Combiner.should_receive(:new).with('javascripts', /\/\/\/require "(.*)";/).and_return(combiner)
+          combiner.should_receive(:process).with(javascript_content).and_return(combined_javascript_content)
+
+          step.call(response)
+        end
+
+        it "should set the body of the response to the combined javascript content" do
+          step.call(response)
+
+          response.body.should == combined_javascript_content
+        end
+
+        it "should set the content type to text/javascript" do
+          step.call(response)
+
+          response.headers['Content-Type'].should == 'text/javascript'
+        end
+      end
+    end
+  end
+
+  describe "when the response has already been set" do
+    it "should not alter the response" do
+      response = Stylo::Response.new('javascripts/test.js')
+      response.set_body('some-content', :css)
+
+      step.call(response)
+
+      response.body.should == 'some-content'
+    end
+  end
+end

data/spec/pipeline_steps/sass_spec.rb

@@ -0,0 +1,94 @@
+require 'spec_helper'
+
+describe Stylo::PipelineSteps::Sass do
+  let(:step) { Stylo::PipelineSteps::Sass.new }
+
+  describe "when the response has not already been set" do
+    describe "and the request is not for a stylesheet" do
+      it "should not set the response" do
+        response = Stylo::Response.new('javascripts/test.js')
+        Stylo::AssetLoader.stub(:load_content).with(response.path).and_return('some-content')
+
+        step.call(response)
+
+        response.has_content?.should be_false
+      end
+    end
+
+    describe "and the request is for a stylesheet" do
+      let(:requested_path) { 'stylesheets/test.css' }
+      let(:sass_path) { 'stylesheets/test.scss' }
+      let(:response) { Stylo::Response.new(requested_path) }
+
+      it "should ask the asset loader to load the sass stylesheet content" do
+        Stylo::AssetLoader.should_receive(:load_content).with(sass_path).and_return(nil)
+
+        step.call(response)
+      end
+
+      describe "and the asset does not exist" do
+        it "should not set the response" do
+          Stylo::AssetLoader.stub(:load_content).and_return(nil)
+
+          step.call(response)
+
+          response.has_content?.should be_false
+        end
+      end
+
+      describe "and the asset exists" do
+        let(:stylesheet_content) { "html { color: $color; }" }
+        let(:combined_stylesheet_content) { "html { color: $red; }" }
+        let(:processed_content) { "html { color: red; }" }
+        let(:sass_engine) { mock(:sass_engine) }
+        let(:combiner) { mock(:combiner) }
+
+        before(:each) do
+          Stylo::AssetLoader.stub(:load_content).and_return(stylesheet_content)
+          Stylo::Combiner.stub(:new).with('stylesheets', /@import "(.*)";/).and_return(combiner)
+          combiner.stub(:process).with(stylesheet_content).and_return(combined_stylesheet_content)
+          ::Sass::Engine.stub(:new).with(combined_stylesheet_content, {:syntax => :scss}).and_return(sass_engine)
+          sass_engine.stub(:render).and_return(processed_content)
+        end
+
+        it "should tell the combiner to process the stylesheet content" do
+          Stylo::Combiner.should_receive(:new).with('stylesheets', /@import "(.*)";/).and_return(combiner)
+          combiner.should_receive(:process).with(stylesheet_content).and_return(combined_stylesheet_content)
+
+          step.call(response)
+        end
+
+        it "should run the combined content through sass" do
+          ::Sass::Engine.should_receive(:new).with(combined_stylesheet_content, {:syntax => :scss}).and_return(sass_engine)
+          sass_engine.should_receive(:render).and_return(processed_content)
+
+          step.call(response)
+        end
+
+        it "should set the body of the response to the combined stylesheet content" do
+          step.call(response)
+
+          response.body.should == processed_content
+        end
+
+        it "should set the content type to text/css" do
+          step.call(response)
+
+          response.headers['Content-Type'].should == 'text/css'
+        end
+      end
+    end
+  end
+
+  describe "when the response has already been set" do
+    it "should not alter the response" do
+      response = Stylo::Response.new('stylesheets/test.css')
+      response.set_body('some-content', :css)
+
+      step.call(response)
+
+      response.body.should == 'some-content'
+    end
+  end
+
+end

data/spec/pipeline_steps/stylesheet_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe Stylo::PipelineSteps::Stylesheet do
+  let(:step) { Stylo::PipelineSteps::Stylesheet.new }
+
+  describe "when the response has not already been set" do
+    describe "and the request is not for a stylesheet" do
+      it "should not set the response" do
+        response = Stylo::Response.new('javascripts/test.js')
+        Stylo::AssetLoader.stub(:load_content).with(response.path).and_return('some-content')
+
+        step.call(response)
+
+        response.has_content?.should be_false
+      end
+    end
+
+    describe "and the request is for a stylesheet" do
+      let(:response) { Stylo::Response.new('stylesheets/test.css') }
+
+      before(:each) do
+
+      end
+
+      it "should ask the asset loader to load the stylesheet content" do
+        Stylo::AssetLoader.should_receive(:load_content).with(response.path).and_return(nil)
+
+        step.call(response)
+      end
+
+      describe "and the asset does not exist" do
+        it "should not set the response" do
+          Stylo::AssetLoader.stub(:load_content).and_return(nil)
+
+          step.call(response)
+
+          response.has_content?.should be_false
+        end
+      end
+
+      describe "and the asset exists" do
+        let(:stylesheet_content) { "html { color: $color; }" }
+        let(:combined_stylesheet_content) { "html { color: red; }" }
+        let(:combiner) { mock(:combiner) }
+
+        before(:each) do
+          Stylo::AssetLoader.stub(:load_content).and_return(stylesheet_content)
+          Stylo::Combiner.stub(:new).with('stylesheets', /@import "(.*)";/).and_return(combiner)
+          combiner.stub(:process).with(stylesheet_content).and_return(combined_stylesheet_content)
+        end
+
+        it "should tell the combiner to process the stylesheet content" do
+          Stylo::Combiner.should_receive(:new).with('stylesheets', /@import "(.*)";/).and_return(combiner)
+          combiner.should_receive(:process).with(stylesheet_content).and_return(combined_stylesheet_content)
+
+          step.call(response)
+        end
+
+        it "should set the body of the response to the combined stylesheet content" do
+          step.call(response)
+
+          response.body.should == combined_stylesheet_content
+        end
+
+        it "should set the content type to text/css" do
+          step.call(response)
+
+          response.headers['Content-Type'].should == 'text/css'
+        end
+      end
+    end
+  end
+
+  describe "when the response has already been set" do
+    it "should not alter the response" do
+      response = Stylo::Response.new('stylesheets/test.css')
+      response.set_body('some-content', :css)
+
+      step.call(response)
+
+      response.body.should == 'some-content'
+    end
+  end
+end

data/spec/rack_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+describe Stylo::Rack do
+  let(:app) { mock(:app) }
+  let(:env) { {} }
+
+  describe "call" do
+    let(:pipeline_steps) { [] }
+
+    before(:each) do
+      app.stub(:call)
+
+      Stylo::Config.stub(:pipeline).and_return(pipeline_steps)
+    end
+
+    it "should call each pipeline step" do
+      3.times do |i|
+        step = mock("pipeline step #{i}")
+        step.should_receive(:call).with(instance_of(Stylo::Response))
+        pipeline_steps << step
+      end
+
+      Stylo::Rack.new(app).call(env)
+    end
+
+    describe "when no step has been able to deal with the request" do
+      it "should call back into the app" do
+        app.should_receive(:call).with(env)
+
+        Stylo::Rack.new(app).call(env)
+      end
+    end
+
+    describe "when a step has been able to deal with the request" do
+      let(:response) { mock(:response, :has_content? => true) }
+
+      before(:each) do
+        Stylo::Response.stub(:new).and_return(response)
+      end
+
+      it "should not call back into the app" do
+        app.should_not_receive(:call).with(env)
+        response.stub(:build)
+
+        Stylo::Rack.new(app).call(env)
+      end
+
+      it "should return the built response" do
+        response.stub(:build).and_return([200, {}, "some-content"])
+
+        Stylo::Rack.new(app).call(env).should == response.build
+      end
+    end
+  end
+end
+
+def status_code(response)
+  response[0]
+end
+
+def content_type(response)
+  response[1]['Content-Type']
+end
+
+def content_length(response)
+  response[1]['Content-Length']
+end
+
+def cache_control(response)
+  response[1]['Cache-Control']
+end
+
+def content(response)
+  response[2]
+end

data/spec/spec_helper.rb

@@ -0,0 +1,18 @@
+require 'rubygems'
+require 'rspec'
+
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+require 'stylo'
+require 'stylo_spec_helpers'
+
+RSpec.configure do |config|
+  config.include StyloSpecHelpers
+  config.include FileUtils
+
+  config.before(:each) do
+    reset_paths
+  end
+end
+
+

data/spec/stylo_spec_helpers.rb

@@ -0,0 +1,30 @@
+module StyloSpecHelpers
+  def temp_path(path = '')
+    File.expand_path(File.join(File.dirname(__FILE__), '../tmp', path))
+  end
+
+  def fixture_path(path)
+    File.expand_path(File.join(File.dirname(__FILE__), '../features/fixtures', path))
+  end
+
+  def load_fixture(path)
+    File.read fixture_path(path)
+  end
+
+  def write_content(path, content)
+    File.open(path, 'w') do |f|
+      f.write(content)
+    end
+  end
+
+  def reset_paths
+    @stylesheets_path = temp_path('stylesheets')
+    @javascripts_path = temp_path('javascripts')
+    [@stylesheets_path, @javascripts_path].each do |path|
+      rm_rf path
+      mkdir_p path
+    end
+
+    Stylo::Config.asset_location = temp_path()
+  end
+end

data/stylo.gemspec

@@ -0,0 +1,116 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run the gemspec command
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{stylo}
+  s.version = "0.5"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["mwagg"]
+  s.date = %q{2010-10-18}
+  s.description = %q{Server side stylesheet combining for readonly hosting environments}
+  s.email = %q{michael@guerillatactics.co.uk}
+  s.extra_rdoc_files = [
+    "LICENSE",
+     "README.rdoc",
+     "TODO"
+  ]
+  s.files = [
+    ".document",
+     ".gitignore",
+     "LICENSE",
+     "README.rdoc",
+     "Rakefile",
+     "TODO",
+     "cucumber.yml",
+     "features/fixtures.rb",
+     "features/fixtures/child.css",
+     "features/fixtures/child.js",
+     "features/fixtures/grand_parent.css",
+     "features/fixtures/grand_parent.js",
+     "features/fixtures/grand_parent_with_parent_with_child.css",
+     "features/fixtures/grand_parent_with_parent_with_child.js",
+     "features/fixtures/parent.css",
+     "features/fixtures/parent.js",
+     "features/fixtures/parent_with_child.css",
+     "features/fixtures/parent_with_child.js",
+     "features/fixtures/processed_sass_child.css",
+     "features/fixtures/processed_sass_which_uses_mixin.css",
+     "features/fixtures/sass_child.scss",
+     "features/fixtures/sass_mixins.scss",
+     "features/fixtures/sass_which_uses_mixin.scss",
+     "features/javascripts.feature",
+     "features/response_headers.feature",
+     "features/sass_integration.feature",
+     "features/step_definitions/stylo_steps.rb",
+     "features/stylesheets.feature",
+     "features/stylo_cannot_serve_asset.feature",
+     "features/support/env.rb",
+     "lib/stylo.rb",
+     "lib/stylo/asset_loader.rb",
+     "lib/stylo/combiner.rb",
+     "lib/stylo/config.rb",
+     "lib/stylo/pipeline_steps/caching.rb",
+     "lib/stylo/pipeline_steps/javascript.rb",
+     "lib/stylo/pipeline_steps/sass.rb",
+     "lib/stylo/pipeline_steps/stylesheet.rb",
+     "lib/stylo/processor.rb",
+     "lib/stylo/rack.rb",
+     "lib/stylo/railtie.rb",
+     "lib/stylo/response.rb",
+     "spec/asset_loader_spec.rb",
+     "spec/combiner_spec.rb",
+     "spec/pipeline_steps/caching_spec.rb",
+     "spec/pipeline_steps/javascript_spec.rb",
+     "spec/pipeline_steps/sass_spec.rb",
+     "spec/pipeline_steps/stylesheet_spec.rb",
+     "spec/rack_spec.rb",
+     "spec/spec_helper.rb",
+     "spec/stylo_spec_helpers.rb",
+     "stylo.gemspec"
+  ]
+  s.homepage = %q{http://github.com/mwagg/stylo}
+  s.rdoc_options = ["--charset=UTF-8"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.3.7}
+  s.summary = %q{Server side stylesheet combining for readonly hosting environments}
+  s.test_files = [
+    "spec/asset_loader_spec.rb",
+     "spec/combiner_spec.rb",
+     "spec/pipeline_steps/caching_spec.rb",
+     "spec/pipeline_steps/javascript_spec.rb",
+     "spec/pipeline_steps/sass_spec.rb",
+     "spec/pipeline_steps/stylesheet_spec.rb",
+     "spec/rack_spec.rb",
+     "spec/spec_helper.rb",
+     "spec/stylo_spec_helpers.rb"
+  ]
+
+  if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<rspec>, [">= 0"])
+      s.add_development_dependency(%q<cucumber>, [">= 0"])
+      s.add_development_dependency(%q<rack-test>, [">= 0"])
+      s.add_development_dependency(%q<sinatra>, [">= 0"])
+      s.add_runtime_dependency(%q<haml>, [">= 3.0.21"])
+    else
+      s.add_dependency(%q<rspec>, [">= 0"])
+      s.add_dependency(%q<cucumber>, [">= 0"])
+      s.add_dependency(%q<rack-test>, [">= 0"])
+      s.add_dependency(%q<sinatra>, [">= 0"])
+      s.add_dependency(%q<haml>, [">= 3.0.21"])
+    end
+  else
+    s.add_dependency(%q<rspec>, [">= 0"])
+    s.add_dependency(%q<cucumber>, [">= 0"])
+    s.add_dependency(%q<rack-test>, [">= 0"])
+    s.add_dependency(%q<sinatra>, [">= 0"])
+    s.add_dependency(%q<haml>, [">= 3.0.21"])
+  end
+end
+