Skip to content

Commit

Permalink
fix merge issue on test_file.rb
Browse files Browse the repository at this point in the history
on master classpath: is synonym to uri:classloader:/

the original test did try to create mkdir_p( classpath:/META-INF/jruby.home ) with
uri:classloader:/ this directory exists as directory so the mkdir_p did nothing.

using files which do produce the same "failures" as before

Sponsored by Lookout Inc.
mkristian committed Jun 9, 2015
1 parent ea9411c commit 833d8e8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/jruby/test_file.rb
Original file line number Diff line number Diff line change
@@ -267,8 +267,8 @@ def test_expand_path_looks_like_url
end

def test_mkdir_with_non_file_uri_raises_error
assert_raises(Errno::ENOTDIR) { FileUtils.mkdir_p("classpath:/META-INF/jruby.home") }
assert !File.directory?("classpath:/META-INF/jruby.home")
assert_raises(Errno::ENOTDIR) { FileUtils.mkdir_p("classpath:/META-INF/MANIFEST.MF") }
assert !File.directory?("classpath:/META-INF/MANIFEST.MF")
end

def test_mkdir_with_file_uri_works_as_expected

0 comments on commit 833d8e8

Please sign in to comment.