[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [patch] comment Ruby Binding test and request uncomment when it should pass again

From: Gavin Baumanis <beau_at_palcare.com.au>
Date: Tue, 5 Jan 2010 11:02:48 +1100

Thanks for the update Joe.

Gavin.

On 31/12/2009, at 17:00 , Joe Swatosh wrote:

> Thanks Gavin, but this patch was made moot by philip fixing the
> underlying problem in r891376.
>
> --
> Joe
>
> On Wed, Dec 30, 2009 at 3:51 PM, Gavin 'Beau' Baumanis
> <gavinb_at_thespidernet.com> wrote:
>> Ping.
>> This Submission has received no comments.
>>
>> Gavin.
>>
>>
>> On 13/12/2009, at 03:48 , Joe Swatosh wrote:
>>
>>> [[[
>>> Since the standard Ruby unit test framework has no ability to mark a test as
>>> "XFail," just remove it until it can pass again. Include a request to restore
>>> the test when it should pass again....
>>>
>>> * subversion/bindings/swig/ruby/test/test_client.rb
>>> (SvnClientTest#test_mkdir_p): Comment out
>>>
>>> * subversion/tests/cmdline/basic_tests.py
>>> (test_list): Annotate the "Wimp" test for basic_mkdir_wc_with_parents with
>>> a request to uncomment the Ruby test when functionality is restored.
>>> ]]]
>>>
>>>
>>>
>>> Index: subversion/bindings/swig/ruby/test/test_client.rb
>>> ===================================================================
>>> --- subversion/bindings/swig/ruby/test/test_client.rb (revision 889879)
>>> +++ subversion/bindings/swig/ruby/test/test_client.rb (working copy)
>>> @@ -211,38 +211,38 @@
>>> end
>>> end
>>>
>>> - def test_mkdir_p
>>> - log = "sample log"
>>> - dir = "parent"
>>> - child_dir = "parent/child"
>>> - dir_path = Pathname.new(@wc_path) + dir
>>> - child_dir_path = dir_path + "child"
>>> - full_paths = [dir_path, child_dir_path].collect {|path| path.expand_path}
>>> +# def test_mkdir_p
>>> +# log = "sample log"
>>> +# dir = "parent"
>>> +# child_dir = "parent/child"
>>> +# dir_path = Pathname.new(@wc_path) + dir
>>> +# child_dir_path = dir_path + "child"
>>> +# full_paths = [dir_path, child_dir_path].collect {|path| path.expand_path}
>>> +#
>>> +# make_context(log) do |ctx|
>>> +#
>>> +# infos = []
>>> +# ctx.set_notify_func do |notify|
>>> +# infos << [notify.path, notify]
>>> +# end
>>> +#
>>> +# assert_equal([false, false], [dir_path.exist?, child_dir_path.exist?])
>>> +# ctx.mkdir_p(child_dir_path.to_s)
>>> +# assert_equal(full_paths.collect {|path| path.to_s}.sort,
>>> +# infos.collect{|path, notify| path}.sort)
>>> +# assert_equal([true, true],
>>> +# infos.collect{|path, notify| notify.add?})
>>> +# assert_equal([true, true], [dir_path.exist?, child_dir_path.exist?])
>>> +#
>>> +# infos.clear
>>> +# ctx.commit(@wc_path)
>>> +# assert_equal(full_paths.collect {|path| path.to_s}.sort,
>>> +# infos.collect{|path, notify| path}.sort)
>>> +# assert_equal([true, true],
>>> +# infos.collect{|path, notify| notify.commit_added?})
>>> +# end
>>> +# end
>>>
>>> - make_context(log) do |ctx|
>>> -
>>> - infos = []
>>> - ctx.set_notify_func do |notify|
>>> - infos << [notify.path, notify]
>>> - end
>>> -
>>> - assert_equal([false, false], [dir_path.exist?, child_dir_path.exist?])
>>> - ctx.mkdir_p(child_dir_path.to_s)
>>> - assert_equal(full_paths.collect {|path| path.to_s}.sort,
>>> - infos.collect{|path, notify| path}.sort)
>>> - assert_equal([true, true],
>>> - infos.collect{|path, notify| notify.add?})
>>> - assert_equal([true, true], [dir_path.exist?, child_dir_path.exist?])
>>> -
>>> - infos.clear
>>> - ctx.commit(@wc_path)
>>> - assert_equal(full_paths.collect {|path| path.to_s}.sort,
>>> - infos.collect{|path, notify| path}.sort)
>>> - assert_equal([true, true],
>>> - infos.collect{|path, notify| notify.commit_added?})
>>> - end
>>> - end
>>> -
>>> def test_delete
>>> log = "sample log"
>>> src = "sample source\n"
>>> Index: subversion/tests/cmdline/basic_tests.py
>>> ===================================================================
>>> --- subversion/tests/cmdline/basic_tests.py (revision 889879)
>>> +++ subversion/tests/cmdline/basic_tests.py (working copy)
>>> @@ -2491,6 +2491,9 @@
>>> basic_update,
>>> basic_mkdir_url,
>>> basic_mkdir_url_with_parents,
>>> + # Please uncomment the test_mkdir_p test in
>>> + # subversion/bindings/swig/ruby/test/test_client.rb
>>> + # when un"Wimp"ing this test
>>> Wimp("currently, WC locks are being left behind",
>>> basic_mkdir_wc_with_parents),
>>> basic_corruption,
>>
>>

As always, please contact me if I can be of any further assistance.

Gavin "Beau" Baumanis
Senior Application Developer
PalCare P/L

657 Nicholson Street
Carlton North
Victoria, Australia, 3054

E: beau_at_palcare.com.au
P: +61 -3 9380 3513
M: +61 -438 545 586
W: http://www.palcare.com.au
Received on 2010-01-05 19:46:00 CET

This is an archived mail posted to the Subversion Dev mailing list.