Skip to content

Commit

Permalink
mark 1kHz tests as skipped
Browse files Browse the repository at this point in the history
  • Loading branch information
mikaelarguedas committed Aug 15, 2018
1 parent 88ff1a2 commit 39bf6db
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions rclpy/test/test_timer.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
import multiprocessing
import os
import platform
import pytest
import sys
import time
import traceback
Expand Down Expand Up @@ -152,6 +153,7 @@ def test_timer_zero_callbacks100hertz():
# TODO(mikaelarguedas) reenable these once timer have consistent behaviour
# on every platform at high frequency
# TODO(sloretz) Reenable on arm when executor performance is good enough
@pytest.mark.skip(reason="1kHz tests are too prone to flakiness")
def test_timer_zero_callbacks1000hertz():
if os.name == 'nt' or platform.machine() == 'aarch64':
raise SkipTest
Expand All @@ -167,6 +169,7 @@ def test_timer_number_callbacks100hertz():
func_number_callbacks, ['0.01'], "didn't receive the expected number of callbacks")


@pytest.mark.skip(reason="1kHz tests are too prone to flakiness")
def test_timer_number_callbacks1000hertz():
if os.name == 'nt' or platform.machine() == 'aarch64':
raise SkipTest
Expand All @@ -184,6 +187,7 @@ def test_timer_cancel_reset_100hertz():
func_cancel_reset_timer, ['0.01'], "didn't receive the expected number of callbacks")


@pytest.mark.skip(reason="1kHz tests are too prone to flakiness")
def test_timer_cancel_reset_1000hertz():
if os.name == 'nt' or platform.machine() == 'aarch64':
raise SkipTest
Expand Down

0 comments on commit 39bf6db

Please sign in to comment.