opal_tag should mirror the javascript_tag api

app/helpers/opal_helper.rb

@@ -1,12 +1,18 @@
 require 'opal/sprockets'
 
 module OpalHelper
-  def opal_tag(opal_code = nil, &block)
-    opal_code ||= capture(&block)
+  def opal_tag(opal_code_or_options = nil, html_options = {}, &block)
+    if block_given?
+      html_options = opal_code_or_options if opal_code_or_options.is_a?(Hash)
+      opal_code_or_options = capture(&block)
+    end
+
     compiler_options = Opal::Config.compiler_options.merge(requirable: false)
-    compiler = Opal::Compiler.new(opal_code, compiler_options)
+    compiler = Opal::Compiler.new(opal_code_or_options, compiler_options)
     js_code = compiler.compile
-    javascript_tag js_code
+    javascript_tag html_options do
+      js_code
+    end
   end
 
   def javascript_include_tag(*sources)

spec/helpers/opal_helper_spec.rb

@@ -6,15 +6,38 @@
   let(:helper) { view }
 
   describe '#opal_tag' do
-    it 'compiles to js' do
-      allow(helper).to receive(:javascript_tag) { |code| code }
-      ruby_code = 'puts 5'
-
-      expect(Opal::Compiler).to receive(:new)
+    let(:ruby_code) { 'puts 5' }
+    let(:compiled_ruby_code) { 'self.$puts(5)' }
+    let(:html_options) { { async: true } }
+    before do
+      allow(helper).to receive(:javascript_tag).and_call_original
+      allow(Opal::Compiler).to receive(:new)
         .with(ruby_code, hash_including(requirable: false))
         .and_call_original
+    end
+
+    context 'when the ruby code is passed inline' do
+      it 'compiles the ruby code to js' do
+        expect(helper.opal_tag(ruby_code)).to include(compiled_ruby_code)
+      end
+
+      it 'passes the html_options to the javascript_tag' do
+        helper.opal_tag(ruby_code, html_options)
+        expect(helper).to have_received(:javascript_tag).with(html_options)
+      end
+    end
+
+    context 'when the ruby code is passed as a block' do
+      it 'compiles the block to js' do
+        expect(helper.opal_tag { ruby_code }).to include(compiled_ruby_code)
+      end
 
-      expect(helper.opal_tag(ruby_code)).to include('self.$puts(5)')
+      it 'uses the options as the first argument' do
+        aggregate_failures do
+          expect(helper.opal_tag(html_options) { ruby_code }).to include(compiled_ruby_code)
+          expect(helper).to have_received(:javascript_tag).with(html_options)
+        end
+      end
     end
   end