[DRE-commits] [ruby-rabl-rails] 05/07: Remove skipped tests

[Sebastien Badia]

This is an automated email from the git hooks/post-receive script.

sbadia-guest pushed a commit to branch master
in repository ruby-rabl-rails.

commit 88b5658622d7321bf7f961778fab00cddb4f372c
Author: Sebastien Badia <seb at sebian.fr>
Date:   Thu May 1 17:18:35 2014 +0200

    Remove skipped tests
---
 debian/patches/002_remove_skipped_test.patch | 53 ++++++++++++++++++++++++++++
 debian/patches/series                        |  2 ++
 2 files changed, 55 insertions(+)

diff --git a/debian/patches/002_remove_skipped_test.patch b/debian/patches/002_remove_skipped_test.patch
new file mode 100644
index 0000000..0e60d43
--- /dev/null
+++ b/debian/patches/002_remove_skipped_test.patch
@@ -0,0 +1,53 @@
+Description: « extends should not overwrite nodes previously defined »
+ is a skipped test but « skip » is undefined.
+ (NoMethodError: undefined method `skip'), this patch disable this test.
+Author: Sebastien Badia <seb at sebian.fr>
+Forwarded: no
+Last-Update: 2014-05-01
+
+--- ruby-rabl-rails-0.3.4.orig/test/compiler_test.rb
++++ ruby-rabl-rails-0.3.4/test/compiler_test.rb
+@@ -167,22 +167,22 @@ class CompilerTest < ActiveSupport::Test
+     assert_equal({ :id => :id }, t.source)
+   end
+ 
+-  test "extends should not overwrite nodes previously defined" do
+-    skip('Bug reported by @abrisse')
++  #test "extends should not overwrite nodes previously defined" do
++  #  skip('Bug reported by @abrisse')
+ 
+-    template = mock('file_template', :source => %(condition(-> { true }) { 'foo' }))
+-    lookup_context = mock
+-    lookup_context.stub(:find_template).with('users/xtnd', [], false).and_return(template)
+-    RablRails::Library.reset_instance
+-    RablRails::Library.instance.instance_variable_set(:@lookup_context, lookup_context)
++  #  template = mock('file_template', :source => %(condition(-> { true }) { 'foo' }))
++  #  lookup_context = mock
++  #  lookup_context.stub(:find_template).with('users/xtnd', [], false).and_return(template)
++  #  RablRails::Library.reset_instance
++  #  RablRails::Library.instance.instance_variable_set(:@lookup_context, lookup_context)
++
++  #  t = @compiler.compile_source(%{
++  #    condition(-> { false }) { 'bar' }
++  #    extends('users/xtnd')
++  #  })
+ 
+-    t = @compiler.compile_source(%{
+-      condition(-> { false }) { 'bar' }
+-      extends('users/xtnd')
+-    })
+-
+-    assert_equal 2, t.source.keys.size
+-  end
++  #  assert_equal 2, t.source.keys.size
++  #end
+ 
+   test "node are compiled without evaluating the block" do
+     t = @compiler.compile_source(%{ node(:foo) { bar } })
+@@ -230,4 +230,4 @@ class CompilerTest < ActiveSupport::Test
+     assert_equal [:users, :users], @compiler.send(:extract_data_and_name, :users)
+     assert_equal [:@users, :authors], @compiler.send(:extract_data_and_name, :@users => :authors)
+   end
+-end
+\ No newline at end of file
++end
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..f5e0598
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1,2 @@
+001_fix-minitest-autoload.patch
+002_remove_skipped_test.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-ruby-extras/ruby-rabl-rails.git