Missing metrics for failed tests about pytest-monitor HOT 8 OPEN

Hi @Veritogen / @Diego-Hernandez-Moodys

Sorry for the long delay. I have reworked the plugin a bit to be able to support this feature without having everything break.
I expect to deliver it by end of week. I need to solve a bias in memory snapshot first but I don't expect it to be very long.

from pytest-monitor.

Same issue here.
In my current setup it would be especially intressting to see metrics in tests that fail.

from pytest-monitor.

Hello

Thanks for submitting. Failed tests are filtered out on purpose (primarily designed that way to get consistent metrics).
However, the subject has been previously raised in #27 and at that time decision has been taken to postpone it.

I think it may be time for that improvement. Now the question game:

• shall we add test status (failed or succeeded) or not? I personally tend to think that the status is of interest for those willing to filter out failed test.
• shall we add an option to activate that behavior (by extension, the current behavior will be the default one) or should we consider it default?

from pytest-monitor.

@js-dieu It's useful because I expect to use this to test for memory leaks. To answer your questions

• Yes, add the status to have parity with the pytest output
• It should be on by default since that's what the user would see in the pytest output. They should have an option to turn off the plugin, though.
  One row for each test case please.

from pytest-monitor.

I'd also like to see the results of failed tests in the database by default.

@js-dieu If you can point me to where changes are needed to accomplish that and if its not too much of implementation effort, my employer allows me to implement this kind of stuff during my work time.

from pytest-monitor.

@js-dieu awesome, thats great to hear. Let me know if there's something you could need help with.
I also made a prove of concept for a DBHandler that can handle postgres for persisting the test results. Should I raise a PR for that?

from pytest-monitor.

Is there any progress on that? The feature would be quite handy.

from pytest-monitor.

As the discussion froze and @js-dieu seems to be busy, I decided to do my own changes locally. I'm gonna summarize my insights below so in case this feature will be implemented in the future.

• I added a new table column (boolean) "TEST_PASSED" to the METRICS table to reflect if a test passed or not and changed the wrapper functions accordingly
• As the memory_profiler module is not maintained anymore and the company I work is not able to take maintainership, I stripped out and simplified the logic needed for memory measurement and put it in a new module profiler.py inside of this project.
• I changed the functions calls from the plugin to the profiler accordingly and introduced a new attribute "passed" in the pytest node to be able to hand through the information to the database

Of course I would be open to make a PR if you want, but I'm not sure if I did it the way it was intended by you.

from pytest-monitor.