Always include KM javascript, even if there are no queue items to emit

lib/snogmetrics.rb

@@ -105,22 +105,18 @@ def js!
     # subsequent calls to #js will not send events that have already been sent.
     def js(options={})
       options = {:reset => false}.merge(options)
-
-      if queue.empty?
-        ''
-      else
-        buffer = queue.map { |item| %(_kmq.push(#{item.to_json});) }
-
-        queue.clear if options[:reset]
-
-        <<-JAVASCRIPT
-        <script type="text/javascript">
-        var _kmq = _kmq || [];
-        #{api_js}
-        #{buffer.join("\n")}
-        </script>
-        JAVASCRIPT
-      end
+
+      buffer = queue.map { |item| %(_kmq.push(#{item.to_json});) }
+
+      queue.clear if options[:reset]
+
+      <<-JAVASCRIPT
+      <script type="text/javascript">
+      var _kmq = _kmq || [];
+      #{api_js}
+      #{buffer.join("\n")}
+      </script>
+      JAVASCRIPT
     end
 
   private
@@ -151,4 +147,4 @@ def api_js
       end
     end
   end
-end
+end

spec/snogmetrics_spec.rb

@@ -111,10 +111,6 @@ def self.production?
   end
 
   describe '#js' do
-    it 'outputs nothing if there are no events and no identity' do
-      @context.km.js.should be_empty
-    end
-
     context 'in production' do
       before do
         Rails.stub!(:env).and_return(mock('env', :production? => true))
@@ -135,6 +131,10 @@ def self.production?
         @context.km.js.should_not include('scripts.kissmetrics.com')
       end
     end
+
+    it 'outputs javascript even if there are no events and no identity' do
+      @context.km.js.should include('kmq')
+    end
 
     it 'outputs code that conditionally sets the _kmq variable' do
       @context.km.identify('Phil')
@@ -151,7 +151,7 @@ def self.production?
     it 'resets the session when passed :reset => true' do
       @context.km.record('hello world')
       @context.km.js(:reset => true)
-      @context.km.js.should be_empty
+      @context.km.js.should_not include('hello world')
     end
 
     it 'does not let HTML slip through' do
@@ -164,7 +164,7 @@ def self.production?
     it 'works like #js(:reset => true)' do
       @context.km.record('hello world')
       @context.km.js!
-      @context.km.js.should be_empty
+      @context.km.js!.should_not include('_kmq.push(["record"')
     end
 
     it 'does not push an identify call if the identity has already been sent once' do