nats-io / elixir-nats

Elixir NATS client
MIT License
76 stars 14 forks source link

Capture logs during test #36

Closed mmmries closed 7 years ago

mmmries commented 7 years ago

Use the capture_log feature of exunit to hide noisy error messages during test runs. This will show the log output if the test fails, but hide it when tests are passing.

While I was making this change I also removed some trailing whitespace from the test, added an explicity () in the mix.exs file (avoids compiler warnings in elixir 1.4+) and added the latest erlang and elixir versions to the build matrix.

I'm hoping to open some PRs in the future that add documentation and attempt some performance improvements, but I'm trying to start with basic changes 😄

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 66.338% when pulling 35c8624b6ce39ee7297f3d523d7d6e12ea16d863 on mmmries:capture_logs_during_test into 199678a89b2fe912393b0b9779492365dd219162 on nats-io:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 66.48% when pulling 0f070adb6568419dec4028d8a9dba01d48f93a23 on mmmries:capture_logs_during_test into 23467b162e6ee956cb82f71c538703c0a16bfe10 on nats-io:master.