diff options
| author | Philip Chimento <philip.chimento@gmail.com> | 2018-08-20 22:28:52 -0400 |
|---|---|---|
| committer | Philip Chimento <philip.chimento@gmail.com> | 2018-09-01 19:25:50 -0400 |
| commit | cf57f25f3b9fca36d699b54fe2892304968b14f7 (patch) | |
| tree | aac91394cd2bcb94bd0aadc94ad15bd72f92a427 /test/gjs-tests.cpp | |
| parent | 284586e60cf2451060be58113bc1857790f7c595 (diff) | |
| download | gjs-cf57f25f3b9fca36d699b54fe2892304968b14f7.tar.gz | |
tests: Don't time-limit the unit tests
Previously there was code that would intentionally crash if more than 7
minutes elapsed, in case the tests got stuck on autobuilders. This code
is more harmful than helpful, see the following table:
Times it has killed a stuck test on GitLab: . . . . . . . . 0
Times it has killed a JS_GC_ZEAL test: . . . . . . . . . 34
Times it has killed my GDB session because I forgot
to add the stupid environment variable: . . . . . . . . 478
Diffstat (limited to 'test/gjs-tests.cpp')
| -rw-r--r-- | test/gjs-tests.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/test/gjs-tests.cpp b/test/gjs-tests.cpp index 820be6fa..e482c80f 100644 --- a/test/gjs-tests.cpp +++ b/test/gjs-tests.cpp @@ -406,11 +406,6 @@ int main(int argc, char **argv) { - /* give the unit tests 7 minutes to complete, unless an environment variable - * is set; use this when running under GDB, for example */ - if (!g_getenv("GJS_TEST_SKIP_TIMEOUT")) - gjs_crash_after_timeout(60 * 7); - /* Avoid interference in the tests from stray environment variable */ g_unsetenv("GJS_ENABLE_PROFILER"); |