Don't throw an exception at Future#await

Watson1978 · Feb 8, 2024 · 19fbb08 · 19fbb08
1 parent a46226b
commit 19fbb08
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/ext/ilios/future.c b/ext/ilios/future.c
@@ -303,7 +303,7 @@ static VALUE future_await(VALUE self)
     GET_FUTURE(self, cassandra_future);
 
     if (cassandra_future->already_waited) {
-        rb_raise(eExecutionError, "It should not call twice");
+        return self;
     }
 
     if (cassandra_future->on_success_block || cassandra_future->on_failure_block) {

diff --git a/test/test_future.rb b/test/test_future.rb
@@ -52,7 +52,6 @@ def test_await
     prepare_future.await
 
     assert_equal(50, count)
-    assert_raises(Ilios::Cassandra::ExecutionError) { prepare_future.await }
   end
 
   def test_on_success