[ARVADOS] updated: e2da84c1fd9052791ed2b684741469570e09ea35

git at public.curoverse.com git at public.curoverse.com
Mon Feb 9 17:02:04 EST 2015


Summary of changes:
 sdk/ruby/test/test_keep_manifest.rb | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

       via  e2da84c1fd9052791ed2b684741469570e09ea35 (commit)
      from  2527b9cd7958d89a5ae0dd84856027908c48ae53 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit e2da84c1fd9052791ed2b684741469570e09ea35
Author: Brett Smith <brett at curoverse.com>
Date:   Mon Feb 9 17:02:01 2015 -0500

    5176: Add more Ruby SDK tests for empty manifests.
    
    I wrote these while trying to diagnose other issues in the branch.
    They didn't reveal anything directly, but it seems like a waste to
    write tests and not commit them, so here they are.

diff --git a/sdk/ruby/test/test_keep_manifest.rb b/sdk/ruby/test/test_keep_manifest.rb
index 39ce868..50a3d36 100644
--- a/sdk/ruby/test/test_keep_manifest.rb
+++ b/sdk/ruby/test/test_keep_manifest.rb
@@ -57,6 +57,22 @@ class ManifestTest < Minitest::Test
     assert_empty(Keep::Manifest.new("").each_line.to_a)
   end
 
+  def test_empty_each_file_spec
+    assert_empty(Keep::Manifest.new("").each_file_spec.to_a)
+  end
+
+  def test_empty_files
+    assert_empty(Keep::Manifest.new("").files)
+  end
+
+  def test_empty_files_count
+    assert_equal(0, Keep::Manifest.new("").files_count)
+  end
+
+  def test_empty_has_file?
+    refute(Keep::Manifest.new("").has_file?(""))
+  end
+
   def test_empty_line_within_manifest
     block_s = random_block
     manifest = Keep::Manifest.

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list