Fix save-test-data

[murrant]

But now it buffers all output... and we don't seem to save much from the initial attempt
Needs more testing/thought

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

• Have you followed our code guidelines?

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926

[scrutinizer-notifier]

The inspection completed: No new issues

[tracernz]

Just bisected the lack of test data in #8372 to 2bcc766. Is this the fix for that?

git bisect good
2bcc76625ca576aa2aa673b36569c93d2dd758ca is the first bad commit
commit 2bcc76625ca576aa2aa673b36569c93d2dd758ca
Author: Tony Murray <murraytony@gmail.com>
Date:   Wed Mar 14 17:28:01 2018 -0500

    Module tests update: per module output, only store modules data that are run (#8355)
    
    * Test data: don't save data for modules that were not run
    
    * Sometimes mibs are prefixed with +
    
    * Save output for each module and only output the errored module.
    Also makes it easier to identify modules that were ran.
    
    * Clean up json files.
    Message when no module data exists.
    
    * verbose output when phpunit --debug is used
    order by for ports
    avoid graphite output in tests
    only load module_tables.yaml once
    
    * use explode and strpos instead of regex since it is failing...
    
    * Fix some warnings
    
    * Fix whitespace

:040000 040000 0a9bcb49eefa5ca08c81cda16d3b2bfff274983d 66f60f6d56b9c2c955ee24af0d328c88e286677b M	LibreNMS
:040000 040000 2d98427cc4a0bc52d7c6580abf0493a588f14e1c 6314fa93aad8b11fdb6558453b0013b1f857ede2 M	includes
:040000 040000 09800aa63c96da1c829d335cca6e9becd48a00db 78414df3b68055d5cb9170875f86def516a3a05d M	tests

[tracernz]

Yes, I get test data again with this patch. 👍

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed.