jasmine-headless-webkit 0.7.3.2 → 0.8.0.alpha.1

data/skel/template.html.erb

@@ -15,15 +15,25 @@
         log: function(msg) { JHW.stdout.puts(msg) }
       }
 
-      window._onload = window.onload
+      window._onload = window.onload;
 
       window.onload = function() {
-        if (window._onload) { window._onload() }
+        if (window._onload && (window._onload != window.onload)) {
+          window._onload();
+        }
 
         if (window.JHW) {
           jasmine.getEnv().addReporter(new jasmine.HeadlessConsoleReporter());
         } else {
-          jasmine.getEnv().addReporter(new jasmine.HtmlReporter());
+          types = [ 'HtmlReporter', 'TrivialReporter' ];
+
+          for (var i = 0, j = types.length; i < j; ++i) {
+            var type = jasmine[types[i]]
+            if (type) {
+              jasmine.getEnv().addReporter(new type());
+              break;
+            }
+          }
         }
 
         jasmine.getEnv().execute();

data/spec/integration/randomization_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe 'randomize run order' do
+  let(:seed) { 100 }
+
+  it 'should randomize the run order' do
+    output = %x{bin/jasmine-headless-webkit -j spec/jasmine/success/success.yml --seed #{seed}}
+    $?.exitstatus.should == 0
+
+    output.should include("--seed #{seed}")
+  end
+end
+

data/spec/integration/specify_runner_file_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe 'specify runner in jasmine.yml' do
+  before do
+    File.unlink "spec/temp_out.html" if File.file?('spec/temp_out.html')
+  end
+
+  it 'should randomize the run order' do
+    output = %x{bin/jasmine-headless-webkit -j spec/jasmine/runner_out_in_jasmine_yml/jasmine.yml}
+    $?.exitstatus.should == 2
+
+    output.should include("made it")
+    output.should include("1 test")
+
+    "spec/temp_out.html".should be_a_file
+  end
+
+  after do
+    File.unlink "spec/temp_out.html" if File.file?('spec/temp_out.html')
+  end
+end
+

data/spec/integration/sprockets_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe 'sprockets' do
+  it 'should pull in the code via sprockets' do
+    files = %x{bin/jasmine-headless-webkit -l -j spec/jasmine/with_sprockets_includes/with_sprockets_includes.yml}
+    $?.exitstatus.should == 0
+
+    files.lines.to_a.should contain_in_order_in_file_list(
+      'vendor/assets/javascripts/jquery.js',
+      'templates/that.jst.ejs',
+      'templates/this.jst',
+      'assets/things/required.js',
+      'assets/things/code.js',
+      'assets/things/subcode/more_code.js',
+      'spec_helper.js',
+      'spec/things/code_spec.js'
+    )
+  end
+end

data/spec/jasmine/console_log/console_log.yml

@@ -1,5 +1,5 @@
-
 src_files:
+  - spec/javascripts/support/jquery-1.6.2.min.js
   - spec/jasmine/console_log/console_log.js
 
 spec_files:

data/spec/jasmine/runner_out_in_jasmine_yml/jasmine.yml

@@ -0,0 +1,11 @@
+src_dir: spec/jasmine/runner_out_in_jasmine_yml
+spec_dir: spec/jasmine/runner_out_in_jasmine_yml
+
+src_files:
+- "*.js"
+
+spec_files:
+- "*_spec.coffee"
+
+runner_output: 'spec/temp_out.html'
+

data/spec/jasmine/runner_out_in_jasmine_yml/test.js

@@ -0,0 +1,2 @@
+console.log("made it")
+

data/spec/jasmine/runner_out_in_jasmine_yml/test_spec.coffee

@@ -0,0 +1,4 @@
+describe 'this', ->
+  it 'should be true', ->
+    expect(true).toEqual(true)
+

data/spec/jasmine/with_sprockets_includes/assets/application.js.erb

@@ -0,0 +1,2 @@
+//= require 'jquery'
+//= require_tree '.'

data/spec/jasmine/with_sprockets_includes/assets/things/code.js

@@ -0,0 +1,6 @@
+//= require 'jquery'
+//= require_tree './templates'
+//= require 'things/required'
+
+window.a = '1';
+

data/spec/jasmine/with_sprockets_includes/assets/things/required.js

@@ -0,0 +1,2 @@
+//= require 'jquery'
+

data/spec/jasmine/with_sprockets_includes/assets/things/subcode/more_code.js

@@ -0,0 +1,2 @@
+//= require 'things/required'
+

data/spec/jasmine/with_sprockets_includes/assets/things/templates/that.jst.ejs

@@ -0,0 +1 @@
+'hello'

data/spec/jasmine/with_sprockets_includes/assets/things/templates/this.jst

@@ -0,0 +1,2 @@
+'hello'
+

data/spec/jasmine/with_sprockets_includes/spec/spec_helper.js

@@ -0,0 +1 @@
+//= require 'jquery'

data/spec/jasmine/with_sprockets_includes/spec/things/code_spec.js

@@ -0,0 +1,9 @@
+//= require 'things/code'
+
+describe('code', function() {
+  it('should equal 1', function() {
+    expect(window.a).toEqual(1)
+    expect(jQuery).not.toBeUndefined()
+  });
+});
+

data/spec/jasmine/with_sprockets_includes/with_sprockets_includes.yml

@@ -0,0 +1,12 @@
+src_dir: spec/jasmine/with_sprockets_includes/assets
+spec_dir: spec/jasmine/with_sprockets_includes/spec
+
+spec_files:
+  - "**/*_spec.js"
+
+src_files:
+  - "**/*"
+
+helpers:
+  - "spec_helper.js"
+

data/spec/javascripts/console.log_spec.coffee

@@ -0,0 +1,15 @@
+#= require jquery
+
+describe 'console.log', ->
+  it 'should not eat my precious jqueries', ->
+    _log = JHW.log
+
+    JHW.log = ->
+
+    d = $('<div><div id="inner">b</div></div>');
+    expect(d.find('#inner').length).toBe(1);
+    console.log(d.find('#inner'));
+    expect(d.find('#inner').length).toBe(1);
+
+    JHW.log = _log
+

data/spec/javascripts/support/jasmine.yml

@@ -1,4 +1,7 @@
-src_files: [ 'vendor/assets/coffeescripts/*.coffee' ]
+src_files:
+  - 'spec/javascripts/support/jquery-1.6.2.min.js'
+  - 'vendor/assets/coffeescripts/*.coffee'
+
 spec_files: [ 'spec/javascripts/*_spec.coffee' ]
 src_dir: .
 spec_dir: .

data/spec/lib/jasmine/headless/cacheable_action_spec.rb

@@ -3,12 +3,15 @@ require 'spec_helper'
 describe Jasmine::Headless::CacheableAction do
   include FakeFS::SpecHelpers
 
-  let(:file) { 'file.whatever' }
+  let(:file) { 'dir/file.whatever' }
   let(:data) { 'data' }
   let(:compiled) { 'compiled' }
 
   before do
+    FileUtils.mkdir_p File.dirname(file)
+
     File.open(file, 'wb') { |fh| fh.print(data) }
+
     described_class.cache_dir = cache_dir
     described_class.cache_type = cache_type
   end
@@ -19,7 +22,7 @@ describe Jasmine::Headless::CacheableAction do
 
   let(:cache_type) { 'action' }
   let(:cache_dir) { 'cache' }
-  let(:cache_file) { File.join(cache_dir, cache_type, Digest::SHA1.hexdigest(file)) + '.js' }
+  let(:cache_file) { File.join(cache_dir, cache_type, file) + '.js' }
   let(:cache_file_data) { YAML.load(File.read(cache_file)) }
 
   let(:cache_object) { described_class.new(file) }
@@ -45,7 +48,7 @@ describe Jasmine::Headless::CacheableAction do
         FileUtils.mkdir_p(cache_dir)
 
         File.stubs(:mtime).with(file).returns(Time.at(10))
-        File.stubs(:mtime).with(cache_file).returns(Time.at(cache_file_mtime))
+        File.stubs(:mtime).with(File.expand_path(cache_file)).returns(Time.at(cache_file_mtime))
       end
 
       context 'cache empty' do

data/spec/lib/jasmine/headless/coffee_template_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe Jasmine::Headless::CoffeeTemplate do
+  let(:data) { 'data' }
+  let(:path) { 'path.coffee' }
+
+  let(:template) { described_class.new(path) { data } }
+
+  subject { template.render }
+
+  let(:handle_expectation) { Jasmine::Headless::CoffeeScriptCache.any_instance.stubs(:handle) }
+
+  context 'compilation error' do
+    let(:error) { CoffeeScript::CompilationError.new("fail") }
+
+    before do
+      handle_expectation.raises(error)
+    end
+
+    it 'should pass along the error' do
+      expect { subject }.to raise_error(CoffeeScript::CompilationError)
+    end
+  end
+
+  context 'compiles fine' do
+    let(:source) { 'source' }
+
+    before do
+      Jasmine::Headless::CoffeeScriptCache.any_instance.stubs(:cached?).returns(cache_return)
+      handle_expectation.returns(source)
+    end
+
+    context 'cached' do
+      let(:file_path) { 'dir/file.js' }
+      let(:cache_return) { true }
+
+      before do
+        Jasmine::Headless::CoffeeScriptCache.any_instance.stubs(:cache_file).returns(file_path)
+      end
+
+      it 'should return the cached file' do
+        subject.should include(%{<script type="text/javascript" src="#{file_path}"></script>})
+      end
+    end
+
+    context 'not cached' do
+      let(:cache_return) { false }
+
+      it 'should return the generated js' do
+        subject.should include(%{<script type="text/javascript">#{source}</script>})
+      end
+    end
+  end
+end
+

data/spec/lib/jasmine/headless/css_template_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe Jasmine::Headless::CSSTemplate do
+  include FakeFS::SpecHelpers
+
+  let(:template) { described_class.new(file) { data } }
+  let(:file) { 'file' }
+  let(:data) { 'data' }
+
+  subject { template.render }
+
+  before do
+    File.open(file, 'wb') if file
+  end
+
+  context "no file'" do
+    let(:file) { nil }
+
+    it { should == data }
+  end
+
+  context 'file' do
+    it { should == %{<link rel="stylesheet" href="#{file}" type="text/css" />} }
+  end
+end

data/spec/lib/jasmine/headless/files_list_spec.rb

@@ -0,0 +1,203 @@
+require 'spec_helper'
+require 'fakefs/spec_helpers'
+require 'coffee-script'
+
+describe Jasmine::Headless::FilesList do
+  let(:files_list) { described_class.new }
+
+  describe '#initialize' do
+    it "should have default files" do
+      files_list.files.should == [
+        File.join(Jasmine::Core.path, "jasmine.js"),
+        File.join(Jasmine::Core.path, "jasmine-html.js"),
+        File.join(Jasmine::Core.path, "jasmine.css"),
+        File.expand_path('vendor/assets/javascripts/jasmine-extensions.js'),
+        File.expand_path('vendor/assets/javascripts/intense.js'),
+        File.expand_path('vendor/assets/javascripts/headless_reporter_result.js'),
+        File.expand_path('vendor/assets/javascripts/jasmine.HeadlessConsoleReporter.js'),
+        File.expand_path('vendor/assets/javascripts/jsDump.js'),
+        File.expand_path('vendor/assets/javascripts/beautify-html.js'),
+      ]
+    end
+  end
+
+  def self.no_default_files!
+    before do
+      described_class.stubs(:default_files).returns([])
+    end
+  end
+
+  it 'should have tests for #use_config!'
+  it 'should have tests for #add_files'
+
+  describe '#spec_file_line_numbers' do
+    include FakeFS::SpecHelpers
+
+    no_default_files!
+
+    before do
+      files_list.stubs(:spec_files).returns(['test.coffee', 'test2.coffee'])
+
+      File.open('test.coffee', 'w') { |fh| fh.print "describe('cat')\ndescribe('cat')" }
+      File.open('test2.coffee', 'w') { |fh| fh.print "no matches" }
+    end
+
+    it 'should generate filenames and line number info' do
+      files_list.spec_file_line_numbers.should == {
+        'test.coffee' => { 'cat' => [ 1, 2 ] }
+      }
+    end
+  end
+
+  describe '#search_paths' do
+    no_default_files!
+
+    let(:files_list) { described_class.new(:config => config) }
+
+    let(:config) { {
+      'src_dir' => src_dir,
+      'spec_dir' => spec_dir
+    } }
+
+    let(:src_dir) { 'src dir' }
+    let(:spec_dir) { 'spec dir' }
+    let(:path) { 'path' }
+
+    before do
+      Jasmine::Headless::FilesList.stubs(:vendor_asset_paths).returns([])
+    end
+
+    context 'no vendored gem paths' do
+      it 'should take the src dir and spec dirs' do
+        files_list.search_paths.should == [ Jasmine::Core.path, File.expand_path(src_dir), File.expand_path(spec_dir) ]
+      end
+    end
+
+    context 'vendored gem paths' do
+      before do
+        Jasmine::Headless::FilesList.stubs(:vendor_asset_paths).returns([ path ])
+      end
+
+      it 'should add the vendor gem paths to the list' do
+        files_list.search_paths.should == [ Jasmine::Core.path, path, File.expand_path(src_dir), File.expand_path(spec_dir) ]
+      end
+    end
+
+    context 'src_dir is an array' do
+      let(:dir_1) { 'dir 1' }
+      let(:dir_2) { 'dir 2' }
+
+      let(:src_dir) { [ dir_1, dir_2 ] }
+
+      it 'should take the src dir and spec dirs' do
+        files_list.search_paths.should == [ Jasmine::Core.path, File.expand_path(dir_1), File.expand_path(dir_2), File.expand_path(spec_dir) ]
+      end
+    end
+  end
+
+  describe '.vendor_asset_paths' do
+    include FakeFS::SpecHelpers
+
+    let(:dir_one) { 'dir_one' }
+    let(:dir_two) { 'dir_two' }
+
+    let(:gem_one) { stub(:gem_dir => dir_one) }
+    let(:gem_two) { stub(:gem_dir => dir_two) }
+
+    before do
+      described_class.instance_variable_set(:@vendor_asset_paths, nil)
+
+      FileUtils.mkdir_p File.join(dir_two, 'vendor/assets/javascripts')
+
+      Gem::Specification.stubs(:_all).returns([gem_one, gem_two])
+    end
+
+    it 'should return all matching gems with vendor/assets/javascripts directories' do
+      described_class.vendor_asset_paths.should == [ File.join(dir_two, 'vendor/assets/javascripts') ]
+    end
+  end
+
+  describe '#files' do
+    let(:path_one) { 'one' }
+    let(:path_two) { 'two' }
+    let(:path_three) { 'three' }
+
+    let(:file_one) { stub(:to_a => [ asset_one, asset_two ] ) }
+    let(:file_two) { stub(:to_a => [ asset_two, asset_three ] ) }
+
+    let(:asset_one) { stub(:pathname => Pathname(path_one), :to_ary => nil) }
+    let(:asset_two) { stub(:pathname => Pathname(path_two), :to_ary => nil) }
+    let(:asset_three) { stub(:pathname => Pathname(path_three), :to_ary => nil) }
+
+    before do
+      files_list.stubs(:required_files).returns(Jasmine::Headless::UniqueAssetList.new([ file_one, file_two ]))
+    end
+
+    subject { files_list.files }
+
+    it { should == [ path_one, path_two, path_three ] }
+  end
+
+  describe '#filtered_files' do
+    let(:spec_dir) { 'spec' }
+
+    let(:file_one) { "#{spec_dir}/one" }
+    let(:file_two) { "#{spec_dir}/two" }
+    let(:file_three) { "#{spec_dir}/three" }
+    let(:file_four) { 'other/four' }
+
+    before do
+      files_list.stubs(:files).returns([
+        file_one,
+        file_two,
+        file_three,
+        file_four
+      ])
+
+      files_list.stubs(:potential_files_to_filter).returns([ file_one, file_two, file_three ])
+    end
+
+    subject { files_list.filtered_files }
+
+    context 'empty filter' do
+      before do
+        files_list.stubs(:spec_filter).returns([])
+      end
+
+      it { should == [ file_one, file_two, file_three, file_four ] }
+    end
+
+    context 'with filter' do
+      before do
+        files_list.stubs(:spec_filter).returns([ "#{spec_dir}/one", '**/tw*' ])
+      end
+
+      it { should == [ file_one, file_two, file_four ] }
+    end
+  end
+
+  describe '#add_files' do
+    let(:files_list) { described_class.new(:seed => 100) }
+
+    no_default_files!
+
+    let(:dir) { 'tmp' }
+
+    before do
+      FileUtils.mkdir_p dir
+
+      10.times do |index|
+        File.open(File.join(dir, "file-#{index}.js"), 'wb')
+      end
+    end
+
+    it 'should load spec files in a random order' do
+      files_list.send(:add_files, [ '*' ], 'spec_files', [ dir ])
+
+      files_list.files.collect { |name| name[%r{\d+}] }.should == %w{6 7 1 0 5 3 4 8 2 9}
+
+      FileUtils.rm_rf dir
+    end
+  end
+end
+

data/spec/lib/jasmine/headless/js_template_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Jasmine::Headless::JSTemplate do
+  include FakeFS::SpecHelpers
+
+  let(:template) { described_class.new(file) { data } }
+  let(:file) { 'file' }
+  let(:data) { 'data' }
+
+  subject { template.render }
+
+  before do
+    File.open(file, 'wb') if file
+  end
+
+  context "no file'" do
+    let(:file) { nil }
+
+    it { should == data }
+  end
+
+  context 'file' do
+    it { should == %{<script type="text/javascript" src="#{file}"></script>} }
+  end
+
+  context 'script as first thing' do
+    let(:data) { '<script type="text/javascript"' }
+
+    it { should == data }
+  end
+end

data/spec/lib/jasmine/headless/jst_template_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe Jasmine::Headless::JSTTemplate do
+  include FakeFS::SpecHelpers
+
+  let(:template) { described_class.new(file) }
+  let(:file) { 'file' }
+  let(:data) { 'data' }
+
+  let(:context) { stub(:logical_path => 'path') }
+
+  before do
+    File.open(file, 'wb') { |fh| fh.print data }
+  end
+
+  subject { template.render(context) }
+
+  it { should include(%{<script type="text/javascript">}) }
+  it { should include(data) }
+end
+

data/spec/lib/jasmine/headless/options_spec.rb

@@ -92,6 +92,24 @@ describe Jasmine::Headless::Options do
       end
     end
 
+    context 'specify no seed' do
+      it 'should have a seed' do
+        options[:seed].should_not be_nil
+      end
+    end
+
+    context 'specify random order seed' do
+      let(:seed) { 12345 }
+
+      before do
+        ARGV.replace([ "--seed", seed ])
+      end
+
+      it 'should specify the seed' do
+        options[:seed].should == seed
+      end
+    end
+
     after do
       ARGV.replace(@argv)
     end

data/spec/lib/jasmine/headless/template_writer_spec.rb

@@ -47,17 +47,21 @@ describe Jasmine::Headless::TemplateWriter do
     include FakeFS::SpecHelpers
 
     before do
+      Jasmine::Headless::FilesList.stubs(:default_files).returns([])
+
       File.stubs(:read).returns(nil)
 
       runner.stubs(:keep_runner).returns(true)
       runner.stubs(:runner_filename).returns(false)
+
+      Sprockets::Environment.any_instance.stubs(:find_asset).returns(stub(:body => ''))
     end
 
-    let(:files_list) { Jasmine::FilesList.new }
+    let(:files_list) { Jasmine::Headless::FilesList.new }
 
     before do
-      files_list.files << 'file.js'
-      files_list.filtered_files << 'file.js'
+      files_list.stubs(:files).returns([ 'file.js' ])
+      files_list.stubs(:filtered_files).returns([ 'file.js' ])
     end
 
     context 'no filter' do
@@ -70,7 +74,7 @@ describe Jasmine::Headless::TemplateWriter do
 
     context 'filtered files' do
       before do
-        files_list.files << 'file2.js'
+        files_list.stubs(:files).returns([ 'file.js', 'file2.js' ])
       end
 
       it 'should write two files' do

data/spec/lib/jasmine/headless/unique_asset_list_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Jasmine::Headless::UniqueAssetList do
+  let(:list) { described_class.new }
+
+  let(:first) { stub(:logical_path => 'one') }
+  let(:second) { stub(:logical_path => 'two') }
+  let(:third) { stub(:logical_path => 'two') }
+
+  it 'should raise an exception on a non-asset' do
+    expect { list << "whatever" }.to raise_error(StandardError)
+  end
+
+  it 'should not add the same asset with the same logical path twice' do
+    list << first
+    list << second
+    list << third
+
+    list.to_a.should == [ first, second ]
+  end
+end
+

data/spec/lib/jasmine/headless_spec.rb

@@ -0,0 +1,2 @@
+require 'spec_helper'
+