Transaction: remove untestable assert

Also changes the fromBuffer sanity test to use a clearer message that is
taken from the fixtures.

src/transaction.js

@@ -144,7 +144,6 @@ Transaction.prototype.toBuffer = function () {
   })
 
   writeUInt32(this.locktime)
-  assert.equal(offset, buffer.length, 'Invalid transaction object')
 
   return buffer
 }
@@ -286,7 +285,7 @@ Transaction.fromBuffer = function(buffer) {
   }
 
   tx.locktime = readUInt32()
-  assert.equal(offset, buffer.length, 'Invalid transaction')
+  assert.equal(offset, buffer.length, 'Transaction has unexpected data')
 
   return tx
 }

test/transaction.js

@@ -36,7 +36,7 @@ describe('Transaction', function() {
       it('throws on ' + f.exception, function() {
         assert.throws(function() {
           Transaction.fromHex(f.hex)
-        }, /Invalid transaction/)
+        }, new RegExp(f.exception))
       })
     })
   })