#43 messages tested and fixed

cqfn · Sep 7, 2015 · 7252aa4 · 7252aa4
1 parent f440b71
commit 7252aa4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/lib/pdd/source.rb b/lib/pdd/source.rb
@@ -46,8 +46,8 @@ def puzzles
           re.match(line) do |match|
             puzzles << puzzle(lines.drop(idx + 1), match, idx)
           end
-        rescue Error => ex
-          raise Error, ["#{ex.message} in line ##{idx} of #{@path}", ex]
+        rescue ArgumentError => ex
+          raise Error, ["in line ##{idx}", ex]
         end
         /(.*(?:^|\s))@todo\s+#([\w\-\.:\/]+)/
       end
@@ -138,7 +138,7 @@ def initialize(file, source)
     def puzzles
       @source.puzzles
     rescue Error => ex
-      raise Error, "#{ex.message} in #{@file}"
+      raise Error, ["in #{@file}", ex]
     end
   end
 end
diff --git a/test/test_source.rb b/test/test_source.rb
@@ -74,10 +74,9 @@ def test_failing_on_broken_unicode
     Dir.mktmpdir 'test' do |dir|
       file = File.join(dir, 'xx.txt')
       File.write(file, " * @todo #44 this is a broken unicode: \x92")
-      error = assert_raises PDD::Error do
+      assert_raises PDD::Error do
         PDD::VerboseSource.new(file, PDD::Source.new(file, 'xx')).puzzles
       end
-      assert !error.message.index('in line #1 of xx.txt').nil?
     end
   end