sdk/ruby/test/test_keep_manifest.rb

   1 require "arvados/keep"
   2 require "minitest/autorun"
   3 require "sdk_fixtures"
   4
   5 class ManifestTest < Minitest::Test
   6   include SDKFixtures
   7
   8   def check_stream(stream, exp_name, exp_blocks, exp_files)
   9     assert_equal(exp_name, stream.first)
  10     assert_equal(exp_blocks, stream[1].map(&:to_s))
  11     assert_equal(exp_files, stream.last)
  12   end
  13
  14   def test_simple_each_line_array
  15     manifest = Keep::Manifest.new(SIMPLEST_MANIFEST)
  16     stream_name, block_s, file = SIMPLEST_MANIFEST.strip.split
  17     stream_a = manifest.each_line.to_a
  18     assert_equal(1, stream_a.size, "wrong number of streams")
  19     check_stream(stream_a.first, stream_name, [block_s], [file])
  20   end
  21
  22   def test_simple_each_line_block
  23     manifest = Keep::Manifest.new(SIMPLEST_MANIFEST)
  24     result = []
  25     manifest.each_line do |stream, blocks, files|
  26       result << files
  27     end
  28     assert_equal([[SIMPLEST_MANIFEST.split.last]], result,
  29                  "wrong result from each_line block")
  30   end
  31
  32   def test_multilevel_each_line
  33     manifest = Keep::Manifest.new(MULTILEVEL_MANIFEST)
  34     seen = []
  35     manifest.each_line do |stream, blocks, files|
  36       refute(seen.include?(stream),
  37              "each_line already yielded stream #{stream}")
  38       seen << stream
  39       assert_equal(3, files.size, "wrong file count for stream #{stream}")
  40     end
  41     assert_equal(MULTILEVEL_MANIFEST.count("\n"), seen.size,
  42                  "wrong number of streams")
  43   end
  44
  45   def test_empty_each_line
  46     assert_empty(Keep::Manifest.new("").each_line.to_a)
  47   end
  48
  49   def test_empty_each_file_spec
  50     assert_empty(Keep::Manifest.new("").each_file_spec.to_a)
  51   end
  52
  53   def test_empty_files
  54     assert_empty(Keep::Manifest.new("").files)
  55   end
  56
  57   def test_empty_files_count
  58     assert_equal(0, Keep::Manifest.new("").files_count)
  59   end
  60
  61   def test_empty_files_size
  62     assert_equal(0, Keep::Manifest.new("").files_size)
  63   end
  64
  65   def test_empty_has_file?
  66     refute(Keep::Manifest.new("").has_file?(""))
  67   end
  68
  69   def test_empty_line_within_manifest
  70     block_s = random_block
  71     manifest = Keep::Manifest.
  72       new([". #{block_s} 0:1:file1 1:2:file2\n",
  73            "\n",
  74            ". #{block_s} 3:3:file3 6:4:file4\n"].join(""))
  75     streams = manifest.each_line.to_a
  76     assert_equal(2, streams.size)
  77     check_stream(streams[0], ".", [block_s], ["0:1:file1", "1:2:file2"])
  78     check_stream(streams[1], ".", [block_s], ["3:3:file3", "6:4:file4"])
  79   end
  80
  81   def test_backslash_escape_parsing
  82     manifest = Keep::Manifest.new(MANY_ESCAPES_MANIFEST)
  83     streams = manifest.each_line.to_a
  84     assert_equal(1, streams.size, "wrong number of streams with whitespace")
  85     assert_equal("./dir name", streams.first.first,
  86                  "wrong stream name with whitespace")
  87     assert_equal(["0:9:file\\name\t\\here.txt"], streams.first.last,
  88                  "wrong filename(s) with whitespace")
  89   end
  90
  91   def test_simple_files
  92     manifest = Keep::Manifest.new(SIMPLEST_MANIFEST)
  93     assert_equal([[".", "simple.txt", 9]], manifest.files)
  94   end
  95
  96   def test_multilevel_files
  97     manifest = Keep::Manifest.new(MULTILEVEL_MANIFEST)
  98     seen = Hash.new { |this, key| this[key] = [] }
  99     manifest.files.each do |stream, basename, size|
 100       refute(seen[stream].include?(basename),
 101              "each_file repeated #{stream}/#{basename}")
 102       seen[stream] << basename
 103       assert_equal(3, size, "wrong size for #{stream}/#{basename}")
 104     end
 105     seen.each_pair do |stream, basenames|
 106       assert_equal(%w(file1 file2 file3), basenames.sort,
 107                    "wrong file list for #{stream}")
 108     end
 109   end
 110
 111   def test_files_with_colons_in_names
 112     manifest = Keep::Manifest.new(COLON_FILENAME_MANIFEST)
 113     assert_equal([[".", "file:test.txt", 9]], manifest.files)
 114   end
 115
 116   def test_files_with_escape_sequence_in_filename
 117     manifest = Keep::Manifest.new(ESCAPED_FILENAME_MANIFEST)
 118     assert_equal([[".", "a a.txt", 9]], manifest.files)
 119   end
 120
 121   def test_files_spanning_multiple_blocks
 122     manifest = Keep::Manifest.new(MULTIBLOCK_FILE_MANIFEST)
 123     assert_equal([[".", "repfile", 5],
 124                   [".", "uniqfile", 4],
 125                   [".", "uniqfile2", 7],
 126                   ["./s1", "repfile", 3],
 127                   ["./s1", "uniqfile", 3]],
 128                  manifest.files.sort)
 129   end
 130
 131   def test_minimum_file_count_simple
 132     manifest = Keep::Manifest.new(SIMPLEST_MANIFEST)
 133     assert(manifest.minimum_file_count?(1), "real minimum file count false")
 134     refute(manifest.minimum_file_count?(2), "fake minimum file count true")
 135   end
 136
 137   def test_minimum_file_count_multiblock
 138     manifest = Keep::Manifest.new(MULTIBLOCK_FILE_MANIFEST)
 139     assert(manifest.minimum_file_count?(2), "low minimum file count false")
 140     assert(manifest.minimum_file_count?(5), "real minimum file count false")
 141     refute(manifest.minimum_file_count?(6), "fake minimum file count true")
 142   end
 143
 144   def test_exact_file_count_simple
 145     manifest = Keep::Manifest.new(SIMPLEST_MANIFEST)
 146     assert(manifest.exact_file_count?(1), "exact file count false")
 147     refute(manifest.exact_file_count?(0), "-1 file count true")
 148     refute(manifest.exact_file_count?(2), "+1 file count true")
 149   end
 150
 151   def test_exact_file_count_multiblock
 152     manifest = Keep::Manifest.new(MULTIBLOCK_FILE_MANIFEST)
 153     assert(manifest.exact_file_count?(5), "exact file count false")
 154     refute(manifest.exact_file_count?(4), "-1 file count true")
 155     refute(manifest.exact_file_count?(6), "+1 file count true")
 156   end
 157
 158   def test_files_size_multiblock
 159     assert_equal(22, Keep::Manifest.new(MULTIBLOCK_FILE_MANIFEST).files_size)
 160   end
 161
 162   def test_files_size_with_skipped_overlapping_data
 163     manifest = Keep::Manifest.new(". #{random_block(9)} 3:3:f1 5:3:f2\n")
 164     assert_equal(6, manifest.files_size)
 165   end
 166
 167   def test_has_file
 168     manifest = Keep::Manifest.new(MULTIBLOCK_FILE_MANIFEST)
 169     assert(manifest.has_file?("./repfile"), "one-arg repfile not found")
 170     assert(manifest.has_file?(".", "repfile"), "two-arg repfile not found")
 171     assert(manifest.has_file?("./s1/repfile"), "one-arg s1/repfile not found")
 172     assert(manifest.has_file?("./s1", "repfile"), "two-arg s1/repfile not found")
 173     refute(manifest.has_file?("./s1/uniqfile2"), "one-arg missing file found")
 174     refute(manifest.has_file?("./s1", "uniqfile2"), "two-arg missing file found")
 175     refute(manifest.has_file?("./s2/repfile"), "one-arg missing stream found")
 176     refute(manifest.has_file?("./s2", "repfile"), "two-arg missing stream found")
 177   end
 178
 179   def test_has_file_with_spaces
 180     manifest = Keep::Manifest.new(ESCAPED_FILENAME_MANIFEST)
 181     assert(manifest.has_file?("./a a.txt"), "one-arg path not found")
 182     assert(manifest.has_file?(".", "a a.txt"), "two-arg path not found")
 183     refute(manifest.has_file?("a\\040\\141"), "one-arg unescaped found")
 184     refute(manifest.has_file?(".", "a\\040\\141"), "two-arg unescaped found")
 185   end
 186
 187   def test_parse_all_fixtures
 188     fixtures('collections').each do |name, collection|
 189       parse_collection_manifest name, collection
 190     end
 191   end
 192
 193   def test_raise_on_bogus_fixture
 194     assert_raises ArgumentError do
 195       parse_collection_manifest('bogus collection',
 196                                 {'manifest_text' => ". zzz 0:\n"})
 197     end
 198   end
 199
 200   def parse_collection_manifest name, collection
 201     manifest = Keep::Manifest.new(collection['manifest_text'])
 202     manifest.each_file_spec do |stream_name, start_pos, file_size, file_name|
 203       assert_kind_of String, stream_name
 204       assert_kind_of Integer, start_pos
 205       assert_kind_of Integer, file_size
 206       assert_kind_of String, file_name
 207       assert !stream_name.empty?, "empty stream_name in #{name} fixture"
 208       assert !file_name.empty?, "empty file_name in #{name} fixture"
 209     end
 210   end
 211
 212   def test_collection_with_dirs_in_filenames
 213     manifest = Keep::Manifest.new(MANIFEST_WITH_DIRS_IN_FILENAMES)
 214
 215     seen = Hash.new { |this, key| this[key] = [] }
 216
 217     manifest.files.each do |stream, basename, size|
 218       refute(seen[stream].include?(basename), "each_file repeated #{stream}/#{basename}")
 219       assert_equal(3, size, "wrong size for #{stream}/#{basename}")
 220       seen[stream] << basename
 221     end
 222
 223     assert_equal(%w(. ./dir1 ./dir1/dir2), seen.keys)
 224
 225     seen.each_pair do |stream, basenames|
 226       assert_equal(%w(file1), basenames.sort, "wrong file list for #{stream}")
 227     end
 228   end
 229
 230   def test_multilevel_collection_with_dirs_in_filenames
 231     manifest = Keep::Manifest.new(MULTILEVEL_MANIFEST_WITH_DIRS_IN_FILENAMES)
 232
 233     seen = Hash.new { |this, key| this[key] = [] }
 234     expected_sizes = {'.' => 3, './dir1' => 6, './dir1/dir2' => 11}
 235
 236     manifest.files.each do |stream, basename, size|
 237       refute(seen[stream].include?(basename), "each_file repeated #{stream}/#{basename}")
 238       assert_equal(expected_sizes[stream], size, "wrong size for #{stream}/#{basename}")
 239       seen[stream] << basename
 240     end
 241
 242     assert_equal(%w(. ./dir1 ./dir1/dir2), seen.keys)
 243
 244     seen.each_pair do |stream, basenames|
 245       assert_equal(%w(file1), basenames.sort, "wrong file list for #{stream}")
 246     end
 247   end
 248
 249   [[false, nil],
 250    [false, '+0'],
 251    [false, 'd41d8cd98f00b204e9800998ecf8427'],
 252    [false, 'd41d8cd98f00b204e9800998ecf8427+0'],
 253    [false, 'd41d8cd98f00b204e9800998ecf8427e0'],
 254    [false, 'd41d8cd98f00b204e9800998ecf8427e0+0'],
 255    [false, 'd41d8cd98f00b204e9800998ecf8427e+0 '],
 256    [false, "d41d8cd98f00b204e9800998ecf8427e+0\n"],
 257    [false, ' d41d8cd98f00b204e9800998ecf8427e+0'],
 258    [false, 'd41d8cd98f00b204e9800998ecf8427e+K+0'],
 259    [false, 'd41d8cd98f00b204e9800998ecf8427e+0+0'],
 260    [false, 'd41d8cd98f00b204e9800998ecf8427e++'],
 261    [false, 'd41d8cd98f00b204e9800998ecf8427e+0+K+'],
 262    [false, 'd41d8cd98f00b204e9800998ecf8427e+0++K'],
 263    [false, 'd41d8cd98f00b204e9800998ecf8427e+0+K++'],
 264    [false, 'd41d8cd98f00b204e9800998ecf8427e+0+K++Z'],
 265    [true, 'd41d8cd98f00b204e9800998ecf8427e', nil,nil,nil],
 266    [true, 'd41d8cd98f00b204e9800998ecf8427e+0', '+0','0',nil],
 267    [true, 'd41d8cd98f00b204e9800998ecf8427e+0+Fizz+Buzz','+0','0','+Fizz+Buzz'],
 268    [true, 'd41d8cd98f00b204e9800998ecf8427e+Fizz+Buzz', nil,nil,'+Fizz+Buzz'],
 269    [true, 'd41d8cd98f00b204e9800998ecf8427e+0+Ad41d8cd98f00b204e9800998ecf8427e00000000+Foo', '+0','0','+Ad41d8cd98f00b204e9800998ecf8427e00000000+Foo'],
 270    [true, 'd41d8cd98f00b204e9800998ecf8427e+Ad41d8cd98f00b204e9800998ecf8427e00000000+Foo', nil,nil,'+Ad41d8cd98f00b204e9800998ecf8427e00000000+Foo'],
 271    [true, 'd41d8cd98f00b204e9800998ecf8427e+0+Z', '+0','0','+Z'],
 272    [true, 'd41d8cd98f00b204e9800998ecf8427e+Z', nil,nil,'+Z'],
 273   ].each do |ok, locator, match2, match3, match4|
 274     define_method "test_LOCATOR_REGEXP_on_#{locator.inspect}" do
 275       match = Keep::Locator::LOCATOR_REGEXP.match locator
 276       assert_equal ok, !!match
 277       if ok
 278         assert_equal match2, match[2]
 279         assert_equal match3, match[3]
 280         assert_equal match4, match[4]
 281       end
 282     end
 283     define_method "test_parse_method_on_#{locator.inspect}" do
 284       loc = Keep::Locator.parse locator
 285       if !ok
 286         assert_nil loc
 287       else
 288         refute_nil loc
 289         assert loc.is_a?(Keep::Locator)
 290         #assert loc.hash
 291         #assert loc.size
 292         #assert loc.hints.is_a?(Array)
 293       end
 294     end
 295   end
 296
 297   [
 298     [false, nil, "No manifest found"],
 299     [true, ""],
 300     [false, " ", "Invalid manifest: does not end with newline"],
 301     [true, ". d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n"],
 302     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n"],
 303     [true, ". d41d8cd98f00b204e9800998ecf8427e a41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n"], # 2 locators
 304     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/bar.txt\n"],
 305     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:.foo.txt\n"],
 306     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:.foo\n"],
 307     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:...\n"],
 308     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:.../.foo./.../bar\n"],
 309     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/...\n"],
 310     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/.../bar\n"],
 311     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/.bar/baz.txt\n"],
 312     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/bar./baz.txt\n"],
 313     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 000000000000000000000000000000:0777:foo.txt\n"],
 314     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:0:0\n"],
 315     [true, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:\\040\n"],
 316     [true, ". 00000000000000000000000000000000+0 0:0:0\n"],
 317     [true, ". 00000000000000000000000000000000+0 0:0:d41d8cd98f00b204e9800998ecf8427e+0+Ad41d8cd98f00b204e9800998ecf8427e00000000@ffffffff\n"],
 318     [true, ". d41d8cd98f00b204e9800998ecf8427e+0+Ad41d8cd98f00b204e9800998ecf8427e00000000@ffffffff 0:0:empty.txt\n"],
 319     [false, '. d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt',
 320       "Invalid manifest: does not end with newline"],
 321     [false, "abc d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 322       "invalid stream name \"abc\""],
 323     [false, "abc/./foo d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 324       "invalid stream name \"abc/./foo\""],
 325     [false, "./abc/../foo d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 326       "invalid stream name \"./abc/../foo\""],
 327     [false, "./abc/. d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 328       "invalid stream name \"./abc/.\""],
 329     [false, "./abc/.. d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 330       "invalid stream name \"./abc/..\""],
 331     [false, "./abc/./foo d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 332       "invalid stream name \"./abc/./foo\""],
 333     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:.\n",
 334       "invalid file token \"0:0:.\""],
 335     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:..\n",
 336       "invalid file token \"0:0:..\""],
 337     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:./abc.txt\n",
 338       "invalid file token \"0:0:./abc.txt\""],
 339     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:../abc.txt\n",
 340       "invalid file token \"0:0:../abc.txt\""],
 341     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt/.\n",
 342       "invalid file token \"0:0:abc.txt/.\""],
 343     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt/..\n",
 344       "invalid file token \"0:0:abc.txt/..\""],
 345     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:a/./bc.txt\n",
 346       "invalid file token \"0:0:a/./bc.txt\""],
 347     [false, ". d41d8cd98f00b204e9800998ecf8427e 0:0:a/../bc.txt\n",
 348       "invalid file token \"0:0:a/../bc.txt\""],
 349     [false, "./abc/./foo d41d8cd98f00b204e9800998ecf8427e 0:0:abc.txt\n",
 350       "invalid stream name \"./abc/./foo\""],
 351     [false, "d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n",
 352       "invalid stream name \"d41d8cd98f00b204e9800998ecf8427e+0\""],
 353     [false, ". d41d8cd98f00b204e9800998ecf8427 0:0:abc.txt\n",
 354       "invalid locator \"d41d8cd98f00b204e9800998ecf8427\""],
 355     [false, ". d41d8cd98f00b204e9800998ecf8427e\n",
 356       "Manifest invalid for stream 1: no file tokens"],
 357     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n/dir1 d41d8cd98f00b204e9800998ecf842 0:0:abc.txt\n",
 358       "Manifest invalid for stream 2: missing or invalid stream name \"/dir1\""],
 359     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n./dir1 d41d8cd98f00b204e9800998ecf842 0:0:abc.txt\n",
 360       "Manifest invalid for stream 2: missing or invalid locator \"d41d8cd98f00b204e9800998ecf842\""],
 361     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n./dir1 a41d8cd98f00b204e9800998ecf8427e+0 abc.txt\n",
 362       "Manifest invalid for stream 2: invalid file token \"abc.txt\""],
 363     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n./dir1 a41d8cd98f00b204e9800998ecf8427e+0 0:abc.txt\n",
 364       "Manifest invalid for stream 2: invalid file token \"0:abc.txt\""],
 365     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt\n./dir1 a41d8cd98f00b204e9800998ecf8427e+0 0:0:abc.txt xyz.txt\n",
 366       "Manifest invalid for stream 2: invalid file token \"xyz.txt\""],
 367     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt d41d8cd98f00b204e9800998ecf8427e+0\n",
 368       "Manifest invalid for stream 1: invalid file token \"d41d8cd98f00b204e9800998ecf8427e+0\""],
 369     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:\n",
 370       "Manifest invalid for stream 1: invalid file token \"0:0:\""],
 371     [false, ". d41d8cd98f00b204e9800998ecf8427e+0\n",
 372       "Manifest invalid for stream 1: no file tokens"],
 373     [false, ". 0:0:foo.txt d41d8cd98f00b204e9800998ecf8427e+0\n",
 374       "Manifest invalid for stream 1: missing or invalid locator \"0:0:foo.txt\""],
 375     [false, ". 0:0:foo.txt\n",
 376       "Manifest invalid for stream 1: missing or invalid locator \"0:0:foo.txt\""],
 377     [false, ".\n", "Manifest invalid for stream 1: missing or invalid locator"],
 378     [false, ".", "Invalid manifest: does not end with newline"],
 379     [false, ". \n", "Manifest invalid for stream 1: missing or invalid locator"],
 380     [false, ".  \n", "Manifest invalid for stream 1: missing or invalid locator"],
 381     [false, " . d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 382       "Manifest invalid for stream 1: missing or invalid stream name"],
 383     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt \n",
 384       "stream 1: trailing space"],
 385    # TAB and other tricky whitespace characters:
 386     [false, "\v. d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 387       "Manifest invalid for stream 1: missing or invalid stream name \"\\v."],
 388     [false, "./foo\vbar d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 389       "Manifest invalid for stream 1: missing or invalid stream name \"./foo\\vbar"],
 390     [false, "\t. d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 391       "Manifest invalid for stream 1: missing or invalid stream name \"\\t"],
 392     [false, ".\td41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 393       "Manifest invalid for stream 1: missing or invalid stream name \".\\t"],
 394     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\t\n",
 395       "stream 1: invalid file token \"0:0:foo.txt\\t\""],
 396     [false, ". d41d8cd98f00b204e9800998ecf8427e+0\t 0:0:foo.txt\n",
 397       "stream 1: missing or invalid locator \"d41d8cd98f00b204e9800998ecf8427e+0\\t\""],
 398     [false, "./foo\tbar d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 399       "stream 1: missing or invalid stream name \"./foo\\tbar\""],
 400     # other whitespace errors:
 401     [false, ". d41d8cd98f00b204e9800998ecf8427e+0  0:0:foo.txt\n",
 402       "Manifest invalid for stream 1: invalid file token \"\""],
 403     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n \n",
 404       "Manifest invalid for stream 2: missing stream name"],
 405     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n\n",
 406       "Manifest invalid for stream 2: missing stream name"],
 407     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n ",
 408       "Invalid manifest: does not end with newline"],
 409     [false, "\n. d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 410       "Manifest invalid for stream 1: missing stream name"],
 411     [false, " \n. d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 412       "Manifest invalid for stream 1: missing stream name"],
 413     # empty file and stream name components:
 414     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:/foo.txt\n",
 415       "Manifest invalid for stream 1: invalid file token \"0:0:/foo.txt\""],
 416     [false, "./ d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 417       "Manifest invalid for stream 1: missing or invalid stream name \"./\""],
 418     [false, ".//foo d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 419       "Manifest invalid for stream 1: missing or invalid stream name \".//foo\""],
 420     [false, "./foo/ d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 421       "Manifest invalid for stream 1: missing or invalid stream name \"./foo/\""],
 422     [false, "./foo//bar d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo.txt\n",
 423       "Manifest invalid for stream 1: missing or invalid stream name \"./foo//bar\""],
 424     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo//bar.txt\n",
 425       "Manifest invalid for stream 1: invalid file token \"0:0:foo//bar.txt\""],
 426     [false, ". d41d8cd98f00b204e9800998ecf8427e+0 0:0:foo/\n",
 427       "Manifest invalid for stream 1: invalid file token \"0:0:foo/\""],
 428   ].each do |ok, manifest, expected_error=nil|
 429     define_method "test_validate manifest #{manifest.inspect}" do
 430       assert_equal ok, Keep::Manifest.valid?(manifest)
 431       if ok
 432         assert Keep::Manifest.validate! manifest
 433       else
 434         begin
 435           Keep::Manifest.validate! manifest
 436         rescue ArgumentError => e
 437           msg = e.message
 438         end
 439         refute_nil msg, "Expected ArgumentError"
 440         assert msg.include?(expected_error), "Did not find expected error message. Expected: #{expected_error}; Actual: #{msg}"
 441       end
 442     end
 443   end
 444 end