lukechilds/node - node - Gitea: Git with a cup of tea

Commit Graph

Author	SHA1	Message	Date
Rich Trott	063c9f6886	test: fix flaky test-cluster-shared-leak Wait for worker2 to come online before doing anything that might result in an EPIPE. Fixes flakiness of test on Windows. Fixes: https://github.com/nodejs/node/issues/3956 PR-URL: https://github.com/nodejs/node/pull/4510 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell<jasnell@gmail.com>	9 years ago
Rich Trott	a96d427e08	test: remove flaky designations for tests Three tests designated as flaky on Linux have not failed on the continuous integration server in a long time. Removing flaky designation for these tests. Fixes: https://github.com/nodejs/node/issues/4446 PR-URL: https://github.com/nodejs/node/pull/4519 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Brian White <mscdex@mscdex.net>	9 years ago
Brian White	3809126427	test: fix flaky cluster-net-send Before this commit, it was possible on Windows for the server's 'connection' handler to be called after the client socket's 'connect' handler. This caused the 'message' event to be missed and the test would never end (timing out in CI). This problem was more easily reproducible on a low resource (slow CPU) Windows (2012r2) installation. This commit waits until both handlers have been called before sending the handle to the master process. Fixes: https://github.com/nodejs/node/issues/3957 PR-URL: https://github.com/nodejs/node/pull/4444 Reviewed-By: Rich Trott <rtrott@gmail.com>	9 years ago
Brian White	30c0062dd2	test: fix flaky child-process-fork-regr-gh-2847 Windows would die with ECONNRESET most times when running this particular test. This commit makes handling these errors more tolerable. PR-URL: https://github.com/nodejs/node/pull/4442 Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>	9 years ago
Rich Trott	d09c447ef9	test: mark test-debug-no-context is flaky test-debug-no-context is flaky on Windows. Mark as such. Ref: https://github.com/nodejs/node/issues/4343 PR-URL: https://github.com/nodejs/node/pull/4421 Reviewed-By: Brian White <mscdex@mscdex.net>	9 years ago
Rich Trott	afd512253d	test: mark test-cluster-shared-leak flaky test-cluster-shared-leak is flaky on Windows. Refs: https://github.com/nodejs/node/issues/3956 PR-URL: https://github.com/nodejs/node/pull/4162 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Roman Klauke <romaaan.git@gmail.com> Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>	9 years ago
Rich Trott	c79b145abc	test: fix flaky test-net-socket-local-address test-net-socket-local-address had a race condition that resulted in unreliability on FreeBSD and Windows. This changes fixes the issue. Fixes: https://github.com/nodejs/node/issues/2475 PR-URL: https://github.com/nodejs/node/pull/4109 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu>	9 years ago
Rich Trott	e25f8683f1	test: mark test flaky on FreeBSD Mark test-net-socket-local-address flaky on FreeBSD. It times out with some frequency on CI. Ref: https://github.com/nodejs/node/issues/2475 PR-URL: https://github.com/nodejs/node/pull/4016 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	9 years ago
Rich Trott	16db4c4d5e	test: mark cluster-net-send test flaky on windows See https://github.com/nodejs/node/issues/3957 for details and examples failures. Ref: https://github.com/nodejs/node/issues/3957 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> PR-URL: https://github.com/nodejs/node/pull/4006	9 years ago
Rich Trott	fc47e0f8cb	test: mark fork regression test flaky on windows See https://github.com/nodejs/node/issues/3635 for details and failure examples. Ref: https://github.com/nodejs/node/issues/3635 PR-URL: https://github.com/nodejs/node/pull/4005 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>	9 years ago
Rich Trott	b061e3af55	test: skip test if in FreeBSD jail Test test-net-socket-local-address is flaky in FreeBSD jail but robust otherwise. Fixes: https://github.com/nodejs/node/issues/2475 PR-URL: https://github.com/nodejs/node/pull/3995 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>	9 years ago
Rich Trott	658842280d	test: remove flaky status for cluster test The test did not fail after 9999 runs in continuous integration. Remove it's flaky status. Fixes: https://github.com/nodejs/node/issues/2557 PR-URL: https://github.com/nodejs/node/pull/3975 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Roman Klauke <romaaan.git@gmail.com>	9 years ago
Rich Trott	0966ab9996	src: force line buffering for stderr SmartOS does not line buffer stderr by default, or at least that is the behavior on the Node project Jenkins server. Force line buffering. This resolves the flakiness observed on SmartOS for test-debug-signal-cluster. PR-URL: https://github.com/nodejs/node/pull/3701 Fixes: https://github.com/nodejs/node/issues/2476 Refs: https://github.com/nodejs/node/pull/3615 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	9 years ago
Rich Trott	f6230b9b72	test: remove flaky designation from ls-no-sslv3 This test was marked flaky after failing in CI on arm7-wheezy two months ago. It has not failed there since. This commit removes the flaky designation. Fixes: https://github.com/nodejs/node/issues/2554 PR-URL: https://github.com/nodejs/node/pull/3620 Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>	9 years ago
Matt Loring	fed0b28754	tools: fix flakiness in test-tick-processor Per the discussion on #2471, the JS symbols checked for by this test were occasionally too deep in the stack and were being ignored by the tick processor. I have addressed this by increasing the stack depth inspected by the tick processor and looking for the eval symbol which is more likely to be present. Additional flakiness was caused by occasional misses of the code creation event for the JS function being executed. I now have separate code snippets to test for JS and C++ symbols and if the code creation event is missed for the JS symbol test then I check for a percentage of UNKNOWN symbols in processed output. This is considered a success as the processing scripts in the node repository are still correctly processing the ticks recieved from the v8 scripts. Further investigation is needed into the v8 profiling scripts to determine why code creation events are being missed. PR-URL: https://github.com/nodejs/node/pull/2694 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	9 years ago
Rich Trott	750adddd15	test: unmark test-process-argv-0.js as flaky https://github.com/nodejs/node/pull/2541 fixed flakiness in test-process-argv-0.js. However, it was not removed from the list of flaky tests. This removes it from the list of flaky tests. PR-URL: https://github.com/nodejs/node/pull/2613 Reviewed-By: cjihrig - Colin Ihrig <cjihrig@gmail.com>	9 years ago
Alexis Campailla	34e25b2313	test: initial list of flaky tests This poplulates the lists of flaky tests with tests that failed recently in Jenkins. PR-URL: https://github.com/nodejs/node/pull/2424 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>	9 years ago
Alexis Campailla	ff05953b8d	test: add test configuration templates Template configuration files for marking tests as flaky. PR-URL: https://github.com/nodejs/node/pull/2424 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: João Reis <reis@janeasystems.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>	9 years ago

18 Commits (6588a70463674a361b9a19f9f50ae91483fd82f4)