Merge pull request #2 from h3nrique/main

Update to avoid casts.

Author: sleipnir

src/main/java/io/eigr/spawn/api/ActorRef.java

@@ -91,8 +91,8 @@ protected static ActorRef of(SpawnClient client, String system, String name, Str
      * @return an Optional containing, or not, the response object to the Action call
      * @since 0.0.1
      */
-    public <T extends GeneratedMessageV3> Optional<Object>  invoke(String action, Class<T> outputType) throws Exception {
-        Optional<Object> res = invokeActor(action, Empty.getDefaultInstance(), outputType, Optional.empty());
+    public <T extends GeneratedMessageV3> Optional<T>  invoke(String action, Class<T> outputType) throws Exception {
+        Optional<T> res = invokeActor(action, Empty.getDefaultInstance(), outputType, Optional.empty());
         if(res.isPresent() ){
             return Optional.of(outputType.cast(res.get()));
         }
@@ -111,8 +111,8 @@ public <T extends GeneratedMessageV3> Optional<Object>  invoke(String action, Cl
      * @return an Optional containing, or not, the response object to the Action call
      * @since 0.0.1
      */
-    public <T extends GeneratedMessageV3> Optional<Object>  invoke(String action, Class<T> outputType, InvocationOpts opts) throws Exception {
-        Optional<Object> res = invokeActor(action, Empty.getDefaultInstance(), outputType, Optional.ofNullable(opts));
+    public <T extends GeneratedMessageV3> Optional<T>  invoke(String action, Class<T> outputType, InvocationOpts opts) throws Exception {
+        Optional<T> res = invokeActor(action, Empty.getDefaultInstance(), outputType, Optional.ofNullable(opts));
         if(res.isPresent() ){
             return Optional.of(outputType.cast(res.get()));
         }
@@ -130,8 +130,8 @@ public <T extends GeneratedMessageV3> Optional<Object>  invoke(String action, Cl
      * @return an Optional containing, or not, the response object to the Action call
      * @since 0.0.1
      */
-    public <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<Object> invoke(String action, S value, Class<T> outputType) throws Exception {
-        Optional<Object> res = invokeActor(action, value, outputType, Optional.empty());
+    public <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<T> invoke(String action, S value, Class<T> outputType) throws Exception {
+        Optional<T> res = invokeActor(action, value, outputType, Optional.empty());
         if(res.isPresent() ){
             return Optional.of(outputType.cast(res.get()));
         }
@@ -151,8 +151,8 @@ public <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<Obj
      * @return an Optional containing, or not, the response object to the Action call
      * @since 0.0.1
      */
-    public <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<Object> invoke(String action, S value, Class<T> outputType, InvocationOpts opts) throws Exception {
-        Optional<Object> res = invokeActor(action, value, outputType, Optional.ofNullable(opts));
+    public <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<T> invoke(String action, S value, Class<T> outputType, InvocationOpts opts) throws Exception {
+        Optional<T> res = invokeActor(action, value, outputType, Optional.ofNullable(opts));
         if(res.isPresent() ){
             return Optional.of(outputType.cast(res.get()));
         }
@@ -248,7 +248,7 @@ public boolean isUnNamedActor() {
         return false;
     }
 
-    private <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<Object> invokeActor(
+    private <T extends GeneratedMessageV3, S extends GeneratedMessageV3> Optional<T> invokeActor(
             String cmd, S argument, Class<T> outputType, Optional<InvocationOpts> options) throws Exception {
         Protocol.InvocationRequest.Builder invocationRequestBuilder = Protocol.InvocationRequest.newBuilder();
 

src/test/java/io/eigr/spawn/SpawnTest.java

@@ -42,11 +42,11 @@ public void testApp() throws Exception {
                 .setLanguage("erlang")
                 .build();
 
-        Optional<Object> maybeReply =
+        Optional<Actor.Reply> maybeReply =
                 joeActor.invoke("setLanguage", msg, Actor.Reply.class);
 
         if (maybeReply.isPresent()) {
-            Actor.Reply reply = (Actor.Reply) maybeReply.get();
+            Actor.Reply reply = maybeReply.get();
             assertNotNull(reply);
             assertEquals("Hello From Java", reply.getResponse());
         }