search

boostorg / wave

Boost.org wave module
http://boost.org/libs/wave
21 stars 49 forks source link

Fix missing (uncalled) exception hooks #166

Closed jefftrull closed 2 years ago

jefftrull commented 2 years ago

This PR does two main things:

  1. Ensures we check the hook results in unit tests after an expected error occurs, as well as when no error occurs. We were missing some test coverage.
  2. Fixes a bug the freshened tests reveal: that sometimes hooks are not called (which is also the subject of #161, which this PR resolves). Thanks to @abakhirkin for the fix.