Merge remote-tracking branch 'remotes/jeremie/better-errors' into merge-better-errs-pr

Author: a2800276

lib/iso8583/message.rb

@@ -343,15 +343,19 @@ def bmp_alias(bmp, aliaz)
           # @values[bmp] = bmp_def
         }
       end
-      
+
       # Parse the bytes `str` returning a message of the defined type.
+      #
+      # returns an instance of Message
+      #
+      # will raise an ISO8583Exception if the bitmap definition can't be found
       def parse(str)
         str = str.force_encoding('ASCII-8BIT')
         message = self.new
         message.mti, rest = _mti_format.parse(str)
         bmp,rest = Bitmap.parse(rest)
         bmp.each {|bit|
-          bmp_def      = _definitions[bit]
+          bmp_def      = message._get_definition(bit)
           value, rest  = bmp_def.field.parse(rest)
           message[bit] = value
         }

test/message_test.rb

@@ -45,6 +45,12 @@ def test_parse
     assert_equal 1430, mes.mti
   end
 
+  def test_unknown_field
+    assert_raises(ISO8583Exception.new("no definition for field: 8")) {
+      mes = BerlinMessage.parse "1430A\000\000\000\000\000\000\00012474747474747"
+    }
+  end
+
   def test_rescue_standard_error
     rescued = false
     begin