Swap to basing timeouts on time unit so CI tests get longer timeouts

Author: TrentHouliston

tests/test_util/TestBase.hpp

@@ -28,6 +28,7 @@
 #include <utility>
 
 #include "nuclear"
+#include "test_util/TimeUnit.hpp"     // IWYU pragma: export
 #include "test_util/diff_string.hpp"  // IWYU pragma: export
 
 namespace test_util {
@@ -57,7 +58,7 @@ class TestBase : public NUClear::Reactor {
 private:
     explicit TestBase(std::unique_ptr<NUClear::Environment> environment,
                       const bool& shutdown_on_idle             = true,
-                      const std::chrono::milliseconds& timeout = std::chrono::milliseconds(1000))
+                      const std::chrono::milliseconds& timeout = TimeUnit(20))
         : Reactor(std::move(environment)) {
 
         // Shutdown if the system is idle

tests/tests/dsl/Every.cpp

@@ -39,7 +39,7 @@
 class TestReactor : public test_util::TestBase<TestReactor> {
 public:
     TestReactor(std::unique_ptr<NUClear::Environment> environment)
-        : TestBase(std::move(environment), false, std::chrono::seconds(10)) {
+        : TestBase(std::move(environment), false, test_util::TimeUnit(200)) {
 
         // Trigger on 3 different types of every
         on<Every<1000, Per<std::chrono::seconds>>>().then([this]() { every_times.push_back(NUClear::clock::now()); });

tests/tests/dsl/Idle.cpp

@@ -57,7 +57,7 @@ class TestReactor : public test_util::TestBase<TestReactor> {
     }
 
     explicit TestReactor(std::unique_ptr<NUClear::Environment> environment)
-        : TestBase(std::move(environment), false, std::chrono::seconds(5)) {
+        : TestBase(std::move(environment), false, test_util::TimeUnit(100)) {
 
         start_time = NUClear::clock::now();
 

tests/tests/dsl/IdleSingleGlobal.cpp

@@ -59,7 +59,7 @@ class TestReactor : public test_util::TestBase<TestReactor> {
     static constexpr int n_loops = 10000;
 
     explicit TestReactor(std::unique_ptr<NUClear::Environment> environment)
-        : TestBase(std::move(environment), false, std::chrono::seconds(2)) {
+        : TestBase(std::move(environment), false, test_util::TimeUnit(50)) {
 
         /*
          * Run idle on the default pool, and a task on the main pool.

tests/tests/dsl/SyncOrder.cpp

@@ -39,7 +39,8 @@ class TestReactor : public test_util::TestBase<TestReactor> {
         Message(int val) : val(val) {};
     };
 
-    TestReactor(std::unique_ptr<NUClear::Environment> environment) : TestBase(std::move(environment)) {
+    TestReactor(std::unique_ptr<NUClear::Environment> environment)
+        : TestBase(std::move(environment), true, test_util::TimeUnit(50)) {
 
         on<Trigger<Message<'A'>>, Sync<TestReactor>>().then([this](const Message<'A'>& m) {  //
             events.emplace_back('A', m.val);

tests/tests/dsl/TCP.cpp

@@ -84,7 +84,7 @@ class TestReactor : public test_util::TestBase<TestReactor> {
     }
 
     TestReactor(std::unique_ptr<NUClear::Environment> environment, const std::vector<TestType>& active_tests_)
-        : TestBase(std::move(environment), false, std::chrono::seconds(2)), active_tests(active_tests_) {
+        : TestBase(std::move(environment), false, test_util::TimeUnit(50)), active_tests(active_tests_) {
 
         for (const auto& t : active_tests) {
             switch (t) {

tests/tests/dsl/emit/Delay.cpp

@@ -54,7 +54,7 @@ class TestReactor : public test_util::TestBase<TestReactor> {
 
     struct FinishTest {};
     TestReactor(std::unique_ptr<NUClear::Environment> environment)
-        : TestBase(std::move(environment), false, std::chrono::seconds(2)) {
+        : TestBase(std::move(environment), false, test_util::TimeUnit(50)) {
 
         // Measure when messages were sent and received and print those values
         on<Trigger<DelayedMessage>>().then([this](const DelayedMessage& m) {