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

Commit Graph

Author	SHA1	Message	Date
Rich Trott	4b43bf0385	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	c50003746b	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	054a216b6f	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	d0621c5649	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	19ed33df80	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	a863e8d667	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	02e012e984	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	5977963bce	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>	10 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>	10 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>	10 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>	10 years ago

12 Commits (4c8800c2dec2fc8d0a6b3cf71f22fb7b69e6c83b)