You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2020/03/03 20:26:50 UTC

[GitHub] [commons-codec] nhojpatrick opened a new pull request #42: Codec 285 base32 input stream test test read null

nhojpatrick opened a new pull request #42: Codec 285 base32 input stream test test read null
URL: https://github.com/apache/commons-codec/pull/42
 
 
   Follow up for PR https://github.com/apache/commons-codec/pull/39
   
   From experience people write tests that don't check exceptions correctly, either they are not thrown, or a different than expected line throws that exception, or it's the wrong exception, or wrong message.
   
   Using assertThrows to explicitly check the expected line throws the expected exception.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-codec] nhojpatrick commented on issue #42: Codec 285 base32 input stream test test read null

Posted by GitBox <gi...@apache.org>.
nhojpatrick commented on issue #42: Codec 285 base32 input stream test test read null
URL: https://github.com/apache/commons-codec/pull/42#issuecomment-594157692
 
 
   If easier I can collapse these into the CODEC-285 PR

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-codec] coveralls commented on issue #42: Codec 285 base32 input stream test test read null

Posted by GitBox <gi...@apache.org>.
coveralls commented on issue #42: Codec 285 base32 input stream test test read null
URL: https://github.com/apache/commons-codec/pull/42#issuecomment-594154289
 
 
   
   [![Coverage Status](https://coveralls.io/builds/29108656/badge)](https://coveralls.io/builds/29108656)
   
   Coverage remained the same at 93.84% when pulling **ba00181f48cf2c915a08ba99afe64f9afeceacc8 on nhojpatrick:CODEC-285_Base32InputStreamTest_testReadNull** into **126f90424cc8f62d928686ec20c6c8bd944b527b on apache:master**.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services