From 758a74f8bddbc57a73b6b212291c5095f0c71faa Mon Sep 17 00:00:00 2001
From: Mikael Arguedas <mikael@osrfoundation.org>
Date: Tue, 14 Aug 2018 17:00:31 -0700
Subject: [PATCH] disable 1kHz test on all platforms

---
 rclpy/test/test_timer.py | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/rclpy/test/test_timer.py b/rclpy/test/test_timer.py
index 9376f0ff5..6118f8d4d 100644
--- a/rclpy/test/test_timer.py
+++ b/rclpy/test/test_timer.py
@@ -13,8 +13,9 @@
 # limitations under the License.
 
 import multiprocessing
-import os
-import platform
+# TODO(mikaelarguedas) comment back in if we reenable 1kHz test for some platforms
+# import os
+# import platform
 import sys
 import time
 import traceback
@@ -153,9 +154,10 @@ def test_timer_zero_callbacks100hertz():
 # on every platform at high frequency
 # TODO(sloretz) Reenable on arm when executor performance is good enough
 def test_timer_zero_callbacks1000hertz():
-    if os.name == 'nt' or platform.machine() == 'aarch64':
-        raise SkipTest
-    func_launch(func_zero_callback, ['0.001'], 'received callbacks when not expected')
+    # if os.name == 'nt' or platform.machine() == 'aarch64':
+    #     raise SkipTest
+    # func_launch(func_zero_callback, ['0.001'], 'received callbacks when not expected')
+    raise SkipTest
 
 
 def test_timer_number_callbacks10hertz():
@@ -168,10 +170,11 @@ def test_timer_number_callbacks100hertz():
 
 
 def test_timer_number_callbacks1000hertz():
-    if os.name == 'nt' or platform.machine() == 'aarch64':
-        raise SkipTest
-    func_launch(
-        func_number_callbacks, ['0.001'], "didn't receive the expected number of callbacks")
+    # if os.name == 'nt' or platform.machine() == 'aarch64':
+    #     raise SkipTest
+    # func_launch(
+    #     func_number_callbacks, ['0.001'], "didn't receive the expected number of callbacks")
+    raise SkipTest
 
 
 def test_timer_cancel_reset_10hertz():
@@ -185,7 +188,8 @@ def test_timer_cancel_reset_100hertz():
 
 
 def test_timer_cancel_reset_1000hertz():
-    if os.name == 'nt' or platform.machine() == 'aarch64':
-        raise SkipTest
-    func_launch(
-        func_cancel_reset_timer, ['0.001'], "didn't receive the expected number of callbacks")
+    # if os.name == 'nt' or platform.machine() == 'aarch64':
+    #     raise SkipTest
+    # func_launch(
+    #     func_cancel_reset_timer, ['0.001'], "didn't receive the expected number of callbacks")
+    raise SkipTest