Ensure we use the same GRPC version everywhere (#1137)

* Ensure we use the same GRPC version everywhere

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Fix actors tests assert

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Revert Dapr exception asserts

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Increase sleep to allow Spring Context to bootstrap

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Revert sleep value

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Increase the sleep for messaging test

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Move sleep before each, to ensure Spring context starts

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

* Add more delays to ensure Spring Controller gets the messages

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>

---------

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>

Author: artur-ciocanu

pom.xml

@@ -104,13 +104,13 @@
       <dependency>
         <groupId>io.grpc</groupId>
         <artifactId>grpc-testing</artifactId>
-        <version>1.56.1</version>
+        <version>${grpc.version}</version>
         <scope>test</scope>
       </dependency>
       <dependency>
         <groupId>io.grpc</groupId>
         <artifactId>grpc-inprocess</artifactId>
-        <version>1.59.0</version>
+        <version>${grpc.version}</version>
         <scope>test</scope>
       </dependency>
     </dependencies>

sdk-actors/pom.xml

@@ -18,7 +18,6 @@
 
   <properties>
     <maven.deploy.skip>false</maven.deploy.skip>
-    <grpc.version>1.59.0</grpc.version>
   </properties>
 
   <dependencies>

sdk-actors/src/test/java/io/dapr/actors/client/DaprGrpcClientTest.java

@@ -129,7 +129,7 @@ public void invokeException() {
         ExecutionException.class,
         "UNKNOWN",
         "UNKNOWN: ",
-        () -> result.block());
+        result::block);
   }
 
   @Test

sdk-actors/src/test/java/io/dapr/actors/runtime/DaprGrpcClientTest.java

@@ -96,7 +96,7 @@ public void getActorStateException() {
         assertThrowsDaprException(
                 ExecutionException.class,
                 "UNKNOWN",
-                "UNKNOWN: ",
+                "UNKNOWN: Application error processing RPC",
                 result::block);
     }
 
@@ -112,7 +112,7 @@ public void saveActorStateTransactionallyException() {
         assertThrowsDaprException(
                 ExecutionException.class,
                 "UNKNOWN",
-                "UNKNOWN: ",
+                "UNKNOWN: Application error processing RPC",
                 result::block);
     }
     @Test

sdk-tests/src/test/java/io/dapr/it/spring/messaging/DaprSpringMessagingIT.java

@@ -76,10 +76,15 @@ public class DaprSpringMessagingIT {
   private TestRestController testRestController;
 
   @BeforeAll
-  public static void setup(){
+  public static void beforeAll(){
     org.testcontainers.Testcontainers.exposeHostPorts(8080);
   }
 
+  @BeforeEach
+  public void beforeEach() throws InterruptedException {
+    Thread.sleep(1000);
+  }
+
   @Test
   public void testDaprMessagingTemplate() throws InterruptedException {
     for (int i = 0; i < 10; i++) {
@@ -97,5 +102,4 @@ public void testDaprMessagingTemplate() throws InterruptedException {
 
     assertThat(events.size()).isEqualTo(10);
   }
-
 }

sdk/pom.xml

@@ -18,7 +18,6 @@
 
   <properties>
     <maven.deploy.skip>false</maven.deploy.skip>
-    <grpc.version>1.59.0</grpc.version>
     <argLine>
       --add-opens java.base/java.util=ALL-UNNAMED
     </argLine>