pinterest / elixometer

A light Elixir wrapper around exometer.

Home Page:https://hexdocs.pm/elixometer

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

What is the intentional use of elixometer.TestReporter?

linearregression opened this issue · comments

There is a TestReport In file https://github.com/pinterest/elixometer/blob/master/lib/elixometer.ex

I notice that is is being used in test:
https://github.com/pinterest/elixometer/blob/master/config/test.exs

But is that just for Test environment, but for some reason, it si part of the main module?
If production won't ever use or it, would suggest this should not be part of main module.

The idea is that you use that inside tests if you want to get the output of logging. Elixiometer was written pretty early in our elixir exploration, and this should be made top-level. Better yet, we could alter the mixfile to look in a different directory and remove it entirely from lib and put into test/support or something.