History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: JRUBY-1446
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Charles Oliver Nutter
Reporter: Evan Weaver
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
JRuby

FileUtils#mkdir_p causes test suites to not register if the mkdir_p argument already exists

Created: 18/Oct/07 12:28 AM   Updated: 22/Dec/07 06:28 AM
Component/s: Compiler, Core Classes/Modules
Affects Version/s: JRuby 1.0.1
Fix Version/s: JRuby 1.1b1, JRuby 1.0.2

Time Tracking:
Not Specified

Environment: JDK 1.5.0_07, 32-bit Intel OS X 10.4.10, JRuby trunk as of 10/16/07


 Description  « Hide
FileUtils#mkdir_p causes test suites to not register if the mkdir_p
argument already exists. Easily worked around but seems like a bug.
Here's a minimal example:

require 'test/unit'
require 'fileutils'
FileUtils.mkdir_p "some_dir"

class TrueTest < Test::Unit::TestCase
def test_true; assert true; end
end

It works as long as "some_dir" doesn't already exist, mkdir_p is not
called, or mkdir_p is called from within a TrueTest instance method
rather than at load time.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Charles Oliver Nutter - 19/Oct/07 03:38 AM
Remaining fixes need to be done on 1.0 branch and 1.1 compiler. Marking as blocker.

So I remember: the fix to be done is to make the compiler appropriately reset $! to previous exception, which in this case would be nil...and the change made already to the 1.1 interpreter needs to be followed in the compiler and on 1.0 branch.


Charles Oliver Nutter - 19/Oct/07 02:08 PM
Fixed on both trunk and 1.0 branches.