RubyGems - premailer-rails - Versions diffs - 1.11.0 → 1.12.0 - Mend

premailer-rails 1.11.0 → 1.12.0

Files changed (92) hide show

checksums.yaml +4 -4
data/.github/workflows/test.yml +64 -0
data/CHANGELOG.md +10 -0
data/Gemfile +13 -13
data/README.md +11 -9
data/VERSION +1 -1
data/{example → examples/rails5}/.gitignore +0 -0
data/examples/rails5/Gemfile +4 -0
data/{example → examples/rails5}/README.md +0 -0
data/{example → examples/rails5}/Rakefile +1 -1
data/{example → examples/rails5}/app/assets/stylesheets/email.css +0 -0
data/{example → examples/rails5}/app/mailers/example_mailer.rb +0 -0
data/{example → examples/rails5}/app/views/example_mailer/test_message.html.erb +0 -0
data/{example → examples/rails5}/bin/rails +1 -1
data/{example → examples/rails5}/config/application.rb +4 -1
data/examples/rails5/config/boot.rb +3 -0
data/{example → examples/rails5}/config/environment.rb +1 -1
data/{example → examples/rails5}/config/environments/development.rb +0 -0
data/{example → examples/rails5}/config/environments/production.rb +0 -0
data/{example → examples/rails5}/config/initializers/assets.rb +0 -0
data/{example → examples/rails5}/config/routes.rb +0 -0
data/{example → examples/rails5}/config.ru +2 -1
data/{example → examples/rails5}/test/mailers/previews/example_mailer_preview.rb +0 -0
data/examples/rails6/.gitignore +16 -0
data/examples/rails6/Gemfile +5 -0
data/examples/rails6/README.md +10 -0
data/examples/rails6/Rakefile +6 -0
data/examples/rails6/app/assets/config/manifest.js +1 -0
data/examples/rails6/app/assets/stylesheets/email.css +32 -0
data/examples/rails6/app/mailers/example_mailer.rb +7 -0
data/examples/rails6/app/views/example_mailer/test_message.html.erb +18 -0
data/examples/rails6/bin/rails +4 -0
data/examples/rails6/config/application.rb +15 -0
data/examples/rails6/config/boot.rb +3 -0
data/examples/rails6/config/environment.rb +5 -0
data/examples/rails6/config/environments/development.rb +9 -0
data/examples/rails6/config/environments/production.rb +9 -0
data/examples/rails6/config/routes.rb +3 -0
data/examples/rails6/config.ru +6 -0
data/examples/rails6/test/mailers/previews/example_mailer_preview.rb +5 -0
data/examples/rails7-propshaft/.gitignore +16 -0
data/examples/rails7-propshaft/Gemfile +6 -0
data/examples/rails7-propshaft/README.md +10 -0
data/examples/rails7-propshaft/Rakefile +6 -0
data/examples/rails7-propshaft/app/assets/stylesheets/email.css +32 -0
data/examples/rails7-propshaft/app/mailers/example_mailer.rb +7 -0
data/examples/rails7-propshaft/app/views/example_mailer/test_message.html.erb +18 -0
data/examples/rails7-propshaft/bin/rails +4 -0
data/examples/rails7-propshaft/config/application.rb +14 -0
data/examples/rails7-propshaft/config/boot.rb +3 -0
data/examples/rails7-propshaft/config/environment.rb +5 -0
data/examples/rails7-propshaft/config/environments/development.rb +9 -0
data/examples/rails7-propshaft/config/environments/production.rb +9 -0
data/examples/rails7-propshaft/config/routes.rb +3 -0
data/examples/rails7-propshaft/config.ru +6 -0
data/examples/rails7-propshaft/test/mailers/previews/example_mailer_preview.rb +5 -0
data/examples/rails7-sprockets/.gitignore +16 -0
data/examples/rails7-sprockets/Gemfile +6 -0
data/examples/rails7-sprockets/README.md +10 -0
data/examples/rails7-sprockets/Rakefile +6 -0
data/examples/rails7-sprockets/app/assets/config/manifest.js +1 -0
data/examples/rails7-sprockets/app/assets/stylesheets/email.css +32 -0
data/examples/rails7-sprockets/app/mailers/example_mailer.rb +7 -0
data/examples/rails7-sprockets/app/views/example_mailer/test_message.html.erb +18 -0
data/examples/rails7-sprockets/bin/rails +4 -0
data/examples/rails7-sprockets/config/application.rb +14 -0
data/examples/rails7-sprockets/config/boot.rb +3 -0
data/examples/rails7-sprockets/config/environment.rb +5 -0
data/examples/rails7-sprockets/config/environments/development.rb +9 -0
data/examples/rails7-sprockets/config/environments/production.rb +9 -0
data/examples/rails7-sprockets/config/routes.rb +3 -0
data/examples/rails7-sprockets/config.ru +6 -0
data/examples/rails7-sprockets/test/mailers/previews/example_mailer_preview.rb +5 -0
data/lib/premailer/rails/css_helper.rb +2 -0
data/lib/premailer/rails/css_loaders/asset_pipeline_loader.rb +5 -4
data/lib/premailer/rails/css_loaders/network_loader.rb +1 -1
data/lib/premailer/rails/css_loaders/propshaft_loader.rb +38 -0
data/lib/premailer/rails/css_loaders.rb +1 -0
data/lib/premailer/rails/railtie.rb +1 -1
data/lib/premailer/rails.rb +1 -1
data/premailer-rails.gemspec +3 -1
data/spec/integration/css_helper_spec.rb +173 -71
data/spec/rails_app/config/application.rb +7 -1
data/spec/spec_helper.rb +0 -13
data/spec/unit/css_loaders/asset_pipeline_loader_spec.rb +44 -32
data/spec/unit/css_loaders/propshaft_loader_spec.rb +57 -0
metadata +91 -56
data/.coveralls.yml +0 -1
data/.travis.yml +0 -16
data/example/Gemfile +0 -10
data/example/config/boot.rb +0 -3
data/example/config/secrets.yml +0 -2

data/examples/rails7-sprockets/.gitignore ADDED Viewed

@@ -0,0 +1,16 @@
+# See https://help.github.com/articles/ignoring-files for more about ignoring files.
+#
+# If you find yourself ignoring temporary files generated by your text editor
+# or operating system, you probably want to add a global ignore instead:
+#   git config --global core.excludesfile '~/.gitignore_global'
+# Ignore bundler config.
+/.bundle
+# Ignore the default SQLite database.
+/db/*.sqlite3
+/db/*.sqlite3-journal
+# Ignore all logfiles and tempfiles.
+/log/*.log
+/tmp

data/examples/rails7-sprockets/Gemfile ADDED Viewed

@@ -0,0 +1,6 @@
+source 'https://rubygems.org'
+gem 'rails', '~> 7.0.4'
+gem 'sprockets-rails'
+gem 'webrick' if RUBY_VERSION >= '3'
+gem 'premailer-rails', path: '../..'

data/examples/rails7-sprockets/README.md ADDED Viewed

@@ -0,0 +1,10 @@
+# Example Rails App
+To run this app, run:
+```shell
+bundle
+bundle exec rails s
+```
+Then point your browser at [http://localhost:3000/](http://localhost:3000/).

data/examples/rails7-sprockets/Rakefile ADDED Viewed

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+require_relative 'config/application'
+Rails.application.load_tasks

data/examples/rails7-sprockets/app/assets/config/manifest.js ADDED Viewed

	@@ -0,0 +1 @@
1	+ //= link email.css

data/examples/rails7-sprockets/app/assets/stylesheets/email.css ADDED Viewed

@@ -0,0 +1,32 @@
+body {
+  background: #efefef;
+  font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;
+}
+p {
+  line-height: 1.4;
+}
+.wrap {
+  max-width: 40em;
+  margin: 0 auto;
+  padding: 1em;
+  background: white;
+}
+.greeting {
+  text-align: center;
+  font-weight: bold;
+  font-size: 110%;
+}
+.footer {
+  font-size: 90%;
+  color: #666;
+}
+a {
+  color: green;
+  text-decoration: none;
+  border-bottom: 2px solid green;
+}

data/examples/rails7-sprockets/app/mailers/example_mailer.rb ADDED Viewed

@@ -0,0 +1,7 @@
+class ExampleMailer < ActionMailer::Base
+  default from: "from@example.com"
+  def test_message
+    mail to: 'to@example.org', subject: 'Test Message'
+  end
+end

data/examples/rails7-sprockets/app/views/example_mailer/test_message.html.erb ADDED Viewed

@@ -0,0 +1,18 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset='utf-8'>
+    <%= stylesheet_link_tag :email %>
+  </head>
+  <body>
+    <div class='wrap'>
+      <p class='greeting'>Hi, John Doe</p>
+      <p>
+        Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.
+      </p>
+      <div class='footer'>
+        To unsubscribe, <a href='http://www.google.com/'>click here</a>.
+      </div>
+    </div>
+  </body>
+</html>

data/examples/rails7-sprockets/bin/rails ADDED Viewed

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+APP_PATH = File.expand_path("../config/application", __dir__)
+require_relative "../config/boot"
+require "rails/commands"

data/examples/rails7-sprockets/config/application.rb ADDED Viewed

@@ -0,0 +1,14 @@
+require_relative 'boot'
+require 'action_controller/railtie'
+require 'action_mailer/railtie'
+# Require the gems listed in Gemfile, including any gems
+# you've limited to :test, :development, or :production.
+Bundler.require(*Rails.groups)
+module Example
+  class Application < Rails::Application
+    config.load_defaults 7.0
+  end
+end

data/examples/rails7-sprockets/config/boot.rb ADDED Viewed

@@ -0,0 +1,3 @@
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)
+require 'bundler/setup' # Set up gems listed in the Gemfile.

data/examples/rails7-sprockets/config/environment.rb ADDED Viewed

@@ -0,0 +1,5 @@
+# Load the Rails application.
+require_relative 'application'
+# Initialize the Rails application.
+Rails.application.initialize!

data/examples/rails7-sprockets/config/environments/development.rb ADDED Viewed

@@ -0,0 +1,9 @@
+Rails.application.configure do
+  config.cache_classes = false
+  config.eager_load = false
+  config.consider_all_requests_local = true
+  config.action_controller.perform_caching = false
+  config.assets.debug = true
+  config.assets.digest = true
+  config.assets.raise_runtime_errors = true
+end

data/examples/rails7-sprockets/config/environments/production.rb ADDED Viewed

@@ -0,0 +1,9 @@
+Rails.application.configure do
+  config.cache_classes = true
+  config.eager_load = true
+  config.consider_all_requests_local = false
+  config.action_controller.perform_caching = true
+  config.assets.compile = false
+  config.assets.digest = true
+  config.log_level = :info
+end

data/examples/rails7-sprockets/config/routes.rb ADDED Viewed

@@ -0,0 +1,3 @@
+Rails.application.routes.draw do
+  root to: redirect('rails/mailers/example_mailer/test_message')
+end

data/examples/rails7-sprockets/config.ru ADDED Viewed

@@ -0,0 +1,6 @@
+# This file is used by Rack-based servers to start the application.
+require_relative 'config/environment'
+run Rails.application
+Rails.application.load_server

data/examples/rails7-sprockets/test/mailers/previews/example_mailer_preview.rb ADDED Viewed

@@ -0,0 +1,5 @@
+class ExampleMailerPreview < ActionMailer::Preview
+  def test_message
+    ExampleMailer.test_message
+  end
+end

data/lib/premailer/rails/css_helper.rb CHANGED Viewed

@@ -57,6 +57,8 @@ class Premailer
           CSSLoaders::FileSystemLoader
         when :asset_pipeline
           CSSLoaders::AssetPipelineLoader
+        when :propshaft
+          CSSLoaders::PropshaftLoader
         when :network
           CSSLoaders::NetworkLoader
         else

data/lib/premailer/rails/css_loaders/asset_pipeline_loader.rb CHANGED Viewed

@@ -13,11 +13,11 @@ class Premailer
         end
         def file_name(url)
-          prefix = [
-            ::Rails.configuration.relative_url_root,
-            ::Rails.configuration.assets.prefix,
+          prefix = File.join(
+            ::Rails.configuration.relative_url_root.to_s,
+            ::Rails.configuration.assets.prefix.to_s,
             '/'
-          ].join
+          )
           URI(url).path
             .sub(/\A#{prefix}/, '')
             .sub(/-(\h{32}|\h{64})\.css\z/, '.css')
@@ -25,6 +25,7 @@ class Premailer
         def asset_pipeline_present?
           defined?(::Rails) &&
+            ::Rails.respond_to?(:application) &&
             ::Rails.application &&
             ::Rails.application.respond_to?(:assets_manifest) &&
             ::Rails.application.assets_manifest

data/lib/premailer/rails/css_loaders/network_loader.rb CHANGED Viewed

@@ -6,7 +6,7 @@ class Premailer
         def load(url)
           uri = uri_for_url(url)
-          Net::HTTP.get(uri) if uri
+          Net::HTTP.get(uri, { 'Accept' => 'text/css' }) if uri
         end
         def uri_for_url(url)

data/lib/premailer/rails/css_loaders/propshaft_loader.rb ADDED Viewed

@@ -0,0 +1,38 @@
+class Premailer
+  module Rails
+    module CSSLoaders
+      module PropshaftLoader
+        extend self
+        def load(url)
+          return unless propshaft_present?
+          file = file_name(url)
+          asset = ::Rails.application.assets.load_path.find(file)
+          ::Rails.application.assets.compilers.compile(asset) if asset
+        end
+        def file_name(url)
+          prefix = File.join(
+            ::Rails.configuration.relative_url_root.to_s,
+            ::Rails.configuration.assets.prefix.to_s,
+            '/'
+          )
+          # Path extraction logic from Propshaft.
+          # See https://github.com/rails/propshaft/blob/390381548b125e8721c8aef1b9d894b7cc8bd868/lib/propshaft/server.rb#L35-L41
+          full_path = URI(url).path.sub(/\A#{prefix}/, '')
+          digest = full_path[/-([0-9a-zA-Z]{7,128}(?:\.digested)?)\.[^.]+\z/, 1]
+          path = digest ? full_path.sub("-#{digest}", "") : full_path
+          path
+        end
+        def propshaft_present?
+          defined?(::Propshaft) && defined?(::Rails)
+        end
+      end
+    end
+  end
+end

data/lib/premailer/rails/css_loaders.rb CHANGED Viewed

@@ -2,4 +2,5 @@ require 'uri'
 require 'premailer/rails/css_loaders/file_system_loader'
 require 'premailer/rails/css_loaders/asset_pipeline_loader'
+require 'premailer/rails/css_loaders/propshaft_loader'
 require 'premailer/rails/css_loaders/network_loader'

data/lib/premailer/rails/railtie.rb CHANGED Viewed

@@ -1,7 +1,7 @@
 class Premailer
   module Rails
     class Railtie < ::Rails::Railtie
-      config.after_initialize do
+      ActiveSupport.on_load(:action_mailer) do
         ::Premailer::Rails.register_interceptors
       end
     end

data/lib/premailer/rails.rb CHANGED Viewed

@@ -12,7 +12,7 @@ class Premailer
     @config = {
       input_encoding: 'UTF-8',
       generate_text_part: true,
-      strategies: [:filesystem, :asset_pipeline, :network]
+      strategies: [:filesystem, :asset_pipeline, :propshaft, :network]
     }
     class << self
       attr_accessor :config

data/premailer-rails.gemspec CHANGED Viewed

@@ -16,6 +16,8 @@ Gem::Specification.new do |s|
                      include a CSS file as you do in a normal HTML document and
                      premailer will inline the included CSS.}
+  s.metadata["changelog_uri"] = "https://github.com/fphilipe/premailer-rails/blob/v#{Premailer::Rails::VERSION}/CHANGELOG.md"
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {example,spec}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
@@ -23,8 +25,8 @@ Gem::Specification.new do |s|
   s.add_dependency 'premailer', '~> 1.7', '>= 1.7.9'
   s.add_dependency 'actionmailer', '>= 3'
+  s.add_dependency 'net-smtp' if RUBY_VERSION >= '3'
   s.add_development_dependency 'rspec', '~> 3.3'
   s.add_development_dependency 'nokogiri'
-  s.add_development_dependency 'coveralls' if RUBY_ENGINE == 'ruby'
 end

data/spec/integration/css_helper_spec.rb CHANGED Viewed

@@ -96,111 +96,213 @@ describe Premailer::Rails::CSSHelper do
       end
     end
-    context 'when Rails asset pipeline is used' do
-      before do
-        allow(Rails.configuration)
-          .to receive(:assets).and_return(double(prefix: '/assets'))
-        allow(Rails.configuration)
-          .to receive(:relative_url_root).and_return(nil)
-      end
+    if defined?(::Sprockets)
+      context 'when Rails asset pipeline is used' do
+        before do
+          allow(Rails.configuration)
+            .to receive(:assets).and_return(double(prefix: '/assets'))
+          allow(Rails.configuration)
+            .to receive(:relative_url_root).and_return(nil)
+        end
-      context 'and a precompiled file exists' do
-        it 'returns that file' do
-          path = '/assets/email-digest.css'
-          content = 'read from file'
-          expect_file("public#{path}", content)
-          expect(css_for_url(path)).to eq(content)
+        context 'and a precompiled file exists' do
+          it 'returns that file' do
+            path = '/assets/email-digest.css'
+            content = 'read from file'
+            expect_file("public#{path}", content)
+            expect(css_for_url(path)).to eq(content)
+          end
         end
-      end
-      context "when find_sources raises TypeError" do
-        let(:response) { 'content of base.css' }
-        let(:uri) { URI('http://example.com/assets/base.css') }
+        context "when find_sources raises TypeError" do
+          let(:response) { 'content of base.css' }
+          let(:uri) { URI('http://example.com/assets/base.css') }
+          it "falls back to Net::HTTP" do
+            expect(Rails.application.assets_manifest).to \
+              receive(:find_sources)
+                .with('base.css')
+                .and_raise(TypeError)
+            allow(Net::HTTP).to \
+              receive(:get)
+                .with(uri, { 'Accept' => 'text/css' })
+                .and_return(response)
+            expect(css_for_url('http://example.com/assets/base.css')).to \
+              eq(response)
+          end
+        end
+        context "when find_sources raises Errno::ENOENT" do
+          let(:response) { 'content of base.css' }
+          let(:uri) { URI('http://example.com/assets/base.css') }
+          it "falls back to Net::HTTP" do
+            expect(Rails.application.assets_manifest).to \
+              receive(:find_sources)
+                .with('base.css')
+                .and_raise(Errno::ENOENT)
+            allow(Net::HTTP).to \
+              receive(:get)
+                .with(uri, { 'Accept' => 'text/css' })
+                .and_return(response)
+            expect(css_for_url('http://example.com/assets/base.css')).to \
+              eq(response)
+          end
+        end
-        it "falls back to Net::HTTP" do
+        it 'returns the content of the file compiled by Rails' do
           expect(Rails.application.assets_manifest).to \
             receive(:find_sources)
               .with('base.css')
-              .and_raise(TypeError)
+              .and_return(['content of base.css'])
-          allow(Net::HTTP).to \
-            receive(:get).with(uri).and_return(response)
           expect(css_for_url('http://example.com/assets/base.css')).to \
-            eq(response)
+            eq('content of base.css')
         end
-      end
-      context "when find_sources raises Errno::ENOENT" do
-        let(:response) { 'content of base.css' }
-        let(:uri) { URI('http://example.com/assets/base.css') }
-        it "falls back to Net::HTTP" do
+        it 'returns same file when path contains file fingerprint' do
           expect(Rails.application.assets_manifest).to \
             receive(:find_sources)
               .with('base.css')
-              .and_raise(Errno::ENOENT)
+              .and_return(['content of base.css'])
-          allow(Net::HTTP).to \
-            receive(:get).with(uri).and_return(response)
-          expect(css_for_url('http://example.com/assets/base.css')).to \
-            eq(response)
+          expect(css_for_url(
+            'http://example.com/assets/base-089e35bd5d84297b8d31ad552e433275.css'
+          )).to eq('content of base.css')
         end
-      end
-      it 'returns the content of the file compiled by Rails' do
-        expect(Rails.application.assets_manifest).to \
-          receive(:find_sources)
-            .with('base.css')
-            .and_return(['content of base.css'])
+        context 'when asset can not be found' do
+          let(:response) { 'content of base.css' }
+          let(:path) { '/assets/base-089e35bd5d84297b8d31ad552e433275.css' }
+          let(:url) { "http://assets.example.com#{path}" }
+          let(:asset_host) { 'http://assets.example.com' }
-        expect(css_for_url('http://example.com/assets/base.css')).to \
-          eq('content of base.css')
-      end
+          before do
+            allow(Rails.application.assets_manifest).to \
+              receive(:find_sources).and_return([])
-      it 'returns same file when path contains file fingerprint' do
-        expect(Rails.application.assets_manifest).to \
-          receive(:find_sources)
-            .with('base.css')
-            .and_return(['content of base.css'])
+            config = double(asset_host: asset_host)
+            allow(Rails.configuration).to \
+              receive(:action_controller).and_return(config)
-        expect(css_for_url(
-          'http://example.com/assets/base-089e35bd5d84297b8d31ad552e433275.css'
-        )).to eq('content of base.css')
-      end
+            allow(Net::HTTP).to \
+              receive(:get)
+                .with(URI(url), { 'Accept' => 'text/css' })
+                .and_return(response)
+          end
+          it 'requests the file' do
+            expect(css_for_url(url)).to eq('content of base.css')
+          end
-      context 'when asset can not be found' do
-        let(:response) { 'content of base.css' }
-        let(:path) { '/assets/base-089e35bd5d84297b8d31ad552e433275.css' }
-        let(:url) { "http://assets.example.com#{path}" }
-        let(:asset_host) { 'http://assets.example.com' }
+          context 'when file url does not include the host' do
+            it 'requests the file using the asset host as host' do
+              expect(css_for_url(path)).to eq('content of base.css')
+            end
+            context 'and the asset host uses protocol relative scheme' do
+              let(:asset_host) { '//assets.example.com' }
+              it 'requests the file using http as the scheme' do
+                expect(css_for_url(path)).to eq('content of base.css')
+              end
+            end
+          end
+        end
+      end
+    elsif defined?(::Propshaft)
+      context 'when Propshaft is used' do
         before do
-          allow(Rails.application.assets_manifest).to \
-            receive(:find_sources).and_return([])
+          allow(Rails.configuration)
+            .to receive(:assets).and_return(double(prefix: '/assets'))
+          allow(Rails.configuration)
+            .to receive(:relative_url_root).and_return(nil)
+        end
+        context 'and a precompiled file exists' do
+          it 'returns that file' do
+            path = '/assets/email-digest.css'
+            content = 'read from file'
+            expect_file("public#{path}", content)
+            expect(css_for_url(path)).to eq(content)
+          end
+        end
+        it 'returns the content of the file compiled by Propshaft' do
+          asset = double()
+          expect(Rails.application.assets.load_path).to \
+            receive(:find)
+              .with('base.css')
+              .and_return(asset)
-          config = double(asset_host: asset_host)
-          allow(Rails.configuration).to \
-            receive(:action_controller).and_return(config)
+          expect(Rails.application.assets.compilers).to \
+            receive(:compile)
+              .with(asset)
+              .and_return('content of base.css')
-          allow(Net::HTTP).to \
-            receive(:get).with(URI(url)).and_return(response)
+          expect(css_for_url('http://example.com/assets/base.css')).to \
+            eq('content of base.css')
         end
-        it 'requests the file' do
-          expect(css_for_url(url)).to eq('content of base.css')
+        it 'returns same file when path contains file fingerprint' do
+          asset = double()
+          expect(Rails.application.assets.load_path).to \
+            receive(:find)
+              .with('base.css')
+              .and_return(asset)
+          expect(Rails.application.assets.compilers).to \
+            receive(:compile)
+              .with(asset)
+              .and_return('content of base.css')
+          expect(css_for_url(
+            'http://example.com/assets/base-089e35bd5d84297b8d31ad552e433275.css'
+          )).to eq('content of base.css')
         end
-        context 'when file url does not include the host' do
-          it 'requests the file using the asset host as host' do
-            expect(css_for_url(path)).to eq('content of base.css')
+        context 'when asset can not be found' do
+          let(:response) { 'content of base.css' }
+          let(:path) { '/assets/base-089e35bd5d84297b8d31ad552e433275.css' }
+          let(:url) { "http://assets.example.com#{path}" }
+          let(:asset_host) { 'http://assets.example.com' }
+          before do
+            expect(Rails.application.assets.load_path).to \
+              receive(:find)
+                .with('base.css')
+                .and_return(nil)
+            config = double(asset_host: asset_host)
+            allow(Rails.configuration).to \
+              receive(:action_controller).and_return(config)
+            allow(Net::HTTP).to \
+              receive(:get)
+                .with(URI(url), { 'Accept' => 'text/css' })
+                .and_return(response)
           end
-          context 'and the asset host uses protocol relative scheme' do
-            let(:asset_host) { '//assets.example.com' }
+          it 'requests the file' do
+            expect(css_for_url(url)).to eq('content of base.css')
+          end
-            it 'requests the file using http as the scheme' do
+          context 'when file url does not include the host' do
+            it 'requests the file using the asset host as host' do
               expect(css_for_url(path)).to eq('content of base.css')
             end
+            context 'and the asset host uses protocol relative scheme' do
+              let(:asset_host) { '//assets.example.com' }
+              it 'requests the file using http as the scheme' do
+                expect(css_for_url(path)).to eq('content of base.css')
+              end
+            end
           end
         end
       end

data/spec/rails_app/config/application.rb CHANGED Viewed

@@ -1,8 +1,14 @@
 require_relative 'boot'
+require "action_controller/railtie"
 require "action_mailer/railtie"
 require "action_view/railtie"
-require "sprockets/railtie"
+begin
+  require "sprockets/railtie"
+rescue LoadError
+end
 require "rails/test_unit/railtie"
 Bundler.require(*Rails.groups)

data/spec/spec_helper.rb CHANGED Viewed

@@ -1,16 +1,3 @@
-if RUBY_ENGINE == 'ruby'
-  if ENV['CI']
-    require 'coveralls'
-    Coveralls::Output.silent = true
-    Coveralls.wear! do
-      add_filter 'spec/'
-    end
-  else
-    require 'simplecov'
-    SimpleCov.start
-  end
-end
 # Configure Rails Environment
 ENV["RAILS_ENV"] = "test"
 require File.expand_path("../../spec/rails_app/config/environment.rb", __FILE__)