--- Day changed Wed Sep 19 2018 00:01 < BlueMatt> ariard: its a very clever concept - you shouldnt just have line coverage in your tests, but if anything being tested gets changed, your tests should fail 00:01 < BlueMatt> otherwise your tests weren't really checking that line 00:37 < ariard> I've had a quick skim on README, it's kind of applying ockham razor to your test, really cool 00:38 < ariard> do you want it for #184 ? i prefer to get #138 done, other 0.1 issues and then 0.2 no ? 01:33 < ariard> BlueMatt: is there a way in rust-bitcoin to get size of redeemScript from witness input ? We could use it to assert that target tx spend received/offered htlc outputs ? 01:34 < ariard> we can't assume on input order in commitment tx due to bip 69 07:38 -!- gribble [~gribble@unaffiliated/nanotube/bot/gribble] has quit [Disconnected by services] 07:45 -!- gribble [~gribble@unaffiliated/nanotube/bot/gribble] has joined #rust-bitcoin 09:59 -!- dongcarl [uid321684@gateway/web/irccloud.com/x-rmabtpzhpydqszuw] has joined #rust-bitcoin 10:02 < dongcarl> IRC proxy's been acting up... Back now... 10:03 < dongcarl> #103 and #156 are ready 15:00 < BlueMatt> ariard: the first witness stack element is the redeemScript, I think? I mean it depends on the witness type but its either the redeemScript or the pubkey 18:50 -!- schmidty [~schmidty@unaffiliated/schmidty] has joined #rust-bitcoin 22:47 -!- smoke3 [~snypas@2601:587:0:5bdb:5001:dfd8:1ba1:f6cc] has joined #rust-bitcoin