opal-rspec 0.5.0.beta2 → 0.5.0.beta3

data/spec/mri/unit/opal/rspec/sprockets_environment_spec.rb

@@ -0,0 +1,94 @@
+require 'rspec'
+require 'opal/rspec/sprockets_environment'
+require_relative 'temp_dir_helper'
+
+describe Opal::RSpec::SprocketsEnvironment do
+  include_context :temp_dir
+  let(:args) { [] }
+  subject(:env) { Opal::RSpec::SprocketsEnvironment.new *args }
+  
+  RSpec::Matchers.define :have_pathnames do |expected|
+    expected = expected.map {|p| File.expand_path(p) }
+    
+    match do |actual|
+      actual == expected
+    end
+  end
+  
+  describe '#cached' do
+    subject { env.cached }
+    
+    it { is_expected.to be_a ::Opal::RSpec::CachedEnvironment }
+  end
+  
+  describe '#add_spec_paths_to_sprockets' do
+    let(:args) { [pattern] }
+    
+    subject do
+      # in subject to allow contexts to execute before logic
+      env.add_spec_paths_to_sprockets
+      env.paths.sort
+    end
+        
+    context 'specs all 1 in path' do
+      before do
+        create_dummy_spec_files 'spec/foobar/dummy_spec.rb', 'spec/foobar/ignored_spec.opal'
+      end
+      
+      let(:pattern) { 'spec/foobar/**/*_spec.rb' }
+      
+      it { is_expected.to have_pathnames ['spec/foobar/'] }
+    end
+    
+    context 'multiple patterns' do
+      before do
+        create_dummy_spec_files 'spec/foobar/dummy_spec.rb', 'spec/foobar/ignored_spec.opal'
+      end
+      
+      let(:pattern) { ['spec/foobar/**/*_spec.rb', 'spec/foobar/**/*_spec.opal'] }
+      
+      it { is_expected.to have_pathnames ['spec/foobar/'] }
+    end
+    
+    context 'specs in different paths, same root' do
+      before do
+        create_dummy_spec_files 'spec/foobar/dummy_spec.rb', 'spec/noway/other_spec.rb'
+      end
+      
+      let(:pattern) { ['spec/foobar/**/*y_spec.rb', 'spec/noway/**/*_spec.rb'] }      
+      
+      it { is_expected.to have_pathnames ['spec'] }
+    end
+    
+    context 'specs in different paths, different root' do
+      before do
+        create_dummy_spec_files 'spec/foobar/dummy_spec.rb', 'other_path/other_spec.rb'
+      end
+      
+      let(:pattern) { ['spec/foobar/**/*_spec.rb', 'other_path/**/*.rb'] }
+      
+      it { is_expected.to have_pathnames ['other_path/', 'spec/foobar/'] }
+    end
+    
+    context 'specs in different paths, same name in middle dirs' do
+      before do
+        create_dummy_spec_files 'foobar/spec/something/dummy_spec.rb', 'spec/foobar/other_spec.rb'
+      end
+      
+      let(:pattern) { ['foobar/spec/**/*_spec.rb', 'spec/foobar/other_spec.rb'] }
+      
+      it { is_expected.to have_pathnames ['foobar/spec/', 'spec/foobar'] }
+    end
+    
+    context 'absolute path and relative path that are not in the same tree' do
+      before do
+        create_dummy_spec_files 'spec/foobar/dummy_spec.rb', 'stuff/bar/other_spec.rb'
+      end
+      
+      let(:files) { FileList['spec/foobar/**/*_spec.rb', 'stuff/bar/other_spec.rb'] }
+      let(:args) { [nil, nil, files] }    
+      
+      it { is_expected.to have_pathnames ['spec/foobar', 'stuff/bar'] }      
+    end   
+  end
+end

data/spec/mri/unit/opal/rspec/temp_dir_helper.rb

@@ -0,0 +1,19 @@
+RSpec.shared_context :temp_dir do
+  before do
+    @temp_dir = Dir.mktmpdir
+    @current_dir = Dir.pwd
+    Dir.chdir @temp_dir
+  end
+  
+  after do
+    Dir.chdir @current_dir
+    FileUtils.rm_rf @temp_dir
+  end
+  
+  def create_dummy_spec_files(*files)
+    files.each do |file|
+      FileUtils.mkdir_p File.dirname(file)
+      FileUtils.touch file
+    end
+  end
+end

data/spec/{around_hooks_spec.rb → opal/around_hooks_spec.rb}

@@ -10,57 +10,57 @@ describe 'hooks' do
         @@around_completed = 0
         @@around_failures = []
       end
-      
+
       after :all do
         raise @@around_failures.join "\n" if @@around_failures.any?
         raise 'hooks not empty!' unless @@around_stack.empty?
         unless @@around_completed == @@expected_around_hits
-          msg = "Expected #{@@expected_around_hits} around hits but got #{@@around_completed} for #{self}"        
+          msg = "Expected #{@@expected_around_hits} around hits but got #{@@around_completed} for #{self}"
           `console.error(#{msg})`
-        end        
-      end    
+        end
+      end
     end
     RSpec.shared_context :async_spec_mix do
-      include_context :around_count     
-      
-      context 'matches' do        
+      include_context :around_count
+
+      context 'matches' do
         it 'async match' do
           delay_with_promise 0 do
             1.should == 1
           end
         end
-        
+
         it 'sync match' do
           1.should == 1
-        end   
-        
+        end
+
         it 'sync fails properly' do
           1.should == 2
         end
-        
+
         it 'another async match' do
           delay_with_promise 0 do
             1.should == 1
           end
         end
-        
+
         it 'async match fails properly' do
           delay_with_promise 0 do
             1.should == 2
           end
-        end     
+        end
       end
-      
+
       context 'before(:each) fails' do
         before do
           raise 'before fails properly'
         end
-        
+
         it 'should not reach the example' do
           fail 'we reached the example and we should not have!'
         end
       end
-      
+
       context 'after(:each)' do
         context 'async' do
           after do
@@ -68,156 +68,97 @@ describe 'hooks' do
               raise 'after fails properly' if raise_after_exception
             end
           end
-          
+
           context 'passes' do
             let(:raise_after_exception) { false }
-        
+
             it 'sync match passes' do
               1.should == 1
-            end            
+            end
           end
-          
+
           context 'fails' do
             let(:raise_after_exception) { true }
-            
+
             it 'sync match passes' do
               1.should == 1
-            end            
+            end
           end
         end
-        
+
         context 'sync' do
           after do
             raise 'after fails properly' if raise_after_exception
           end
-          
+
           context 'passes' do
             let(:raise_after_exception) { false }
-        
+
             it 'sync match passes' do
               1.should == 1
-            end            
+            end
           end
-          
+
           context 'fails' do
             let(:raise_after_exception) { true }
-            
+
             it 'sync match passes' do
               1.should == 1
-            end            
+            end
           end
         end
-      end      
+      end
     end
-    
+
     let(:fail_before_example_run) { false }
     let(:fail_after_example_run) { false }
     let(:skip_run) { false }
-    
+
     context 'sync' do
-      subject { 42 }     
-      
+      subject { 42 }
+
       around do |example|
-        raise 'around failed before example properly' if fail_before_example_run        
+        raise 'around failed before example properly' if fail_before_example_run
         look_for = example.description
-        @@around_stack << look_for        
+        @@around_stack << look_for
         example.run unless skip_run
         last = @@around_stack.pop
         @@around_failures << "Around hook kept executing even though test #{@test_in_progress} was running!" if @test_in_progress
         @@around_failures << "Around hooks are messed up because we expected #{look_for} but we popped off #{last}" unless last == look_for
         @@around_completed += 1
-        raise 'around failed after example properly' if fail_after_example_run       
+        raise 'around failed after example properly' if fail_after_example_run
       end
-      
-      context 'succeeds' do        
+
+      context 'succeeds' do
         before :context do
           @@expected_around_hits = 1
         end
         include_context :around_count
-        
+
         it { is_expected.to equal 42 }
       end
-      
+
       context 'fails before example' do
         before :context do
           @@expected_around_hits = 0
         end
         include_context :around_count
-        
+
         let(:fail_before_example_run) { true }
-        
+
         it { is_expected.to equal 42 }
       end
-      
+
       context 'fails after example' do
         before :context do
           @@expected_around_hits = 1
         end
         include_context :around_count
-        
+
         let(:fail_after_example_run) { true }
-        
+
         it { is_expected.to equal 42 }
       end
     end
-    
-    context 'async' do
-      around do |example|
-        raise 'around failed before example properly' if fail_before_example_run        
-        look_for = example.description
-        @@around_stack << look_for
-        clean_ending = lambda do
-          last = @@around_stack.pop
-          @@around_failures << "Around hook kept executing even though test #{@test_in_progress} was running!" if @test_in_progress
-          @@around_failures << "Around hooks are messed up because we expected #{look_for} but we popped off #{last}" unless last == look_for
-          @@around_completed += 1
-          raise 'around failed after example properly' if fail_after_example_run
-        end
-        if skip_run
-          clean_ending.call
-        else
-          example.run.then do
-            clean_ending.call
-          end        
-        end
-      end
-      
-      context 'succeeds' do
-        before :context do
-          @@expected_around_hits = 10
-        end
-      
-        include_context :async_spec_mix
-      end
-    
-      context 'fails before example' do
-        before :context do
-          @@expected_around_hits = 0
-        end
-      
-        let(:fail_before_example_run) { true }
-      
-        include_context :async_spec_mix
-      end
-    
-      context 'fails after example' do
-        before :context do
-          @@expected_around_hits = 10
-        end
-      
-        let(:fail_after_example_run) { true }
-      
-        include_context :async_spec_mix
-      end
-    
-      context 'skip' do
-        before :context do
-          @@expected_around_hits = 10
-        end
-      
-        let(:skip_run) { true }
-        include_context :async_spec_mix
-      end
-    end   
   end
 end

data/spec/opal/expected_failures.txt

@@ -0,0 +1,40 @@
+promise should make example fail properly before async block reached
+promise matcher fails properly
+promise non-assertion failure in promise no args
+promise non-assertion failure in promise string arg
+promise non-assertion failure in promise exception arg
+pending in example no promise would not fail otherwise, thus fails properly FIXED
+async/sync mix fails properly if a sync test is among async tests
+async/sync mix can finish running after a long delay and fail properly
+be_truthy fails properly with truthy values
+subject sync unnamed assertion fails properly should eq 43
+subject sync unnamed fails properly during subject create
+subject async assertion implicit fails properly should eq 43
+subject async fails properly during creation explicit async
+subject async fails properly during creation implicit usage
+subject async assertion explicit async fails properly
+hooks around sync fails after example should equal 42
+hooks around sync fails before example
+hooks before async with async subject async match fails properly
+hooks before async with async subject before :each fails properly should not reach the example
+hooks before async with async subject before :each succeeds, assertion fails properly should not eq 42
+hooks before async with async subject before :each succeeds, subject fails properly should not reach the example
+hooks before async with async subject both subject and before(:each) fail properly should not reach the example
+hooks before async with sync subject async match fails properly
+hooks before async with sync subject before :each fails properly should not reach the example
+hooks before async with sync subject match fails properly should not eq 42
+hooks before sync with sync subject context fails properly should not reach the example
+hooks before sync with sync subject before :each fails properly should not reach the example
+hooks before sync with sync subject match fails properly should not eq 42
+hooks before sync with sync subject first before :each in chain triggers failure inner context should not reach the example
+hooks after sync after fails should eq 42
+hooks after sync before fails should not reach the example
+hooks after sync match fails async match
+hooks after sync match fails sync match should eq 43
+hooks after async after(:each) fails properly
+hooks after async before(:each) fails properly
+hooks after async match fails properly async match
+hooks after async match fails properly sync match should eq 43
+exception handling should fail properly if an exception is raised
+exception handling should ignore an exception after a failed assertion
+legacy async fails properly after a long delay

data/spec/{subject_spec.rb → opal/subject_spec.rb}

@@ -1,4 +1,4 @@
-describe 'subject' do    
+describe 'subject' do
   context 'sync' do
     context 'named' do
       subject(:named_subject) { [1, 2, 3] }
@@ -9,30 +9,30 @@ describe 'subject' do
 
       it "should be the named subject" do
         subject.should eql(named_subject)
-      end     
-    end    
-    
+      end
+    end
+
     context 'unnamed' do
       subject { 42 }
-    
+
       context 'passes' do
         it { is_expected.to eq 42 }
       end
-    
+
       context 'assertion fails properly' do
         it { is_expected.to eq 43 }
       end
-    
+
       context 'fails properly during subject create' do
         subject do
           raise 'did not work'
         end
-    
+
         it { is_expected.to eq 42 }
-      end      
+      end
     end
   end
-  
+
   context 'async' do
     describe 'assertion' do
       subject do
@@ -40,21 +40,21 @@ describe 'subject' do
           42
         end
       end
-      
+
       context 'explicit async' do
         it 'passes' do
           delay_with_promise 0 do
             expect(subject).to eq 42
           end
         end
-        
+
         it 'fails properly' do
           delay_with_promise 0 do
             expect(subject).to eq 43
           end
         end
       end
-      
+
       context 'implicit' do
         context 'passes' do
           it { is_expected.to eq 42 }
@@ -62,8 +62,8 @@ describe 'subject' do
 
         context 'fails properly' do
           it { is_expected.to eq 43 }
-        end        
-      end      
+        end
+      end
     end
 
     context 'fails properly during creation' do
@@ -76,7 +76,7 @@ describe 'subject' do
       context 'implicit usage' do
         it { is_expected.to eq 42 }
       end
-      
+
       it 'explicit async' do
         delay_with_promise 0 do
           expect(subject).to eq 42