Don't fail tests as a logging side-effect

Open ranweiler opened this issue 7 years ago • 2 comments

This relationship should be inverted. The current implementation was just convenient when implementing assertions that should trigger failures. See the commit message for d81cbba13788b262e2d10a8da35ca88121b22d9f.

Feb 11 '18 00:02 ranweiler

Note that this also occurs in common.py in the DeepState::api_log_stream() method.

Feb 14 '18 18:02 ranweiler

See this commit, and keep in mind when addressing #48: https://github.com/trailofbits/deepstate/commit/a81f816d89182e49aeba1a9ada4651d662f33871

Feb 21 '18 00:02 ranweiler