Home Original page

test: increase dgram timeout for armv6 by Trott · Pull Request #2808 · nodejs/node

@Trott 

test-dgram-broadcast-multi-process.js and
test-dgram-multicast-multi-process.js were failing on Pi 1 because the
test was timing out. Changed static 5000ms timeout to a dynamically
determined timeout based on the processor using
common.platformTimeout().

@ChALkeR added test

Issues and PRs related to the tests.

arm

Issues and PRs related to the ARM platform.

labels

Sep 11, 2015

Trott added a commit that referenced this pull request

Sep 12, 2015 
test-dgram-broadcast-multi-process.js and
test-dgram-multicast-multi-process.js were failing on Pi 1 because the
test was timing out. Changed static 5000ms timeout to a dynamically
determined timeout based on the processor using
common.platformTimeout().

PR-URL: #2808
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>

@Trott Trott mentioned this pull request

Sep 14, 2015

Trott added a commit that referenced this pull request

Sep 15, 2015 
test-dgram-broadcast-multi-process.js and
test-dgram-multicast-multi-process.js were failing on Pi 1 because the
test was timing out. Changed static 5000ms timeout to a dynamically
determined timeout based on the processor using
common.platformTimeout().

PR-URL: #2808
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>

Trott added a commit that referenced this pull request

Sep 15, 2015 
test-dgram-broadcast-multi-process.js and
test-dgram-multicast-multi-process.js were failing on Pi 1 because the
test was timing out. Changed static 5000ms timeout to a dynamically
determined timeout based on the processor using
common.platformTimeout().

PR-URL: #2808
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>

@rvagg rvagg mentioned this pull request

Sep 15, 2015

@rvagg rvagg mentioned this pull request

Sep 22, 2015

@Trott Trott deleted the test-dgram-fixes branch

January 9, 2022 22:03