merge: Clean up compute service (v1)

This pull request is one in a series of changes that attempt to clean up the 
warts of the OpenDC Compute Service. These changes should make the
API more robust for future developments.

## Implementation Notes :hammer_and_pick:

* Do not suspend in compute API
* Expose state directly to clients
* Do not split interface and implementation

## External Dependencies :four_leaf_clover:

* N/A

## Breaking API Changes :warning:

* The methods of `ComputeClient` do not have the suspend modifier anymore.
* Changes to the classes of `ComputeClient` are now immediately visible on the server side.
   It is not necessary to call `refresh` anymore.
* `ComputeService` must be constructed via its builder class now.

1 files changed, 8 insertions, 8 deletions

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() {}
     }
 }