encore icon indicating copy to clipboard operation
encore copied to clipboard

Cleaned up stress tests.

Open supercooldave opened this issue 7 years ago • 6 comments

Added entries into IGNORED_FILES.grep to reduce number of warning produced when running stress tests.

Made one Makefile a little more robust.

supercooldave avatar Jun 22 '17 12:06 supercooldave

Listing each directory explicitly under stress seems an unnecessary burden to maintain, for we know that they are stress tests, not ordinary tests. (Doesn't the ignore scheme in the testing script support whole directory exclusion?)

Using explicit relative path while invoking the compiler seems a bit odd as well. ../../../../../release/encorec -O3 main.enc

albertnetymk avatar Jun 22 '17 23:06 albertnetymk

If you don't mention the directories, then the screen is filled with warnings, though the tests will still pass. It was decided not to suppress such warnings and to force you, the Encore developer, to explicitly add entries to IGNORED_FILES.grep, to avoid accidentally submitting a test file (.enc) without the corresponding other files.

I added the explicit path to the encore compiler to make sure that it was running the right one — perhaps this is redundant and handled already by the testing script.

supercooldave avatar Jun 23 '17 04:06 supercooldave

@kikofernandez Ping!

supercooldave avatar Jul 09 '17 18:07 supercooldave

@kikofernandez Are you fishing for more cake?

supercooldave avatar Jul 13 '17 06:07 supercooldave

CI fails

Sent from my Samsung GT-I9506 using FastHub

kikofernandez avatar Jul 13 '17 10:07 kikofernandez

CI fails because I keep updating src/tests/IGNORED_FILES.grep whenever I add new (stress) tests. I'll fix soon and checkout that Makefile.

supercooldave avatar Jul 13 '17 11:07 supercooldave