From 6bbfbd7aeb99475308a140222316f3e9006aeec3 Mon Sep 17 00:00:00 2001
From: Fabian Mastenbroek <mail.fabianm@gmail.com>
Date: Tue, 1 Nov 2022 10:38:26 +0100
Subject: refactor(compute/api): Do not suspend in compute API

This change updates the API interface of the OpenDC Compute service to
not suspend execution using Kotlin Coroutines.

The suspending modifiers were introduced in case the ComputeClient would
communicate with the service over a network connection. However, the main
use-case has been together with the ComputeService, where the suspending
modifiers only frustrate the user experience when writing experiments.

Furthermore, with the advent of Project Loom, it is not necessarily a
problem to block the (virtual) thread during network communications.
---
 .../kotlin/org/opendc/compute/simulator/SimHostTest.kt   | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

(limited to 'opendc-compute/opendc-compute-simulator/src/test')

diff --git a/opendc-compute/opendc-compute-simulator/src/test/kotlin/org/opendc/compute/simulator/SimHostTest.kt b/opendc-compute/opendc-compute-simulator/src/test/kotlin/org/opendc/compute/simulator/SimHostTest.kt
index a496cc99..1734daf5 100644
--- a/opendc-compute/opendc-compute-simulator/src/test/kotlin/org/opendc/compute/simulator/SimHostTest.kt
+++ b/opendc-compute/opendc-compute-simulator/src/test/kotlin/org/opendc/compute/simulator/SimHostTest.kt
@@ -305,11 +305,11 @@ internal class SimHostTest {
         override val labels: Map<String, String> = emptyMap()
         override val meta: Map<String, Any> = emptyMap()
 
-        override suspend fun delete() {
+        override fun delete() {
             throw NotImplementedError()
         }
 
-        override suspend fun refresh() {
+        override fun reload() {
             throw NotImplementedError()
         }
     }
@@ -320,11 +320,11 @@ internal class SimHostTest {
         override val labels: Map<String, String>,
         override val meta: Map<String, Any>
     ) : Image {
-        override suspend fun delete() {
+        override fun delete() {
             throw NotImplementedError()
         }
 
-        override suspend fun refresh() {
+        override fun reload() {
             throw NotImplementedError()
         }
     }
@@ -343,16 +343,16 @@ internal class SimHostTest {
 
         override val launchedAt: Instant? = null
 
-        override suspend fun start() {}
+        override fun start() {}
 
-        override suspend fun stop() {}
+        override fun stop() {}
 
-        override suspend fun delete() {}
+        override fun delete() {}
 
         override fun watch(watcher: ServerWatcher) {}
 
         override fun unwatch(watcher: ServerWatcher) {}
 
-        override suspend fun refresh() {}
+        override fun reload() {}
     }
 }
-- 
cgit v1.2.3