opal · Apr 10, 2014 · Apr 10, 2014 · Apr 10, 2014 · Apr 10, 2014
diff --git a/.rspec b/.rspec
@@ -1,3 +1,3 @@
 --color
 --format progress
---pattern mri_spec/**/*_spec.rb
+--pattern spec/cli/**/*_spec.rb
diff --git a/lib/opal/sprockets/environment.rb b/lib/opal/sprockets/environment.rb
@@ -1,6 +1,15 @@
 require 'sprockets'
+require 'opal/sprockets/processor'
+require 'opal/sprockets/erb'
 
 module Opal
+  # Proccess using Sprockets
+  #
+  #   Opal.process('opal-jquery')   # => String
+  def self.process asset
+    Environment.new[asset].to_s
+  end
+
   # Environment is a subclass of Sprockets::Environment which already has our opal
   # load paths loaded. This makes it easy for stand-alone rack apps, or test runners
   # that have opal load paths ready to use. You can also add an existing gem's lib

diff --git a/lib/opal/sprockets/erb.rb b/lib/opal/sprockets/erb.rb
@@ -1,22 +1,30 @@
 require 'opal'
-require 'opal/compiler'
+require 'tilt'
+require 'opal/erb'
 require 'sprockets'
 
 module Opal
   module ERB
     class Processor < Tilt::Template
+      # vvv BOILERPLATE vvv
       self.default_mime_type = 'application/javascript'
 
       def self.engine_initialized?
         true
       end
 
+      def self.version
+        ::Opal::VERSION
+      end
+
       def initialize_engine
         require_template_library 'opal'
       end
 
       def prepare
       end
+      # ^^^ BOILERPLATE ^^^
+
 
       def evaluate(context, locals, &block)
         context.require_asset 'erb'

diff --git a/lib/opal/sprockets/processor.rb b/lib/opal/sprockets/processor.rb
@@ -5,13 +5,6 @@
 $OPAL_SOURCE_MAPS = {}
 
 module Opal
-  # Proccess using Sprockets
-  #
-  #   Opal.process('opal-jquery')   # => String
-  def self.process asset
-    Environment.new[asset].to_s
-  end
-
   # The Processor class is used to make ruby files (with rb or opal extensions)
   # available to any sprockets based server. Processor will then get passed any
   # ruby source file to build. There are some options you can override globally
@@ -26,6 +19,7 @@ def self.process asset
   #   * irb_enabled                 [false by default]
   #
   class Processor < Tilt::Template
+    # vvv BOILERPLATE vvv
     self.default_mime_type = 'application/javascript'
 
     def self.engine_initialized?
@@ -36,6 +30,15 @@ def self.version
       ::Opal::VERSION
     end
 
+    def initialize_engine
+      require_template_library 'opal'
+    end
+
+    def prepare
+    end
+    # ^^^ BOILERPLATE ^^^
+
+
     class << self
       attr_accessor :method_missing_enabled
       attr_accessor :arity_check_enabled
@@ -60,13 +63,6 @@ def self.stubbed_files
       @stubbed_files ||= Set.new
     end
 
-    def initialize_engine
-      require_template_library 'opal'
-    end
-
-    def prepare
-    end
-
     def evaluate(context, locals, &block)
       options = {
         :method_missing           => self.class.method_missing_enabled,

diff --git a/spec/cli/fixtures/sprockets_file.js.rb b/spec/cli/fixtures/sprockets_file.js.rb
@@ -0,0 +1,3 @@
+require 'opal'
+require 'native'
+puts 'sprockets!'
diff --git a/spec/cli/sprockets/environment_spec.rb b/spec/cli/sprockets/environment_spec.rb
@@ -0,0 +1,14 @@
+require 'cli/spec_helper'
+require 'opal/sprockets/environment'
+
+describe Opal::Environment do
+  let(:env) { described_class.new }
+  let(:logical_path) { 'sprockets_file' }
+
+  before { env.append_path File.expand_path('../../fixtures/', __FILE__) }
+
+  it 'compiles Ruby to JS' do
+    expect(env[logical_path].source).to include('$puts(')
+    expect(env[logical_path+'.js'].source).to include('$puts(')
+  end
+end
diff --git a/spec/cli/sprockets/erb_spec.rb b/spec/cli/sprockets/erb_spec.rb
@@ -0,0 +1,25 @@
+require 'cli/spec_helper'
+require 'opal/sprockets/erb'
+
+describe Opal::ERB::Processor do
+  let(:pathname) { Pathname("/Code/app/mylib/opal/foo.#{ext}") }
+  let(:_context) { double('_context', :logical_path => "foo.#{ext}", :pathname => pathname) }
+  let(:required_assets) { [] }
+  let(:template) { described_class.new { |t| %Q{<a href="<%= url %>"><%= name %></a>} } }
+  before { _context.stub(:require_asset) {|asset| required_assets << asset } }
+
+  let(:ext) { 'opalerb' }
+
+  it "is registered for '.opalerb' files" do
+    expect(Tilt["test.#{ext}"]).to eq(described_class)
+  end
+
+  it 'renders the template' do
+    expect(template.render(_context)).to include('"<a href=\""')
+  end
+
+  it 'implicitly requires "erb"' do
+    template.render(_context)
+    expect(required_assets).to eq(['erb'])
+  end
+end
diff --git a/spec/cli/sprockets/processor_spec.rb b/spec/cli/sprockets/processor_spec.rb
@@ -0,0 +1,28 @@
+require 'cli/spec_helper'
+require 'opal/sprockets/processor'
+
+describe Opal::Processor do
+  let(:pathname) { Pathname("/Code/app/mylib/opal/foo.#{ext}") }
+  let(:_context) { double('_context', :logical_path => "foo.#{ext}", :pathname => pathname, :resolve => pathname.expand_path) }
+
+  %w[rb js.rb opal js.opal].each do |ext|
+    let(:ext) { ext }
+
+    describe %Q{with extension ".#{ext}"} do
+      it "is registered for '.#{ext}' files" do
+        expect(Tilt["test.#{ext}"]).to eq(described_class)
+      end
+
+      it "compiles and evaluates the template on #render" do
+        template = described_class.new { |t| "puts 'Hello, World!'\n" }
+        expect(template.render(_context)).to include('"Hello, World!"')
+      end
+
+      it "can be rendered more than once" do
+        template = described_class.new(_context) { |t| "puts 'Hello, World!'\n" }
+        3.times { expect(template.render(_context)).to include('"Hello, World!"') }
+      end
+    end
+  end
+
+end